X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/a18117755285a374ec3d5830309c924f2830e3ba..a92d7b716f51a53dea7f59db8524d4add713b910:/src/kernel/routing/ClusterZone.cpp diff --git a/src/kernel/routing/ClusterZone.cpp b/src/kernel/routing/ClusterZone.cpp index dc72ef325a..73ab218e53 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,32 +16,34 @@ 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 (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)); + 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); @@ -55,9 +58,10 @@ 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) @@ -70,7 +74,7 @@ void AsCluster::getLocalRoute(NetCard* src, NetCard* dst, sg_platf_route_cbarg_t } } -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."); @@ -80,18 +84,18 @@ void AsCluster::getGraph(xbt_graph_t graph, xbt_dict_t nodes, xbt_dict_t edges) xbt_node_t backboneNode = nullptr; if (backbone_) { - backboneNode = new_xbt_graph_node(graph, backbone_->getName(), nodes); + backboneNode = new_xbt_graph_node(graph, backbone_->cname(), nodes); new_xbt_graph_edge(graph, routerNode, backboneNode, edges); } for (auto src : vertices_) { - if (!src->isRouter()) { + 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,32 +119,32 @@ 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)); + LinkCreationArgs link; link.id = link_id; link.bandwidth = cluster->bw; 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); + 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}}); } } }