static int id=0;
// pour t->id -> rajouter une nouvelle struct dans le dict, pour stocker les comms actives
- if(((NetworkIBModel*)surf_network_model)->active_nodes==nullptr)
- ((NetworkIBModel*)surf_network_model)->active_nodes = xbt_dict_new_homogeneous(nullptr);
IBNode* act = new IBNode(id);
id++;
- xbt_dict_set(((NetworkIBModel*)surf_network_model)->active_nodes, host.cname(), act, nullptr);
+ ((NetworkIBModel*)surf_network_model)->active_nodes.insert({host.getName(), act});
}
static void IB_action_state_changed_callback(simgrid::surf::NetworkAction* action)
simgrid::s4u::Host* dst)
{
simgrid::surf::NetworkIBModel* ibModel = (simgrid::surf::NetworkIBModel*)surf_network_model;
+ simgrid::surf::IBNode* act_src;
+ simgrid::surf::IBNode* act_dst;
- simgrid::surf::IBNode* act_src = (simgrid::surf::IBNode*)xbt_dict_get_or_null(ibModel->active_nodes, src->cname());
- xbt_assert(act_src, "could not find src node active comms !");
+ try {
+ act_src = ibModel->active_nodes.at(src->getName());
+ } catch (std::out_of_range& unfound) {
+ throw std::out_of_range(std::string("Could not find '") + src->getCname() + "' active comms !");
+ }
- simgrid::surf::IBNode* act_dst = (simgrid::surf::IBNode*)xbt_dict_get_or_null(ibModel->active_nodes, dst->cname());
- xbt_assert(act_dst, "could not find dst node active comms !");
+ try {
+ act_dst = ibModel->active_nodes.at(dst->getName());
+ } catch (std::out_of_range& unfound) {
+ throw std::out_of_range(std::string("Could not find '") + dst->getCname() + "' active comms !");
+ }
ibModel->active_comms[action]=std::make_pair(act_src, act_dst);
}
namespace simgrid {
- namespace surf {
-
- NetworkIBModel::NetworkIBModel()
- : NetworkSmpiModel() {
- haveGap_=false;
- active_nodes=nullptr;
-
- const char* IB_factors_string=xbt_cfg_get_string("smpi/IB-penalty-factors");
- std::vector<std::string> radical_elements;
- boost::split(radical_elements, IB_factors_string, boost::is_any_of(";"));
-
- surf_parse_assert(radical_elements.size() == 3, "smpi/IB-penalty-factors should be provided and contain 3 "
- "elements, semi-colon separated. Example: 0.965;0.925;1.35");
-
- Be = xbt_str_parse_double(radical_elements.front().c_str(),
- "First part of smpi/IB-penalty-factors is not numerical: %s");
- Bs = xbt_str_parse_double((radical_elements.at(1)).c_str(),
- "Second part of smpi/IB-penalty-factors is not numerical: %s");
- ys = xbt_str_parse_double(radical_elements.back().c_str(),
- "Third part of smpi/IB-penalty-factors is not numerical: %s");
- }
-
- NetworkIBModel::~NetworkIBModel()
- {
- xbt_dict_cursor_t cursor = nullptr;
- IBNode* instance = nullptr;
- char *name = nullptr;
- xbt_dict_foreach(active_nodes, cursor, name, instance)
- delete instance;
- xbt_dict_free(&active_nodes);
- }
-
- void NetworkIBModel::computeIBfactors(IBNode *root) {
- double num_comm_out = (double) root->ActiveCommsUp.size();
- double max_penalty_out=0.0;
- //first, compute all outbound penalties to get their max
- for (std::vector<ActiveComm*>::iterator it= root->ActiveCommsUp.begin(); it != root->ActiveCommsUp.end(); ++it) {
- double my_penalty_out = 1.0;
-
- if(num_comm_out!=1){
- if((*it)->destination->nbActiveCommsDown > 2)//number of comms sent to the receiving node
- my_penalty_out = num_comm_out * Bs * ys;
- else
- my_penalty_out = num_comm_out * Bs;
- }
-
- max_penalty_out = std::max(max_penalty_out,my_penalty_out);
- }
+namespace surf {
- for (std::vector<ActiveComm*>::iterator it= root->ActiveCommsUp.begin(); it != root->ActiveCommsUp.end(); ++it) {
+NetworkIBModel::NetworkIBModel() : NetworkSmpiModel()
+{
+ haveGap_ = false;
+ const char* IB_factors_string = xbt_cfg_get_string("smpi/IB-penalty-factors");
+ std::vector<std::string> radical_elements;
+ boost::split(radical_elements, IB_factors_string, boost::is_any_of(";"));
+
+ surf_parse_assert(radical_elements.size() == 3, "smpi/IB-penalty-factors should be provided and contain 3 "
+ "elements, semi-colon separated. Example: 0.965;0.925;1.35");
+
+ try {
+ Be = std::stod(radical_elements.front());
+ } catch (std::invalid_argument& ia) {
+ throw std::invalid_argument(std::string("First part of smpi/IB-penalty-factors is not numerical:") + ia.what());
+ }
- //compute inbound penalty
- double my_penalty_in = 1.0;
- int nb_comms = (*it)->destination->nbActiveCommsDown;//total number of incoming comms
- if(nb_comms!=1)
- my_penalty_in = ((*it)->destination->ActiveCommsDown)[root] //number of comm sent to dest by root node
- * Be
- * (*it)->destination->ActiveCommsDown.size();//number of different nodes sending to dest
+ try {
+ Bs = std::stod(radical_elements.at(1));
+ } catch (std::invalid_argument& ia) {
+ throw std::invalid_argument(std::string("Second part of smpi/IB-penalty-factors is not numerical:") + ia.what());
+ }
- double penalty = std::max(my_penalty_in,max_penalty_out);
+ try {
+ ys = std::stod(radical_elements.back());
+ } catch (std::invalid_argument& ia) {
+ throw std::invalid_argument(std::string("Third part of smpi/IB-penalty-factors is not numerical:") + ia.what());
+ }
+}
- double rate_before_update = (*it)->action->getBound();
- //save initial rate of the action
- if((*it)->init_rate==-1)
- (*it)->init_rate= rate_before_update;
+NetworkIBModel::~NetworkIBModel()
+{
+ for (auto instance : active_nodes)
+ delete instance.second;
+}
- double penalized_bw = num_comm_out ? (*it)->init_rate / penalty : (*it)->init_rate;
+void NetworkIBModel::computeIBfactors(IBNode* root)
+{
+ double num_comm_out = static_cast<double>(root->ActiveCommsUp.size());
+ double max_penalty_out = 0.0;
+ // first, compute all outbound penalties to get their max
+ for (std::vector<ActiveComm*>::iterator it = root->ActiveCommsUp.begin(); it != root->ActiveCommsUp.end(); ++it) {
+ double my_penalty_out = 1.0;
+
+ if (num_comm_out != 1) {
+ if ((*it)->destination->nbActiveCommsDown > 2) // number of comms sent to the receiving node
+ my_penalty_out = num_comm_out * Bs * ys;
+ else
+ my_penalty_out = num_comm_out * Bs;
+ }
- if (not double_equals(penalized_bw, rate_before_update, sg_surf_precision)) {
- XBT_DEBUG("%d->%d action %p penalty updated : bw now %f, before %f , initial rate %f", root->id,(*it)->destination->id,(*it)->action,penalized_bw, (*it)->action->getBound(), (*it)->init_rate );
- lmm_update_variable_bound(maxminSystem_, (*it)->action->getVariable(), penalized_bw);
- }else{
- XBT_DEBUG("%d->%d action %p penalty not updated : bw %f, initial rate %f", root->id,(*it)->destination->id,(*it)->action,penalized_bw, (*it)->init_rate );
- }
+ max_penalty_out = std::max(max_penalty_out, my_penalty_out);
+ }
- }
- XBT_DEBUG("Finished computing IB penalties");
+ for (std::vector<ActiveComm*>::iterator it = root->ActiveCommsUp.begin(); it != root->ActiveCommsUp.end(); ++it) {
+ // compute inbound penalty
+ double my_penalty_in = 1.0;
+ int nb_comms = (*it)->destination->nbActiveCommsDown; // total number of incoming comms
+ if (nb_comms != 1)
+ my_penalty_in = ((*it)->destination->ActiveCommsDown)[root] // number of comm sent to dest by root node
+ * Be * (*it)->destination->ActiveCommsDown.size(); // number of different nodes sending to dest
+
+ double penalty = std::max(my_penalty_in, max_penalty_out);
+
+ double rate_before_update = (*it)->action->getBound();
+ // save initial rate of the action
+ if ((*it)->init_rate == -1)
+ (*it)->init_rate = rate_before_update;
+
+ double penalized_bw = num_comm_out ? (*it)->init_rate / penalty : (*it)->init_rate;
+
+ if (not double_equals(penalized_bw, rate_before_update, sg_surf_precision)) {
+ XBT_DEBUG("%d->%d action %p penalty updated : bw now %f, before %f , initial rate %f", root->id,
+ (*it)->destination->id, (*it)->action, penalized_bw, (*it)->action->getBound(), (*it)->init_rate);
+ lmm_update_variable_bound(maxminSystem_, (*it)->action->getVariable(), penalized_bw);
+ } else {
+ XBT_DEBUG("%d->%d action %p penalty not updated : bw %f, initial rate %f", root->id, (*it)->destination->id,
+ (*it)->action, penalized_bw, (*it)->init_rate);
}
+ }
+ XBT_DEBUG("Finished computing IB penalties");
+}
- void NetworkIBModel::updateIBfactors_rec(IBNode *root, bool* updatedlist) {
- if(updatedlist[root->id]==0){
- XBT_DEBUG("IB - Updating rec %d", root->id);
- computeIBfactors(root);
- updatedlist[root->id]=1;
- for (std::vector<ActiveComm*>::iterator it= root->ActiveCommsUp.begin(); it != root->ActiveCommsUp.end(); ++it) {
- if(updatedlist[(*it)->destination->id]!=1)
- updateIBfactors_rec((*it)->destination, updatedlist);
- }
- for (std::map<IBNode*, int>::iterator it= root->ActiveCommsDown.begin(); it != root->ActiveCommsDown.end(); ++it) {
- if(updatedlist[it->first->id]!=1)
- updateIBfactors_rec(it->first, updatedlist);
- }
- }
+void NetworkIBModel::updateIBfactors_rec(IBNode* root, bool* updatedlist)
+{
+ if (updatedlist[root->id] == 0) {
+ XBT_DEBUG("IB - Updating rec %d", root->id);
+ computeIBfactors(root);
+ updatedlist[root->id] = 1;
+ for (std::vector<ActiveComm*>::iterator it = root->ActiveCommsUp.begin(); it != root->ActiveCommsUp.end(); ++it) {
+ if (updatedlist[(*it)->destination->id] != 1)
+ updateIBfactors_rec((*it)->destination, updatedlist);
+ }
+ for (std::map<IBNode*, int>::iterator it = root->ActiveCommsDown.begin(); it != root->ActiveCommsDown.end(); ++it) {
+ if (updatedlist[it->first->id] != 1)
+ updateIBfactors_rec(it->first, updatedlist);
}
+ }
+}
+void NetworkIBModel::updateIBfactors(NetworkAction* action, IBNode* from, IBNode* to, int remove)
+{
+ if (from == to) // disregard local comms (should use loopback)
+ return;
- void NetworkIBModel::updateIBfactors(NetworkAction *action, IBNode *from, IBNode * to, int remove) {
- if (from == to)//disregard local comms (should use loopback)
- return;
-
- bool* updated=(bool*)xbt_malloc0(xbt_dict_size(active_nodes)*sizeof(bool));
- ActiveComm* comm=nullptr;
- if(remove){
- if(to->ActiveCommsDown[from]==1)
- to->ActiveCommsDown.erase(from);
- else
- to->ActiveCommsDown[from]-=1;
-
- to->nbActiveCommsDown--;
- for (std::vector<ActiveComm*>::iterator it= from->ActiveCommsUp.begin();
- it != from->ActiveCommsUp.end(); ++it) {
- if((*it)->action==action){
- comm=(*it);
- from->ActiveCommsUp.erase(it);
- break;
- }
- }
- action->unref();
-
- }else{
- action->ref();
- ActiveComm* comm=new ActiveComm();
- comm->action=action;
- comm->destination=to;
- from->ActiveCommsUp.push_back(comm);
-
- to->ActiveCommsDown[from]+=1;
- to->nbActiveCommsDown++;
+ bool* updated = (bool*)xbt_malloc0(active_nodes.size() * sizeof(bool));
+ ActiveComm* comm = nullptr;
+ if (remove) {
+ if (to->ActiveCommsDown[from] == 1)
+ to->ActiveCommsDown.erase(from);
+ else
+ to->ActiveCommsDown[from] -= 1;
+
+ to->nbActiveCommsDown--;
+ for (std::vector<ActiveComm*>::iterator it = from->ActiveCommsUp.begin(); it != from->ActiveCommsUp.end(); ++it) {
+ if ((*it)->action == action) {
+ comm = (*it);
+ from->ActiveCommsUp.erase(it);
+ break;
}
- XBT_DEBUG("IB - Updating %d", from->id);
- updateIBfactors_rec(from, updated);
- XBT_DEBUG("IB - Finished updating %d", from->id);
- if(comm)
- delete comm;
- xbt_free(updated);
}
-
+ action->unref();
+ } else {
+ action->ref();
+ ActiveComm* comm = new ActiveComm();
+ comm->action = action;
+ comm->destination = to;
+ from->ActiveCommsUp.push_back(comm);
+
+ to->ActiveCommsDown[from] += 1;
+ to->nbActiveCommsDown++;
}
+ XBT_DEBUG("IB - Updating %d", from->id);
+ updateIBfactors_rec(from, updated);
+ XBT_DEBUG("IB - Finished updating %d", from->id);
+ if (comm)
+ delete comm;
+ xbt_free(updated);
+}
+}
}