X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/a18117755285a374ec3d5830309c924f2830e3ba..b67343d6e9fc74fc159a51a48d0ea7d9a2374712:/src/kernel/routing/VivaldiZone.cpp diff --git a/src/kernel/routing/VivaldiZone.cpp b/src/kernel/routing/VivaldiZone.cpp index df9854721e..5eb06947e5 100644 --- a/src/kernel/routing/VivaldiZone.cpp +++ b/src/kernel/routing/VivaldiZone.cpp @@ -1,15 +1,17 @@ -/* Copyright (c) 2013-2016. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2013-2019. The SimGrid Team. All rights reserved. */ /* This program is free software; you can redistribute it and/or modify it * under the terms of the license (GNU LGPL) which comes with this package. */ -#include - -#include - -#include "src/kernel/routing/NetCard.hpp" -#include "src/kernel/routing/VivaldiZone.hpp" +#include "simgrid/kernel/routing/VivaldiZone.hpp" +#include "simgrid/kernel/routing/NetPoint.hpp" +#include "simgrid/s4u/Engine.hpp" +#include "simgrid/s4u/Host.hpp" #include "src/surf/network_interface.hpp" +#include "src/surf/xml/platf_private.hpp" +#include "surf/surf.hpp" + +#include XBT_LOG_NEW_DEFAULT_SUBCATEGORY(surf_route_vivaldi, surf, "Routing part of surf"); @@ -17,25 +19,28 @@ namespace simgrid { namespace kernel { namespace routing { namespace vivaldi { -simgrid::xbt::Extension Coords::EXTENSION_ID; +simgrid::xbt::Extension Coords::EXTENSION_ID; -Coords::Coords(NetCard* netcard, const char* coordStr) +Coords::Coords(NetPoint* netpoint, const std::string& coordStr) { - if (!Coords::EXTENSION_ID.valid()) - Coords::EXTENSION_ID = NetCard::extension_create(); + if (not Coords::EXTENSION_ID.valid()) + Coords::EXTENSION_ID = NetPoint::extension_create(); std::vector string_values; boost::split(string_values, coordStr, boost::is_any_of(" ")); - xbt_assert(string_values.size() == 3, "Coordinates of %s must have 3 dimensions", netcard->cname()); + xbt_assert(string_values.size() == 3, "Coordinates of %s must have 3 dimensions", netpoint->get_cname()); - for (auto str : string_values) - coords.push_back(xbt_str_parse_double(str.c_str(), "Invalid coordinate: %s")); + for (auto const& str : string_values) + try { + coords.push_back(std::stod(str)); + } catch (std::invalid_argument const& ia) { + throw std::invalid_argument(std::string("Invalid coordinate: ") + ia.what()); + } coords.shrink_to_fit(); - netcard->extension_set(this); - XBT_DEBUG("Coords of %s %p: %s", netcard->cname(), netcard, coordStr); + netpoint->extension_set(this); + XBT_DEBUG("Coords of %s %p: %s", netpoint->get_cname(), netpoint, coordStr.c_str()); } -Coords::~Coords() = default; }; // namespace vivaldi static inline double euclidean_dist_comp(int index, std::vector* src, std::vector* dst) @@ -46,68 +51,73 @@ static inline double euclidean_dist_comp(int index, std::vector* src, st return (src_coord - dst_coord) * (src_coord - dst_coord); } -static std::vector* getCoordsFromNetcard(NetCard* nc) +static std::vector* netpoint_get_coords(NetPoint* np) { - simgrid::kernel::routing::vivaldi::Coords* coords = nc->extension(); + simgrid::kernel::routing::vivaldi::Coords* coords = np->extension(); xbt_assert(coords, "Please specify the Vivaldi coordinates of %s %s (%p)", - (nc->isAS() ? "AS" : (nc->isHost() ? "Host" : "Router")), nc->cname(), nc); + (np->is_netzone() ? "Netzone" : (np->is_host() ? "Host" : "Router")), np->get_cname(), np); return &coords->coords; } -AsVivaldi::AsVivaldi(As* father, const char* name) : AsCluster(father, name) + +VivaldiZone::VivaldiZone(NetZoneImpl* father, const std::string& name, resource::NetworkModel* netmodel) + : ClusterZone(father, name, netmodel) { } -void AsVivaldi::setPeerLink(NetCard* netcard, double bw_in, double bw_out, double latency, const char* coord) +void VivaldiZone::set_peer_link(NetPoint* netpoint, double bw_in, double bw_out, const std::string& coord) { - xbt_assert(netcard->containingAS() == this, "Cannot add a peer link to a netcard that is not in this AS"); - - new simgrid::kernel::routing::vivaldi::Coords(netcard, coord); + xbt_assert(netpoint->get_englobing_zone() == this, + "Cannot add a peer link to a netpoint that is not in this netzone"); - char* link_up = bprintf("link_%s_UP", netcard->cname()); - char* link_down = bprintf("link_%s_DOWN", netcard->cname()); - Link* linkUp = surf_network_model->createLink(link_up, bw_out, latency, SURF_LINK_SHARED); - Link* linkDown = surf_network_model->createLink(link_down, bw_in, latency, SURF_LINK_SHARED); - privateLinks_.insert({netcard->id(), {linkUp, linkDown}}); + new simgrid::kernel::routing::vivaldi::Coords(netpoint, coord); - free(link_up); - free(link_down); + std::string link_up = "link_" + netpoint->get_name() + "_UP"; + std::string link_down = "link_" + netpoint->get_name() + "_DOWN"; + resource::LinkImpl* linkUp = network_model_->create_link(link_up, bw_out, 0, s4u::Link::SharingPolicy::SHARED); + resource::LinkImpl* linkDown = network_model_->create_link(link_down, bw_in, 0, s4u::Link::SharingPolicy::SHARED); + private_links_.insert({netpoint->id(), {linkUp, linkDown}}); } -void AsVivaldi::getLocalRoute(NetCard* src, NetCard* dst, sg_platf_route_cbarg_t route, double* lat) +void VivaldiZone::get_local_route(NetPoint* src, NetPoint* dst, RouteCreationArgs* route, double* lat) { - XBT_DEBUG("vivaldi getLocalRoute from '%s'[%d] '%s'[%d]", src->cname(), src->id(), dst->cname(), dst->id()); - - if (src->isAS()) { - char* srcName = bprintf("router_%s", src->cname()); - char* dstName = bprintf("router_%s", dst->cname()); - route->gw_src = (sg_netcard_t)xbt_lib_get_or_null(as_router_lib, srcName, ROUTING_ASR_LEVEL); - route->gw_dst = (sg_netcard_t)xbt_lib_get_or_null(as_router_lib, dstName, ROUTING_ASR_LEVEL); - xbt_free(srcName); - xbt_free(dstName); + XBT_DEBUG("vivaldi getLocalRoute from '%s'[%u] '%s'[%u]", src->get_cname(), src->id(), dst->get_cname(), dst->id()); + + if (src->is_netzone()) { + std::string srcName = "router_" + src->get_name(); + std::string dstName = "router_" + dst->get_name(); + route->gw_src = simgrid::s4u::Engine::get_instance()->netpoint_by_name_or_null(srcName); + route->gw_dst = simgrid::s4u::Engine::get_instance()->netpoint_by_name_or_null(dstName); } /* Retrieve the private links */ - if (privateLinks_.find(src->id()) != privateLinks_.end()) { - std::pair info = privateLinks_.at(src->id()); + auto src_link = private_links_.find(src->id()); + if (src_link != private_links_.end()) { + std::pair info = src_link->second; if (info.first) { - route->link_list->push_back(info.first); + route->link_list.push_back(info.first); if (lat) - *lat += info.first->latency(); + *lat += info.first->get_latency(); } + } else { + XBT_DEBUG("Source of private link (%u) doesn't exist", src->id()); } - if (privateLinks_.find(dst->id()) != privateLinks_.end()) { - std::pair info = privateLinks_.at(dst->id()); + + auto dst_link = private_links_.find(dst->id()); + if (dst_link != private_links_.end()) { + std::pair info = dst_link->second; if (info.second) { - route->link_list->push_back(info.second); + route->link_list.push_back(info.second); if (lat) - *lat += info.second->latency(); + *lat += info.second->get_latency(); } + } else { + XBT_DEBUG("Destination of private link (%u) doesn't exist", dst->id()); } /* Compute the extra latency due to the euclidean distance if needed */ if (lat) { - std::vector* srcCoords = getCoordsFromNetcard(src); - std::vector* dstCoords = getCoordsFromNetcard(dst); + std::vector* srcCoords = netpoint_get_coords(src); + std::vector* dstCoords = netpoint_get_coords(dst); double euclidean_dist = sqrt(euclidean_dist_comp(0, srcCoords, dstCoords) + euclidean_dist_comp(1, srcCoords, dstCoords)) +