Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
factor parce_factor
[simgrid.git] / src / surf / network_smpi.cpp
index 1490eb1..3575a83 100644 (file)
@@ -5,26 +5,20 @@
  * under the terms of the license (GNU LGPL) which comes with this package. */
 
 #include <cstddef>
-
 #include <algorithm>
 
 #include <xbt/log.h>
 
 #include "network_smpi.hpp"
 #include "simgrid/sg_config.h"
+#include "smpi/smpi_utils.hpp"
 
 XBT_LOG_EXTERNAL_DEFAULT_CATEGORY(surf_network);
 
-xbt_dynar_t smpi_bw_factor = NULL;
-xbt_dynar_t smpi_lat_factor = NULL;
-
-typedef struct s_smpi_factor *smpi_factor_t;
-typedef struct s_smpi_factor { // FIXME: s_smpi_factor_multival (defined in smpi_base) should be used instead to dedupplicate this code
-  long factor;
-  double value;
-} s_smpi_factor_t;
+std::vector<s_smpi_factor_t> smpi_bw_factor;
+std::vector<s_smpi_factor_t> smpi_lat_factor;
 
-xbt_dict_t gap_lookup = NULL;
+xbt_dict_t gap_lookup = nullptr;
 
 static int factor_cmp(const void *pa, const void *pb)
 {
@@ -33,40 +27,6 @@ static int factor_cmp(const void *pa, const void *pb)
 }
 
 #include "src/surf/xml/platf.hpp" // FIXME: move that back to the parsing area
-static xbt_dynar_t parse_factor(const char *smpi_coef_string)
-{
-  char *value = NULL;
-  unsigned int iter = 0;
-  s_smpi_factor_t fact;
-  xbt_dynar_t smpi_factor, radical_elements, radical_elements2 = NULL;
-
-  smpi_factor = xbt_dynar_new(sizeof(s_smpi_factor_t), NULL);
-  radical_elements = xbt_str_split(smpi_coef_string, ";");
-  xbt_dynar_foreach(radical_elements, iter, value) {
-
-    radical_elements2 = xbt_str_split(value, ":");
-    surf_parse_assert(xbt_dynar_length(radical_elements2) == 2,
-        "Malformed radical '%s' for smpi factor. I was expecting something like 'a:b'", value);
-
-    char *errmsg = bprintf("Invalid factor in chunk #%d: %%s", iter+1);
-    fact.factor = xbt_str_parse_int(xbt_dynar_get_as(radical_elements2, 0, char *), errmsg);
-    xbt_free(errmsg);
-    fact.value = xbt_str_parse_double(xbt_dynar_get_as(radical_elements2, 1, char *), errmsg);
-    errmsg = bprintf("Invalid factor value in chunk #%d: %%s", iter+1);
-    xbt_free(errmsg);
-
-    xbt_dynar_push_as(smpi_factor, s_smpi_factor_t, fact);
-    XBT_DEBUG("smpi_factor:\t%ld : %f", fact.factor, fact.value);
-    xbt_dynar_free(&radical_elements2);
-  }
-  xbt_dynar_free(&radical_elements);
-  xbt_dynar_sort(smpi_factor, &factor_cmp);
-  xbt_dynar_foreach(smpi_factor, iter, fact) {
-    XBT_DEBUG("ordered smpi_factor:\t%ld : %f", fact.factor, fact.value);
-
-  }
-  return smpi_factor;
-}
 
 /*********
  * Model *
@@ -83,17 +43,15 @@ static xbt_dynar_t parse_factor(const char *smpi_coef_string)
 /*  month=may, */
 /*  year={2011} */
 /*  } */
-void surf_network_model_init_SMPI(void)
+void surf_network_model_init_SMPI()
 {
-
   if (surf_network_model)
     return;
-  simgrid::surf::on_link.connect(netlink_parse_init);
   surf_network_model = new simgrid::surf::NetworkSmpiModel();
-  xbt_dynar_push(all_existing_models, &surf_network_model);
+  all_existing_models->push_back(surf_network_model);
 
-  xbt_cfg_setdefault_double(_sg_cfg_set, "network/sender_gap", 10e-6);
-  xbt_cfg_setdefault_double(_sg_cfg_set, "network/weight_S", 8775);
+  xbt_cfg_setdefault_double("network/sender-gap", 10e-6);
+  xbt_cfg_setdefault_double("network/weight-S", 8775);
 }
 
 namespace simgrid {
@@ -101,13 +59,11 @@ namespace simgrid {
 
     NetworkSmpiModel::NetworkSmpiModel()
     : NetworkCm02Model() {
-      m_haveGap=true;
+      haveGap_=true;
     }
 
     NetworkSmpiModel::~NetworkSmpiModel(){
       xbt_dict_free(&gap_lookup);
-      xbt_dynar_free(&smpi_bw_factor);
-      xbt_dynar_free(&smpi_lat_factor);
     }
 
     void NetworkSmpiModel::gapAppend(double size, Link* link, NetworkAction *act)
@@ -118,19 +74,19 @@ namespace simgrid {
 
       if (sg_sender_gap > 0.0) {
         if (!gap_lookup) {
-          gap_lookup = xbt_dict_new_homogeneous(NULL);
+          gap_lookup = xbt_dict_new_homogeneous(nullptr);
         }
         fifo = (xbt_fifo_t) xbt_dict_get_or_null(gap_lookup, src);
-        action->m_senderGap = 0.0;
+        action->senderGap_ = 0.0;
         if (fifo && xbt_fifo_size(fifo) > 0) {
           /* Compute gap from last send */
           /*last_action =
           (surf_action_network_CM02_t)
           xbt_fifo_get_item_content(xbt_fifo_get_last_item(fifo));*/
           // bw = net_get_link_bandwidth(link);
-          action->m_senderGap = sg_sender_gap;
+          action->senderGap_ = sg_sender_gap;
           /*  max(sg_sender_gap,last_action->sender.size / bw);*/
-          action->m_latency += action->m_senderGap;
+          action->latency_ += action->senderGap_;
         }
         /* Append action as last send */
         /*action->sender.link_name = link->lmm_resource.generic_resource.name;
@@ -139,10 +95,10 @@ namespace simgrid {
                                           action->sender.link_name);
     if (!fifo) {
       fifo = xbt_fifo_new();
-      xbt_dict_set(gap_lookup, action->sender.link_name, fifo, NULL);
+      xbt_dict_set(gap_lookup, action->sender.link_name, fifo, nullptr);
     }
     action->sender.fifo_item = xbt_fifo_push(fifo, action);*/
-        action->m_senderSize = size;
+        action->senderSize_ = size;
       }
     }
 
@@ -152,16 +108,16 @@ namespace simgrid {
       size_t size;
       NetworkCm02Action *action = static_cast<NetworkCm02Action*>(lmm_action);
 
-      if (sg_sender_gap > 0.0 && action->p_senderLinkName
-          && action->p_senderFifoItem) {
+      if (sg_sender_gap > 0.0 && action->senderLinkName_
+          && action->senderFifoItem_) {
         fifo =
             (xbt_fifo_t) xbt_dict_get_or_null(gap_lookup,
-                action->p_senderLinkName);
-        xbt_fifo_remove_item(fifo, action->p_senderFifoItem);
+                action->senderLinkName_);
+        xbt_fifo_remove_item(fifo, action->senderFifoItem_);
         size = xbt_fifo_size(fifo);
         if (size == 0) {
           xbt_fifo_free(fifo);
-          xbt_dict_remove(gap_lookup, action->p_senderLinkName);
+          xbt_dict_remove(gap_lookup, action->senderLinkName_);
           size = xbt_dict_length(gap_lookup);
           if (size == 0) {
             xbt_dict_free(&gap_lookup);
@@ -172,42 +128,36 @@ namespace simgrid {
 
     double NetworkSmpiModel::bandwidthFactor(double size)
     {
-      if (!smpi_bw_factor)
-        smpi_bw_factor =
-            parse_factor(sg_cfg_get_string("smpi/bw_factor"));
+      if (smpi_bw_factor.empty())
+        smpi_bw_factor = parse_factor(xbt_cfg_get_string("smpi/bw-factor"));
 
-      unsigned int iter = 0;
-      s_smpi_factor_t fact;
       double current=1.0;
-      xbt_dynar_foreach(smpi_bw_factor, iter, fact) {
+      for (auto fact: smpi_bw_factor) {
         if (size <= fact.factor) {
           XBT_DEBUG("%f <= %ld return %f", size, fact.factor, current);
           return current;
         }else
-          current=fact.value;
+          current=fact.values.front();
       }
-      XBT_DEBUG("%f > %ld return %f", size, fact.factor, current);
+      XBT_DEBUG("%f > %ld return %f", size, smpi_bw_factor.back().factor, current);
 
       return current;
     }
 
     double NetworkSmpiModel::latencyFactor(double size)
     {
-      if (!smpi_lat_factor)
-        smpi_lat_factor =
-            parse_factor(sg_cfg_get_string("smpi/lat_factor"));
+      if (smpi_lat_factor.empty())
+        smpi_lat_factor = parse_factor(xbt_cfg_get_string("smpi/lat-factor"));
 
-      unsigned int iter = 0;
-      s_smpi_factor_t fact;
       double current=1.0;
-      xbt_dynar_foreach(smpi_lat_factor, iter, fact) {
+      for (auto fact: smpi_lat_factor) {
         if (size <= fact.factor) {
           XBT_DEBUG("%f <= %ld return %f", size, fact.factor, current);
           return current;
         }else
-          current=fact.value;
+          current=fact.values.front();
       }
-      XBT_DEBUG("%f > %ld return %f", size, fact.factor, current);
+      XBT_DEBUG("%f > %ld return %f", size, smpi_lat_factor.back().factor, current);
 
       return current;
     }