X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/550c0e45cefecbf4acf3b0a2597cd10dfd92be70..aa252590f8b8836497d92ebda9afd9a8fa192e13:/src/surf/surf_routing.c diff --git a/src/surf/surf_routing.c b/src/surf/surf_routing.c index 45a8b14097..fef83e49b4 100644 --- a/src/surf/surf_routing.c +++ b/src/surf/surf_routing.c @@ -1,4 +1,5 @@ -/* Copyright (c) 2009 The SimGrid team. All rights reserved. */ +/* Copyright (c) 2009, 2010. 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. */ @@ -87,6 +88,9 @@ static void routing_full_parse_Shost(void) { DEBUG2("Seen host %s (#%d)",A_surfxml_host_id,used_routing->host_count); *val = used_routing->host_count++; xbt_dict_set(used_routing->host_id,A_surfxml_host_id,val,xbt_free); +#ifdef HAVE_TRACING + TRACE_surf_host_define_id (A_surfxml_host_id, *val); +#endif } static void routing_full_parse_Srouter(void) { @@ -95,6 +99,10 @@ static void routing_full_parse_Srouter(void) { HOST2ROUTER(used_routing->router_count)); *val = HOST2ROUTER(used_routing->router_count++); xbt_dict_set(used_routing->host_id,A_surfxml_router_id,val,xbt_free); +#ifdef HAVE_TRACING + TRACE_surf_host_define_id (A_surfxml_host_id, *val); + TRACE_surf_host_declaration (A_surfxml_host_id, 0); +#endif } static int src_id = -1; @@ -167,7 +175,9 @@ static void routing_full_parse_Scluster(void) surfxml_bufferstack_push(1); - /* Make set */ + /* Make set a set to parse the prefix/suffix/radical into a neat list of names */ + DEBUG4("Make ", + cluster_id,cluster_prefix,cluster_suffix,cluster_radical); SURFXML_BUFFER_SET(set_id, cluster_id); SURFXML_BUFFER_SET(set_prefix, cluster_prefix); SURFXML_BUFFER_SET(set_suffix, cluster_suffix); @@ -176,27 +186,26 @@ static void routing_full_parse_Scluster(void) SURFXML_START_TAG(set); SURFXML_END_TAG(set); - /* Make foreach */ - SURFXML_BUFFER_SET(foreach_set_id, cluster_id); - - SURFXML_START_TAG(foreach); - - /* Make host for the foreach */ - routing_full_parse_change_cpu_data("$1", cluster_power, "1.0", "", ""); - A_surfxml_host_state = A_surfxml_host_state_ON; + xbt_dynar_t names = xbt_dict_get(set_list,cluster_id); - SURFXML_START_TAG(host); - SURFXML_END_TAG(host); + unsigned int it1,it2; + char *name1,*name2; + xbt_dynar_foreach(names,it1,name1) { + /* create the host */ + routing_full_parse_change_cpu_data(name1, cluster_power, "1.0", "", ""); + A_surfxml_host_state = A_surfxml_host_state_ON; - /* Make link for the foreach */ - routing_full_parse_change_link_data("$1", cluster_bw, "", cluster_lat, "", ""); - A_surfxml_link_state = A_surfxml_link_state_ON; - A_surfxml_link_sharing_policy = A_surfxml_link_sharing_policy_SHARED; + SURFXML_START_TAG(host); + SURFXML_END_TAG(host); - SURFXML_START_TAG(link); - SURFXML_END_TAG(link); + /* Here comes the link */ + routing_full_parse_change_link_data(name1, cluster_bw, "", cluster_lat, "", ""); + A_surfxml_link_state = A_surfxml_link_state_ON; + A_surfxml_link_sharing_policy = A_surfxml_link_sharing_policy_SHARED; - SURFXML_END_TAG(foreach); + SURFXML_START_TAG(link); + SURFXML_END_TAG(link); + } /* Make backbone link */ backbone_name = bprintf("%s_bb", cluster_id); @@ -208,11 +217,53 @@ static void routing_full_parse_Scluster(void) SURFXML_START_TAG(link); SURFXML_END_TAG(link); + /* And now the internal routes */ + xbt_dynar_foreach(names,it1,name1) { + xbt_dynar_foreach(names,it2,name2) { + if (strcmp(name1,name2)) { + A_surfxml_route_action = A_surfxml_route_action_POSTPEND; + SURFXML_BUFFER_SET(route_src,name1); + SURFXML_BUFFER_SET(route_dst,name2); + SURFXML_START_TAG(route); { + /* FIXME: name1 link is added by error about 20 lines below, so don't add it here + SURFXML_BUFFER_SET(link_c_ctn_id, name1); + SURFXML_START_TAG(link_c_ctn); + SURFXML_END_TAG(link_c_ctn); + */ + SURFXML_BUFFER_SET(link_c_ctn_id, backbone_name); + SURFXML_START_TAG(link_c_ctn); + SURFXML_END_TAG(link_c_ctn); + + SURFXML_BUFFER_SET(link_c_ctn_id, name2); + SURFXML_START_TAG(link_c_ctn); + SURFXML_END_TAG(link_c_ctn); + + } SURFXML_END_TAG(route); + } + } + } + /* Make route multi with the outside world, i.e. cluster->$* */ + + /* FIXME + * This also adds an elements to the routes within the cluster, + * and I guess it's wrong, but since this element is commented out in the above + * code creating the internal routes, we're good. + * To fix it, I'd say that we need a way to understand "$*-${cluster_id}" as "whole world, but the guys in that cluster" + * But for that, we need to install a real expression parser for src/dst attributes + * + * FIXME + * This also adds a dumb element (the private link) in place of the loopback. Then, since + * the loopback is added only if no link to self already exist, this fails. + * That's really dumb. + * + * FIXME + * It seems to me that it does not add the backbone to the path to outside world... + */ SURFXML_BUFFER_SET(route_c_multi_src, cluster_id); SURFXML_BUFFER_SET(route_c_multi_dst, "$*"); A_surfxml_route_c_multi_symmetric = A_surfxml_route_c_multi_symmetric_NO; - A_surfxml_route_c_multi_action = A_surfxml_route_c_multi_action_OVERRIDE; + A_surfxml_route_c_multi_action = A_surfxml_route_c_multi_action_PREPEND; SURFXML_START_TAG(route_c_multi); @@ -223,21 +274,6 @@ static void routing_full_parse_Scluster(void) SURFXML_END_TAG(route_c_multi); - /* Make route multi between cluster hosts, i.e. cluster->cluster */ - SURFXML_BUFFER_SET(route_c_multi_src, cluster_id); - SURFXML_BUFFER_SET(route_c_multi_dst, cluster_id); - A_surfxml_route_c_multi_action = A_surfxml_route_c_multi_action_POSTPEND; - A_surfxml_route_c_multi_symmetric = A_surfxml_route_c_multi_symmetric_NO; - - SURFXML_START_TAG(route_c_multi); - - SURFXML_BUFFER_SET(link_c_ctn_id, backbone_name); - - SURFXML_START_TAG(link_c_ctn); - SURFXML_END_TAG(link_c_ctn); - - SURFXML_END_TAG(route_c_multi); - free(backbone_name); /* Restore buff */ @@ -282,6 +318,9 @@ static void routing_full_parse_end(void) { new_link->link_ptr = xbt_dict_get_or_null(surf_network_model->resource_set, link_name); DEBUG3("Adding onelink route from (#%d) to (#%d), link_name %s",src_id, dst_id, link_name); xbt_dict_set(onelink_routes, link_name, (void *)new_link, onelink_route_elem_free); +#ifdef HAVE_TRACING + TRACE_surf_link_save_endpoints (link_name, src_id, dst_id); +#endif } if(ISROUTER(src_id) || ISROUTER(dst_id)) { @@ -338,6 +377,7 @@ static void routing_full_finalize(void) { xbt_dynar_free(&ROUTE_FULL(i, j)); free(routing->routing_table); xbt_dict_free(&used_routing->host_id); + xbt_dict_free(&onelink_routes); free(routing); routing=NULL; }