Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Reindent.
[simgrid.git] / src / surf / surf_routing_full.c
index 3c2c05d..5e22b48 100644 (file)
@@ -34,16 +34,13 @@ static xbt_dynar_t full_get_onelink_routes(AS_t rc)
     xbt_dict_foreach(routing->generic_routing.to_index, c2, k2, d2) {
       int *src_id = xbt_dict_get_or_null(routing->generic_routing.to_index, k1);
       int *dst_id = xbt_dict_get_or_null(routing->generic_routing.to_index, k2);
-      xbt_assert(src_id
-                  && dst_id,
-                  "Ask for route \"from\"(%s)  or \"to\"(%s) no found in the local table",
-                  k1, k2);
+      xbt_assert(src_id && dst_id,
+                 "Ask for route \"from\"(%s)  or \"to\"(%s) "
+                 "no found in the local table", k1, k2);
       route_t route = TO_ROUTE_FULL(*src_id, *dst_id);
       if (route) {
         if (xbt_dynar_length(route->link_list) == 1) {
-          void *link =
-              *(void **) xbt_dynar_get_ptr(route->link_list,
-                                           0);
+          void *link = *(void **) xbt_dynar_get_ptr(route->link_list, 0);
           onelink_t onelink = xbt_new0(s_onelink_t, 1);
           onelink->link_ptr = link;
           if (routing->generic_routing.hierarchy == SURF_ROUTING_BASE) {
@@ -63,8 +60,8 @@ static xbt_dynar_t full_get_onelink_routes(AS_t rc)
 }
 
 static void full_get_route_and_latency(AS_t rc,
-                           const char *src, const char *dst,
-                           route_t res,double *lat)
+                                       const char *src, const char *dst,
+                                       route_t res, double *lat)
 {
 
   /* set utils vars */
@@ -75,7 +72,7 @@ static void full_get_route_and_latency(AS_t rc,
   int *dst_id = xbt_dict_get_or_null(routing->generic_routing.to_index, dst);
 
   if (!src_id || !dst_id)
-    THROWF(arg_error,0,"No route from '%s' to '%s'",src,dst);
+    THROWF(arg_error, 0, "No route from '%s' to '%s'", src, dst);
 
   route_t e_route = NULL;
   void *link;
@@ -118,9 +115,9 @@ AS_t model_full_create(void)
 
   new_component->generic_routing.parse_route = model_full_set_route;
   new_component->generic_routing.parse_ASroute = model_full_set_route;
-  new_component->generic_routing.get_route_and_latency = full_get_route_and_latency;
-  new_component->generic_routing.get_onelink_routes =
-      full_get_onelink_routes;
+  new_component->generic_routing.get_route_and_latency =
+      full_get_route_and_latency;
+  new_component->generic_routing.get_onelink_routes = full_get_onelink_routes;
   new_component->generic_routing.finalize = full_finalize;
 
   return (AS_t) new_component;
@@ -137,8 +134,8 @@ void model_full_end(AS_t current_routing)
   size_t table_size = xbt_dict_length(routing->generic_routing.to_index);
 
   /* Create table if necessary */
-  if(!routing->routing_table)
-         routing->routing_table = xbt_new0(route_t, table_size * table_size);
+  if (!routing->routing_table)
+    routing->routing_table = xbt_new0(route_t, table_size * table_size);
 
   /* Add the loopback if needed */
   if (current_routing->hierarchy == SURF_ROUTING_BASE) {
@@ -149,127 +146,135 @@ void model_full_end(AS_t current_routing)
         e_route->src_gateway = NULL;
         e_route->dst_gateway = NULL;
         e_route->link_list = xbt_dynar_new(global_routing->size_of_link, NULL);
-        xbt_dynar_push(e_route->link_list,
-                       &global_routing->loopback);
+        xbt_dynar_push(e_route->link_list, &global_routing->loopback);
         TO_ROUTE_FULL(i, i) = e_route;
       }
     }
   }
 }
 
-static int full_pointer_resource_cmp(const void *a, const void *b) {
+static int full_pointer_resource_cmp(const void *a, const void *b)
+{
   return a != b;
 }
 
 void model_full_set_route(AS_t rc, const char *src,
-               const char *dst, route_t route)
+                          const char *dst, route_t route)
 {
-       int *src_id, *dst_id;
-       src_id = xbt_dict_get_or_null(rc->to_index, src);
-       dst_id = xbt_dict_get_or_null(rc->to_index, dst);
-       routing_component_full_t routing = ((routing_component_full_t) rc);
-       size_t table_size = xbt_dict_length(routing->generic_routing.to_index);
-
-       xbt_assert(src_id, "Network elements %s not found", src);
-       xbt_assert(dst_id, "Network elements %s not found", dst);
-
-       xbt_assert(!xbt_dynar_is_empty(route->link_list),
-                         "Invalid count of links, must be greater than zero (%s,%s)",
-                         src, dst);
-
-       if(!routing->routing_table)
-               routing->routing_table = xbt_new0(route_t, table_size * table_size);
-
-       if(TO_ROUTE_FULL(*src_id, *dst_id))
-       {
-               char * link_name;
-               unsigned int i;
-               xbt_dynar_t link_route_to_test = xbt_dynar_new(global_routing->size_of_link, NULL);
-               xbt_dynar_foreach(route->link_list,i,link_name)
-               {
-                       void *link = xbt_lib_get_or_null(link_lib, link_name, SURF_LINK_LEVEL);
-                       xbt_assert(link,"Link : '%s' doesn't exists.",link_name);
-                       xbt_dynar_push(link_route_to_test,&link);
-               }
-               if (xbt_dynar_compare((void*)TO_ROUTE_FULL(*src_id, *dst_id)->link_list,
-                                             (void*)link_route_to_test,
-                                             (int_f_cpvoid_cpvoid_t) full_pointer_resource_cmp)) {
-                       surf_parse_error("A route between \"%s\" and \"%s\" already exists with a different content. "
-                                                "If you are trying to define a reverse route, you must set the symmetrical=no "
-                                                "attribute to your routes tags.", src,dst);
-               } else {
-                       surf_parse_warn("Ignoring the identical redefinition of the route between \"%s\" and \"%s\"",src,dst);
-               }
-       }
-       else
-       {
-                 if(!route->dst_gateway && !route->src_gateway)
-                         XBT_DEBUG("Load Route from \"%s\" to \"%s\"", src, dst);
-                 else{
-                   AS_t subas = xbt_dict_get_or_null(rc->routing_sons, src);
-                   if (subas==NULL)
-                     surf_parse_error("The source of an ASroute must be a sub-AS declared within the current AS, but '%s' is not an AS within '%s'",
-                         src,rc->name);
-                   if (xbt_dict_get_or_null(subas->to_index, route->src_gateway) == NULL)
-                     surf_parse_error("In an ASroute, source gateway must be part of the source sub-AS (in particular, being in a sub-sub-AS is not allowed), but '%s' is not in '%s'.",
-                         route->src_gateway,subas->name);
-
-                   subas = xbt_dict_get_or_null(rc->routing_sons, dst);
-                   if (subas==NULL)
-                     surf_parse_error("The destination of an ASroute must be a sub-AS declared within the current AS, but '%s' is not an AS within '%s'",
-                         dst,rc->name);
-                   if (xbt_dict_get_or_null(subas->to_index, route->dst_gateway) == NULL)
-                     surf_parse_error("In an ASroute, destination gateway must be part of the destination sub-AS (in particular, in a sub-sub-AS is not allowed), but '%s' is not in '%s'.",
-                         route->dst_gateway,subas->name);
-
-                         XBT_DEBUG("Load ASroute from \"%s(%s)\" to \"%s(%s)\"", src,
-                                route->src_gateway, dst, route->dst_gateway);
-                         if(routing_get_network_element_type((const char*)route->dst_gateway) == SURF_NETWORK_ELEMENT_NULL)
-                                 xbt_die("The dst_gateway '%s' does not exist!",route->dst_gateway);
-                         if(routing_get_network_element_type((const char*)route->src_gateway) == SURF_NETWORK_ELEMENT_NULL)
-                                 xbt_die("The src_gateway '%s' does not exist!",route->src_gateway);
-                 }
-             TO_ROUTE_FULL(*src_id, *dst_id) = generic_new_extended_route(rc->hierarchy,route,1);
-             xbt_dynar_shrink(TO_ROUTE_FULL(*src_id, *dst_id)->link_list, 0);
-       }
-
-       if( A_surfxml_route_symmetrical == A_surfxml_route_symmetrical_YES
-               || A_surfxml_ASroute_symmetrical == A_surfxml_ASroute_symmetrical_YES )
-       {
-               if(route->dst_gateway && route->src_gateway)
-               {
-                  char *gw_tmp ;
-                  gw_tmp = route->src_gateway;
-                  route->src_gateway = route->dst_gateway;
-                  route->dst_gateway = gw_tmp;
-               }
-               if(TO_ROUTE_FULL(*dst_id, *src_id))
-               {
-                       char * link_name;
-                       unsigned int i;
-                       xbt_dynar_t link_route_to_test = xbt_dynar_new(global_routing->size_of_link, NULL);
-                       for(i=xbt_dynar_length(route->link_list) ;i>0 ;i--)
-                       {
-                               link_name = xbt_dynar_get_as(route->link_list,i-1,void *);
-                               void *link = xbt_lib_get_or_null(link_lib, link_name, SURF_LINK_LEVEL);
-                               xbt_assert(link,"Link : '%s' doesn't exists.",link_name);
-                               xbt_dynar_push(link_route_to_test,&link);
-                       }
-                       xbt_assert(!xbt_dynar_compare(
-                                 (void*)TO_ROUTE_FULL(*dst_id, *src_id)->link_list,
-                             (void*)link_route_to_test,
-                                 (int_f_cpvoid_cpvoid_t) full_pointer_resource_cmp),
-                                 "The route between \"%s\" and \"%s\" already exists", src,dst);
-               }
-               else
-               {
-                         if(!route->dst_gateway && !route->src_gateway)
-                                 XBT_DEBUG("Load Route from \"%s\" to \"%s\"", dst, src);
-                         else
-                                 XBT_DEBUG("Load ASroute from \"%s(%s)\" to \"%s(%s)\"", dst,
-                                        route->src_gateway, src, route->dst_gateway);
-                     TO_ROUTE_FULL(*dst_id, *src_id) = generic_new_extended_route(rc->hierarchy,route,0);
-                     xbt_dynar_shrink(TO_ROUTE_FULL(*dst_id, *src_id)->link_list, 0);
-               }
-       }
+  int *src_id, *dst_id;
+  src_id = xbt_dict_get_or_null(rc->to_index, src);
+  dst_id = xbt_dict_get_or_null(rc->to_index, dst);
+  routing_component_full_t routing = ((routing_component_full_t) rc);
+  size_t table_size = xbt_dict_length(routing->generic_routing.to_index);
+
+  xbt_assert(src_id, "Network elements %s not found", src);
+  xbt_assert(dst_id, "Network elements %s not found", dst);
+
+  xbt_assert(!xbt_dynar_is_empty(route->link_list),
+             "Invalid count of links, must be greater than zero (%s,%s)",
+             src, dst);
+
+  if (!routing->routing_table)
+    routing->routing_table = xbt_new0(route_t, table_size * table_size);
+
+  if (TO_ROUTE_FULL(*src_id, *dst_id)) {
+    char *link_name;
+    unsigned int i;
+    xbt_dynar_t link_route_to_test =
+        xbt_dynar_new(global_routing->size_of_link, NULL);
+    xbt_dynar_foreach(route->link_list, i, link_name) {
+      void *link = xbt_lib_get_or_null(link_lib, link_name, SURF_LINK_LEVEL);
+      xbt_assert(link, "Link : '%s' doesn't exists.", link_name);
+      xbt_dynar_push(link_route_to_test, &link);
+    }
+    if (xbt_dynar_compare((void *) TO_ROUTE_FULL(*src_id, *dst_id)->link_list,
+                          (void *) link_route_to_test,
+                          (int_f_cpvoid_cpvoid_t) full_pointer_resource_cmp)) {
+      surf_parse_error("A route between \"%s\" and \"%s\" already exists "
+                       "with a different content. "
+                       "If you are trying to define a reverse route, "
+                       "you must set the symmetrical=no attribute to "
+                       "your routes tags.", src, dst);
+    } else {
+      surf_parse_warn("Ignoring the identical redefinition of the route "
+                      "between \"%s\" and \"%s\"", src, dst);
+    }
+  } else {
+    if (!route->dst_gateway && !route->src_gateway)
+      XBT_DEBUG("Load Route from \"%s\" to \"%s\"", src, dst);
+    else {
+      AS_t subas = xbt_dict_get_or_null(rc->routing_sons, src);
+      if (subas == NULL)
+        surf_parse_error("The source of an ASroute must be a sub-AS "
+                         "declared within the current AS, "
+                         "but '%s' is not an AS within '%s'", src, rc->name);
+      if (xbt_dict_get_or_null(subas->to_index, route->src_gateway) == NULL)
+        surf_parse_error("In an ASroute, source gateway must be part of "
+                         "the source sub-AS (in particular, being in a "
+                         "sub-sub-AS is not allowed), "
+                         "but '%s' is not in '%s'.",
+                         route->src_gateway, subas->name);
+
+      subas = xbt_dict_get_or_null(rc->routing_sons, dst);
+      if (subas == NULL)
+        surf_parse_error("The destination of an ASroute must be a sub-AS "
+                         "declared within the current AS, "
+                         "but '%s' is not an AS within '%s'", dst, rc->name);
+      if (xbt_dict_get_or_null(subas->to_index, route->dst_gateway) == NULL)
+        surf_parse_error("In an ASroute, destination gateway must be "
+                         "part of the destination sub-AS (in particular, "
+                         "in a sub-sub-AS is not allowed), "
+                         "but '%s' is not in '%s'.",
+                         route->dst_gateway, subas->name);
+
+      XBT_DEBUG("Load ASroute from \"%s(%s)\" to \"%s(%s)\"",
+                src, route->src_gateway, dst, route->dst_gateway);
+      if (routing_get_network_element_type((const char *) route->dst_gateway) ==
+          SURF_NETWORK_ELEMENT_NULL)
+        xbt_die("The dst_gateway '%s' does not exist!", route->dst_gateway);
+      if (routing_get_network_element_type((const char *) route->src_gateway) ==
+          SURF_NETWORK_ELEMENT_NULL)
+        xbt_die("The src_gateway '%s' does not exist!", route->src_gateway);
+    }
+    TO_ROUTE_FULL(*src_id, *dst_id) =
+        generic_new_extended_route(rc->hierarchy, route, 1);
+    xbt_dynar_shrink(TO_ROUTE_FULL(*src_id, *dst_id)->link_list, 0);
+  }
+
+  if (A_surfxml_route_symmetrical == A_surfxml_route_symmetrical_YES
+      || A_surfxml_ASroute_symmetrical == A_surfxml_ASroute_symmetrical_YES) {
+    if (route->dst_gateway && route->src_gateway) {
+      char *gw_tmp;
+      gw_tmp = route->src_gateway;
+      route->src_gateway = route->dst_gateway;
+      route->dst_gateway = gw_tmp;
+    }
+    if (TO_ROUTE_FULL(*dst_id, *src_id)) {
+      char *link_name;
+      unsigned int i;
+      xbt_dynar_t link_route_to_test =
+          xbt_dynar_new(global_routing->size_of_link, NULL);
+      for (i = xbt_dynar_length(route->link_list); i > 0; i--) {
+        link_name = xbt_dynar_get_as(route->link_list, i - 1, void *);
+        void *link = xbt_lib_get_or_null(link_lib, link_name, SURF_LINK_LEVEL);
+        xbt_assert(link, "Link : '%s' doesn't exists.", link_name);
+        xbt_dynar_push(link_route_to_test, &link);
+      }
+      xbt_assert(!xbt_dynar_compare
+                 ((void *) TO_ROUTE_FULL(*dst_id, *src_id)->link_list,
+                  (void *) link_route_to_test, (int_f_cpvoid_cpvoid_t)
+                  full_pointer_resource_cmp),
+                 "The route between \"%s\" and \"%s\" already exists", src,
+                 dst);
+    } else {
+      if (!route->dst_gateway && !route->src_gateway)
+        XBT_DEBUG("Load Route from \"%s\" to \"%s\"", dst, src);
+      else
+        XBT_DEBUG("Load ASroute from \"%s(%s)\" to \"%s(%s)\"",
+                  dst, route->src_gateway, src, route->dst_gateway);
+      TO_ROUTE_FULL(*dst_id, *src_id) =
+          generic_new_extended_route(rc->hierarchy, route, 0);
+      xbt_dynar_shrink(TO_ROUTE_FULL(*dst_id, *src_id)->link_list, 0);
+    }
+  }
 }