X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/a22eb0ac7d22d5beaa6e33adf8f85c28dac59024..288deb04fbcfc6a916717e31e8009d209d3314cf:/src/kernel/routing/ClusterZone.cpp?ds=sidebyside diff --git a/src/kernel/routing/ClusterZone.cpp b/src/kernel/routing/ClusterZone.cpp index 520ac663e7..1a191c8b6a 100644 --- a/src/kernel/routing/ClusterZone.cpp +++ b/src/kernel/routing/ClusterZone.cpp @@ -4,7 +4,8 @@ * under the terms of the license (GNU LGPL) which comes with this package. */ #include "src/kernel/routing/ClusterZone.hpp" -#include "src/kernel/routing/NetCard.hpp" +#include "src/kernel/routing/NetPoint.hpp" +#include "src/kernel/routing/RoutedZone.hpp" #include "src/surf/network_interface.hpp" XBT_LOG_NEW_DEFAULT_SUBCATEGORY(surf_route_cluster, surf, "Routing part of surf"); @@ -15,39 +16,40 @@ XBT_LOG_NEW_DEFAULT_SUBCATEGORY(surf_route_cluster, surf, "Routing part of surf" namespace simgrid { namespace kernel { namespace routing { -AsCluster::AsCluster(As* father, const char* name) : AsImpl(father, name) +ClusterZone::ClusterZone(NetZone* father, const char* name) : NetZoneImpl(father, name) { } -void AsCluster::getLocalRoute(NetCard* src, NetCard* dst, sg_platf_route_cbarg_t route, double* lat) +void ClusterZone::getLocalRoute(NetPoint* src, NetPoint* dst, sg_platf_route_cbarg_t route, double* lat) { XBT_VERB("cluster getLocalRoute from '%s'[%d] to '%s'[%d]", src->cname(), src->id(), dst->cname(), dst->id()); - xbt_assert(!privateLinks_.empty(), + xbt_assert(not privateLinks_.empty(), "Cluster routing: no links attached to the source node - did you use host_link tag?"); - if (! src->isRouter()) { // No specific link for router + if ((src->id() == dst->id()) && hasLoopback_) { + xbt_assert(not src->isRouter(), "Routing from a cluster private router to itself is meaningless"); - if((src->id() == dst->id()) && hasLoopback_ ){ - std::pair info = privateLinks_.at(src->id() * linkCountPerNode_); - route->link_list->push_back(info.first); - if (lat) - *lat += info.first->latency(); - return; - } + std::pair info = privateLinks_.at(src->id() * linkCountPerNode_); + route->link_list->push_back(info.first); + if (lat) + *lat += info.first->latency(); + return; + } - if (hasLimiter_){ // limiter for sender - std::pair info = privateLinks_.at(src->id() * linkCountPerNode_ + (hasLoopback_ ? 1 : 0)); + if (not src->isRouter()) { // No private link for the private router + if (hasLimiter_) { // limiter for sender + std::pair info = + privateLinks_.at(src->id() * linkCountPerNode_ + (hasLoopback_ ? 1 : 0)); route->link_list->push_back(info.first); } - std::pair info = + std::pair info = privateLinks_.at(src->id() * linkCountPerNode_ + (hasLoopback_ ? 1 : 0) + (hasLimiter_ ? 1 : 0)); if (info.first) { // link up route->link_list->push_back(info.first); if (lat) *lat += info.first->latency(); } - } if (backbone_) { @@ -56,42 +58,44 @@ void AsCluster::getLocalRoute(NetCard* src, NetCard* dst, sg_platf_route_cbarg_t *lat += backbone_->latency(); } - if (! dst->isRouter()) { // No specific link for router - std::pair info = privateLinks_.at(dst->id() * linkCountPerNode_ + hasLoopback_ + hasLimiter_); + if (not dst->isRouter()) { // No specific link for router + std::pair info = + privateLinks_.at(dst->id() * linkCountPerNode_ + hasLoopback_ + hasLimiter_); if (info.second) { // link down route->link_list->push_back(info.second); if (lat) *lat += info.second->latency(); } - if (hasLimiter_){ // limiter for receiver - info = privateLinks_.at(dst->id() * linkCountPerNode_ + hasLoopback_); - route->link_list->push_back(info.first); + if (hasLimiter_) { // limiter for receiver + info = privateLinks_.at(dst->id() * linkCountPerNode_ + hasLoopback_); + route->link_list->push_back(info.first); } } } -void AsCluster::getGraph(xbt_graph_t graph, xbt_dict_t nodes, xbt_dict_t edges) +void ClusterZone::getGraph(xbt_graph_t graph, xbt_dict_t nodes, xbt_dict_t edges) { - xbt_assert(router_,"Malformed cluster. This may be because your platform file is a hypergraph while it must be a graph."); + xbt_assert(router_, + "Malformed cluster. This may be because your platform file is a hypergraph while it must be a graph."); /* create the router */ xbt_node_t routerNode = new_xbt_graph_node(graph, router_->cname(), nodes); xbt_node_t backboneNode = nullptr; - if(backbone_) { - backboneNode = new_xbt_graph_node(graph, backbone_->getName(), nodes); + if (backbone_) { + backboneNode = new_xbt_graph_node(graph, backbone_->cname(), nodes); new_xbt_graph_edge(graph, routerNode, backboneNode, edges); } - for (auto src: vertices_){ - if (! src->isRouter()) { + for (auto src : getVertices()) { + if (not src->isRouter()) { xbt_node_t previous = new_xbt_graph_node(graph, src->cname(), nodes); - std::pair info = privateLinks_.at(src->id()); + std::pair info = privateLinks_.at(src->id()); if (info.first) { // link up - xbt_node_t current = new_xbt_graph_node(graph, info.first->getName(), nodes); + xbt_node_t current = new_xbt_graph_node(graph, info.first->cname(), nodes); new_xbt_graph_edge(graph, previous, current, edges); if (backbone_) { @@ -102,7 +106,7 @@ void AsCluster::getGraph(xbt_graph_t graph, xbt_dict_t nodes, xbt_dict_t edges) } if (info.second) { // link down - xbt_node_t current = new_xbt_graph_node(graph, info.second->getName(), nodes); + xbt_node_t current = new_xbt_graph_node(graph, info.second->cname(), nodes); new_xbt_graph_edge(graph, previous, current, edges); if (backbone_) { @@ -115,31 +119,33 @@ void AsCluster::getGraph(xbt_graph_t graph, xbt_dict_t nodes, xbt_dict_t edges) } } -void AsCluster::create_links_for_node(sg_platf_cluster_cbarg_t cluster, int id, int , int position){ +void ClusterZone::create_links_for_node(sg_platf_cluster_cbarg_t cluster, int id, int /*rank*/, int position) +{ char* link_id = bprintf("%s_link_%d", cluster->id, id); - s_sg_platf_link_cbarg_t link; - memset(&link, 0, sizeof(link)); - link.id = link_id; + LinkCreationArgs link; + link.id = link_id; link.bandwidth = cluster->bw; - link.latency = cluster->lat; - link.policy = cluster->sharing_policy; + link.latency = cluster->lat; + link.policy = cluster->sharing_policy; sg_platf_new_link(&link); - Link *linkUp, *linkDown; + surf::LinkImpl *linkUp; + surf::LinkImpl *linkDown; if (link.policy == SURF_LINK_FULLDUPLEX) { - char *tmp_link = bprintf("%s_UP", link_id); - linkUp = Link::byName(tmp_link); + char* tmp_link = bprintf("%s_UP", link_id); + linkUp = surf::LinkImpl::byName(tmp_link); xbt_free(tmp_link); tmp_link = bprintf("%s_DOWN", link_id); - linkDown = Link::byName(tmp_link); + linkDown = surf::LinkImpl::byName(tmp_link); xbt_free(tmp_link); } else { - linkUp = Link::byName(link_id); + linkUp = surf::LinkImpl::byName(link_id); linkDown = linkUp; } - privateLinks_.insert({position, {linkUp, linkDown}}); xbt_free(link_id); + privateLinks_.insert({position, {linkUp, linkDown}}); +} +} +} } - -}}}