X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/809ce498a496be4898e4ed739c54b129b55be4c0..788d983294561658582f8dba0aaa81f074df5272:/src/surf/network_gtnets.c

diff --git a/src/surf/network_gtnets.c b/src/surf/network_gtnets.c
index e3a6b0d750..ae7a6cebf4 100644
--- a/src/surf/network_gtnets.c
+++ b/src/surf/network_gtnets.c
@@ -1,67 +1,55 @@
-/* 	$Id$	 */
-
 /* Copyright (c) 2005 Henri Casanova. 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 "network_gtnets_private.h"
+#include "gtnets/gtnets_interface.h"
+#include "xbt/str.h"
+#include "surf_private.h"
+#include "surf/datatypes.h"
 
-XBT_LOG_EXTERNAL_DEFAULT_CATEGORY(surf_network_gtnets);
+/* ************************************************************************** */
+/* *************************** FULL ROUTING ********************************* */
+typedef struct {
+  s_routing_t generic_routing;
+  xbt_dynar_t *routing_table;
+  void *loopback;
+  size_t size_of_link;
+} s_routing_full_t,*routing_full_t;
 
-/* surf_network_resource_t surf_network_resource = NULL; */
-static xbt_dict_t network_link_set = NULL;
 
-/* xbt_dict_t network_card_set = NULL; */
+static double time_to_next_flow_completion = -1;
 
-#if 0
-static int card_number = 0;
-static network_link_GTNETS_t **routing_table = NULL;
-static int *routing_table_size = NULL;
+XBT_LOG_NEW_DEFAULT_SUBCATEGORY(surf_network_gtnets, surf,
+                                "Logging specific to the SURF network GTNetS module");
 
-#define ROUTE(i,j) routing_table[(i)+(j)*card_number]
-#define ROUTE_SIZE(i,j) routing_table_size[(i)+(j)*card_number]
-#endif
+extern routing_t used_routing;
 
 /** QUESTIONS for GTNetS integration
- **   1. What's the deal with name_service and get_resource_name
- **      Right now our local dictionaries contain only integers
- **      and not structures, but it seems that surf_network_resource->common_public
- **      needs something in it, for the above.
+ **   1. Check that we did the right thing with name_service and get_resource_name
  **   2. Right now there is no "kill flow" in our GTNetS implementation. Do we
  **      need to do something about this?
- **   3. We didn't reall do anything with the "action" stuff, which we assume
- **      is fine.
- **   4. We ignore the fact there is some max_duration on flows (see #2 above)
- **   5. share_resources() returns a duration, not a date, right?
- **   6. How do we tell the user that "rates" are not supported?
- **   7. We don't update "remaining" for ongoing flows. Is it bad?
+ **   3. We ignore the fact there is some max_duration on flows (see #2 above)
+ **   4. share_resources() returns a duration, not a date, right?
+ **   5. We don't suppoer "rates"
+ **   6. We don't update "remaining" for ongoing flows. Is it bad?
  **/
 
-/* Free memory for a network link: REMOVED BY KF */
-static void network_link_free(void *nw_link)
-{
-  free(((network_link_GTNETS_t)nw_link)->name);
-  free(nw_link);
-}
+static int src_id = -1;
+static int dst_id = -1;
 
-/* Instantiate a new network link: UNMODIFIED BY HC */
+/* Instantiate a new network link */
 /* name: some name for the link, from the XML */
-/* bw_initial: The bandwidth value            */
-/* bw_trace: unused here                      */
-/* lat_initial: The latency value             */
-/* lat_trace: not used here                   */
-/* state_initial: SURF_NETWORK_LINK_ON        */
-/* state_trace: not used here                 */
-static void network_link_new(char *name,
-	                     double bw,
-	                     double lat)
+/* bw: The bandwidth value            */
+/* lat: The latency value             */
+static void link_new(char *name, double bw, double lat, xbt_dict_t props)
 {
-  static int link_count=-1;
+  static int link_count = -1;
   network_link_GTNETS_t gtnets_link;
 
-  /* KF: Check that the link wasn't added before */
-  if (xbt_dict_get_or_null(network_link_set, name)) {
+  /* If link already exists, nothing to do (FIXME: check that multiple definition match?) */
+  if (xbt_dict_get_or_null(surf_network_model->resource_set, name)) {
     return;
   }
 
@@ -69,7 +57,7 @@ static void network_link_new(char *name,
   link_count++;
 
 /*
-  nw_link->resource = (surf_resource_t) surf_network_resource;
+  nw_link->model =  surf_network_model;
   nw_link->name = name;
   nw_link->bw_current = bw_initial;
   if (bw_trace)
@@ -86,214 +74,310 @@ static void network_link_new(char *name,
 */
 
   /* KF: Add the link to the GTNetS simulation */
-  if (GTNetS_add_link(link_count, bw, lat)) {
-    xbt_assert0(0,"Cannot create GTNetS link");
+  if (gtnets_add_link(link_count, bw, lat)) {
+    xbt_assert0(0, "Cannot create GTNetS link");
   }
 
   /* KF: Insert entry in the dictionary */
-  gtnets_link = xbt_new0(s_network_link_GTNETS_t,1);
-  gtnets_link->name = strcpy(name);
+  gtnets_link = xbt_new0(s_network_link_GTNETS_t, 1);
+  gtnets_link->generic_resource.name = name;
+  gtnets_link->generic_resource.properties = props;
   gtnets_link->bw_current = bw;
   gtnets_link->lat_current = lat;
   gtnets_link->id = link_count;
-  xbt_dict_set(network_link_set, name, gtnets_link, network_link_free);
 
-  return;
-}
+  xbt_dict_set(surf_network_model->resource_set, name, gtnets_link,
+               surf_resource_free);
 
-/* free the network card: REMOVED by KF */
-static void network_card_free(void *nw_card)
-{
-  free(((network_card_GTNETS_t)nw_card)->name);
-  free(nw_card);
+  return;
 }
 
 /* Instantiate a new network card: MODIFYED BY KF */
 static int network_card_new(const char *name)
 {
-  static int card_count=-1;
-
-  /* KF: Check that we haven't seen the network card before */ 
-  if (xbt_dict_get_or_null(network_card_set, name)) 
-    return;
-
-  /* KF: Increment the card counter for GTNetS */
-  card_count++;
-
-  /* KF: just use the dictionary to map link names to link indices */
-  gtnets_network_card = xbt_new0(s_network_card_GTNETS_t,1);
-  gtnets_network_card->name = strcpy(name);
-  gtnets_network_card->id = card_count;
-  xbt_dict_set(network_card_set, name, gtnets_network_card, network_card_free);
+  static int card_count = -1;
+
+  XBT_IN1("(%s)", name);
+  /* KF: Check that we haven't seen the network card before */
+  network_card_GTNETS_t card =
+    surf_model_resource_by_name(surf_network_model, name);
+
+  if (!card) {
+    /* KF: Increment the card counter for GTNetS */
+    card_count++;
+
+    /* KF: just use the dictionary to map link names to link indices */
+    card = xbt_new0(s_network_card_GTNETS_t, 1);
+    card->name = xbt_strdup(name);
+    card->id = card_count;
+    xbt_dict_set(surf_model_resource_set(surf_network_model), name, card,
+                 surf_resource_free);
+  }
 
+  LOG1(xbt_log_priority_trace, "   return %d", card->id);
+  XBT_OUT;
   /* KF: just return the GTNetS ID as the SURF ID */
-  return card_count;
+  return card->id;
 }
 
 /* Instantiate a new route: MODIFY BY KF */
-static void route_new(int src_id, int dst_id, char **links, int nb_link)
+static void route_new(int src_id, int dst_id, network_link_GTNETS_t * links,
+                      int nb_link)
 {
-#if 0
-  network_link_GTNETS_t *link_list = NULL;
   int i;
+  int *gtnets_links;
+  XBT_IN4("(src_id=%d, dst_id=%d, links=%p, nb_link=%d)",
+          src_id, dst_id, links, nb_link);
 
-  ROUTE_SIZE(src_id, dst_id) = nb_link;
-  link_list = (ROUTE(src_id, dst_id) = xbt_new0(network_link_GTNETS_t, nb_link));
+  /* KF: Build the list of gtnets link IDs */
+  gtnets_links = (int *) calloc(nb_link, sizeof(int));
   for (i = 0; i < nb_link; i++) {
-    link_list[i] = xbt_dict_get_or_null(network_link_set, links[i]);
-    free(links[i]);
+    gtnets_links[i] = links[i]->id;
   }
-  free(links);
-#endif
-  int i;
-  int *gtnets_links;
- 
-  /* KF: Build the list of gtnets link IDs */
-  gtnets_links = (int *)calloc(nb_link, sizeof(int));
-  for (i=0; i<nb_link; i++) {
-    gtnets_links[i]=(int)(xbt_dict_get(network_link_set, links[i]))
+
+  /* KF: Create the GTNets route */
+  if (gtnets_add_route(src_id, dst_id, gtnets_links, nb_link)) {
+    xbt_assert0(0, "Cannot create GTNetS route");
+  }
+  XBT_OUT;
+}
+
+/* Instantiate a new route: MODIFY BY KF */
+static void route_onehop_new(int src_id, int dst_id,
+                             network_link_GTNETS_t * links, int nb_link)
+{
+  int linkid;
+
+  if (nb_link != 1) {
+    xbt_assert0(0, "In onehop_new, nb_link should be 1");
   }
 
+  /* KF: Build the linbst of gtnets link IDs */
+  linkid = links[0]->id;
+
   /* KF: Create the GTNets route */
-  if (GTNetS_add_route(src_id, dst_id, gtnets_links, nb_link)) {
-    xbt_assert0(0,"Cannot create GTNetS route");
+  if (gtnets_add_onehop_route(src_id, dst_id, linkid)) {
+    xbt_assert0(0, "Cannot create GTNetS route");
   }
 }
 
+/* Read hosts using a full-fledged topollogy */
+static void routing_full_parse_Shost(void) {
+  int *val = xbt_malloc(sizeof(int));
+  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);
+}
+
+
 /* Parse the XML for a network link */
-static void parse_network_link(void)
+static void parse_link_init(void)
 {
   char *name;
   double bw;
   double lat;
-  e_surf_network_link_state_t state;
-
-  name = xbt_strdup(A_surfxml_network_link_name);
-  surf_parse_get_double(&bw,A_surfxml_network_link_bandwidth);
-  surf_parse_get_double(&lat,A_surfxml_network_link_latency);
-  state = SURF_NETWORK_LINK_ON;
-
-  /* Print values when no traces are specified */
-  {
-    tmgr_trace_t bw_trace;
-    tmgr_trace_t state;
-    tmgr_trace_t lat_trace;
-
-    surf_parse_get_trace(&bw_trace, A_surfxml_network_link_bandwidth_file);
-    surf_parse_get_trace(&lat_trace, A_surfxml_network_link_latency_file);
-    surf_parse_get_trace(&state_trace,A_surfxml_network_link_state_file);
-  
-    if (bw_trace) 
-      WARNING0("The GTNetS network model doesn't support bandwidth state traces");
-    if (lat_trace)
-      WARNING0("The GTNetS network model doesn't support latency state traces");
-    if (state_trace)
-      WARNING0("The GTNetS network model doesn't support link state traces");
-  }
+  e_surf_resource_state_t state;
 
-  /* KF: remove several arguments to network_link_new */
-  network_link_new(name, bw, lat);
-}
+  name = xbt_strdup(A_surfxml_link_id);
+  surf_parse_get_double(&bw, A_surfxml_link_bandwidth);
+  surf_parse_get_double(&lat, A_surfxml_link_latency);
+  state = SURF_RESOURCE_ON;
 
-static int nb_link = 0;
-static char **link_name = NULL;
-static int src_id = -1;
-static int dst_id = -1;
+  tmgr_trace_t bw_trace;
+  tmgr_trace_t state_trace;
+  tmgr_trace_t lat_trace;
+
+  bw_trace = tmgr_trace_new(A_surfxml_link_bandwidth_file);
+  lat_trace = tmgr_trace_new(A_surfxml_link_latency_file);
+  state_trace = tmgr_trace_new(A_surfxml_link_state_file);
+
+  if (bw_trace)
+    INFO0("The GTNetS network model doesn't support bandwidth state traces");
+  if (lat_trace)
+    INFO0("The GTNetS network model doesn't support latency state traces");
+  if (state_trace)
+    INFO0("The GTNetS network model doesn't support link state traces");
+
+  current_property_set = xbt_dict_new();
+  link_new(name, bw, lat, current_property_set);
+}
 
 /* Parses a route from the XML: UNMODIFIED BY HC */
 static void parse_route_set_endpoints(void)
 {
   src_id = network_card_new(A_surfxml_route_src);
   dst_id = network_card_new(A_surfxml_route_dst);
-  nb_link = 0;
-  link_name = NULL;
+  route_action = A_surfxml_route_action;
 }
 
-/* Parses a route element from the XML: UNMODIFIED BY HC */
-static void parse_route_elem(void)
+/* KF*/
+static void parse_route_set_routers(void)
 {
-  nb_link++;
-  link_name = xbt_realloc(link_name, (nb_link) * sizeof(char *));
-  link_name[(nb_link) - 1] = xbt_strdup(A_surfxml_route_element_name);
+  int id = network_card_new(A_surfxml_router_id);
+
+  /* KF: Create the GTNets router */
+  if (gtnets_add_router(id)) {
+    xbt_assert0(0, "Cannot add GTNetS router");
+  }
 }
 
-/* Create the route: UNMODIFIED BY HC */
+/* Create the route (more than one hops): MODIFIED BY KF */
 static void parse_route_set_route(void)
 {
-  route_new(src_id, dst_id, link_name, nb_link);
-}
+  char *name;
+  if (src_id != -1 && dst_id != -1) {
+    name = bprintf("%x#%x", src_id, dst_id);
+    DEBUG0("Calling manage route with route_action=");
+    switch (route_action) {
+           case A_surfxml_route_action_PREPEND:
+				   DEBUG0("PREPEND");
+				   break;
+           case A_surfxml_route_action_POSTPEND:
+				   DEBUG0("POSTPEND");
+   				   break;
+           case A_surfxml_route_action_OVERRIDE:
+   				   DEBUG0("OVERRIDE");
+                   break;
+           default:break;
+    }
+    manage_route(route_table, name, route_action, 0);
 
-/* Main XML parsing */
-static void parse_file(const char *file)
-{
-  /* Figuring out the network links */
-  surf_parse_reset_parser();
-  ETag_surfxml_network_link_fun=parse_network_link;
-  surf_parse_open(file);
-  xbt_assert1((!surf_parse()),"Parse error in %s",file);
-  surf_parse_close();
-
-  /* Figuring out the network cards used */
-  surf_parse_reset_parser();
-  STag_surfxml_route_fun=parse_route_set_endpoints;
-  surf_parse_open(file);
-  xbt_assert1((!surf_parse()),"Parse error in %s",file);
-  surf_parse_close();
-
-  /* Building the routes */
-  surf_parse_reset_parser();
-  STag_surfxml_route_fun=parse_route_set_endpoints;
-  ETag_surfxml_route_element_fun=parse_route_elem;
-  ETag_surfxml_route_fun=parse_route_set_route;
-  surf_parse_open(file);
-  xbt_assert1((!surf_parse()),"Parse error in %s",file);
-  surf_parse_close();
+    DEBUG2("Setting a route from NIC %d to NIC %d", src_id, dst_id);
+    free(name);
+  }
 }
 
-static void *name_service(const char *name)
+static void add_route()
 {
-  return xbt_dict_get_or_null(network_card_set, name);
+  xbt_ex_t e;
+  unsigned int cpt = 0;
+  int link_list_capacity = 0;
+  int nb_link = 0;
+  xbt_dict_cursor_t cursor = NULL;
+  char *key, *data, *end;
+  const char *sep = "#";
+  xbt_dynar_t links, keys;
+  static network_link_GTNETS_t *link_list = NULL;
+
+
+  XBT_IN;
+  xbt_dict_foreach(route_table, cursor, key, data) {
+    char *link = NULL;
+    nb_link = 0;
+    links = (xbt_dynar_t) data;
+    keys = xbt_str_split_str(key, sep);
+
+    link_list_capacity = xbt_dynar_length(links);
+    link_list = xbt_new(network_link_GTNETS_t, link_list_capacity);
+
+    src_id = strtol(xbt_dynar_get_as(keys, 0, char *), &end, 16);
+    dst_id = strtol(xbt_dynar_get_as(keys, 1, char *), &end, 16);
+    xbt_dynar_free(&keys);
+
+    xbt_dynar_foreach(links, cpt, link) {
+      TRY {
+        link_list[nb_link++] =
+          xbt_dict_get(surf_network_model->resource_set, link);
+      }
+      CATCH(e) {
+        RETHROW1("Link %s not found (dict raised this exception: %s)", link);
+      }
+    }
+    if (nb_link == 1){
+      DEBUG0("Calling a one link route");
+      route_onehop_new(src_id, dst_id, link_list, nb_link);
+    }
+  }
+
+  xbt_dict_foreach(route_table, cursor, key, data) {
+    char *link = NULL;
+    nb_link = 0;
+    links = (xbt_dynar_t) data;
+    keys = xbt_str_split_str(key, sep);
+
+    link_list_capacity = xbt_dynar_length(links);
+    link_list = xbt_new(network_link_GTNETS_t, link_list_capacity);
+
+    src_id = strtol(xbt_dynar_get_as(keys, 0, char *), &end, 16);
+    dst_id = strtol(xbt_dynar_get_as(keys, 1, char *), &end, 16);
+    xbt_dynar_free(&keys);
+
+    xbt_dynar_foreach(links, cpt, link) {
+      TRY {
+        link_list[nb_link++] =
+          xbt_dict_get(surf_network_model->resource_set, link);
+      }
+      CATCH(e) {
+        RETHROW1("Link %s not found (dict raised this exception: %s)", link);
+      }
+    }
+    if (nb_link >= 1)
+      route_new(src_id, dst_id, link_list, nb_link);
+  }
+
+  xbt_dict_free(&route_table);
+  if (XBT_LOG_ISENABLED(surf_network_gtnets, xbt_log_priority_debug)) {
+	  gtnets_print_topology();
+  }
+  XBT_OUT;
 }
 
-static const char *get_resource_name(void *resource_id)
+/* Main XML parsing */
+static void define_callbacks(const char *file)
 {
-  return ((network_card_GTNETS_t) resource_id)->name;
+  routing_full_t routing = xbt_new0(s_routing_full_t,1);
+  routing->generic_routing.name = "Full";
+  routing->generic_routing.host_count = 0;
+  routing->generic_routing.host_id = xbt_dict_new();
+  used_routing = (routing_t) routing;
+  surfxml_add_callback(STag_surfxml_host_cb_list, &routing_full_parse_Shost);
+
+  surfxml_add_callback(STag_surfxml_router_cb_list, &parse_route_set_routers);
+  surfxml_add_callback(STag_surfxml_link_cb_list, &parse_link_init);
+  surfxml_add_callback(STag_surfxml_route_cb_list,
+                       &parse_route_set_endpoints);
+  surfxml_add_callback(ETag_surfxml_route_cb_list, &parse_route_set_route);
+  surfxml_add_callback(ETag_surfxml_platform_cb_list, &add_route);
+
 }
 
 /* We do not care about this: only used for traces */
 static int resource_used(void *resource_id)
 {
-  return 0; /* We don't care */
+  return 0;                     /* We don't care */
 }
 
-static int action_free(surf_action_t action)
+static int action_unref(surf_action_t action)
 {
-  action->using--;
-  if(!action->using) {
+  action->refcount--;
+  if (!action->refcount) {
     xbt_swag_remove(action, action->state_set);
     /* KF: No explicit freeing needed for GTNeTS here */
     free(action);
+    return 1;
   }
-  return 1;
-}
-
-static void action_use(surf_action_t action)
-{
-  action->using++;
+  return 0;
 }
 
 static void action_cancel(surf_action_t action)
 {
+  xbt_die("Cannot cancel GTNetS flow");
   return;
 }
 
 static void action_recycle(surf_action_t action)
 {
+  xbt_die("Cannot recycle GTNetS flow");
   return;
 }
 
-static void action_change_state(surf_action_t action,
-				e_surf_action_state_t state)
+static double action_get_remains(surf_action_t action)
+{
+  return action->remains;
+}
+
+static void action_state_set(surf_action_t action,
+                             e_surf_action_state_t state)
 {
 /*   if((state==SURF_ACTION_DONE) || (state==SURF_ACTION_FAILED)) */
 /*     if(((surf_action_network_GTNETS_t)action)->variable) { */
@@ -301,7 +385,7 @@ static void action_change_state(surf_action_t action,
 /*       ((surf_action_network_GTNETS_t)action)->variable = NULL; */
 /*     } */
 
-  surf_action_change_state(action, state);
+  surf_action_state_set(action, state);
   return;
 }
 
@@ -309,18 +393,25 @@ static void action_change_state(surf_action_t action,
 /* share_resources() */
 static double share_resources(double now)
 {
-#if 0
-  s_surf_action_network_GTNETS_t s_action;
-  surf_action_network_GTNETS_t action = NULL;
-  xbt_swag_t running_actions = surf_network_resource->common_public->states.running_action_set;
-#endif
+  xbt_swag_t running_actions = surf_network_model->states.running_action_set;
+
+  //get the first relevant value from the running_actions list
+  if (!xbt_swag_size(running_actions))
+    return -1.0;
 
-  return GTNetS_get_time_to_next_flow_completion();
+  xbt_assert0(time_to_next_flow_completion,
+              "Time to next flow completion not initialized!\n");
+
+  DEBUG0("Calling gtnets_get_time_to_next_flow_completion");
+  time_to_next_flow_completion = gtnets_get_time_to_next_flow_completion();
+  DEBUG1("gtnets_get_time_to_next_flow_completion received %lg", time_to_next_flow_completion);
+
+  return time_to_next_flow_completion;
 }
 
 /* delta: by how many time units the simulation must advance */
 /* In this function: change the state of actions that terminate */
-/* The delta may not come from the network, and thus may be different (smaller) 
+/* The delta may not come from the network, and thus may be different (smaller)
    than the one returned by the function above */
 /* If the delta is a network-caused min, then do not emulate any timer in the
    network simulation, otherwise fake a timer somehow to advance the simulation of min seconds */
@@ -328,73 +419,90 @@ static double share_resources(double now)
 static void update_actions_state(double now, double delta)
 {
   surf_action_network_GTNETS_t action = NULL;
-  surf_action_network_GTNETS_t next_action = NULL;
-  xbt_swag_t running_actions =
-      surf_network_resource->common_public->states.running_action_set;
-
-  double time_to_next_flow_completion =  GTNetS_get_time_to_next_flow_completion();
-  
-  if (time_to_next_flow_completion < delta) {
-    void *metadata; 
-    surf_action_t action;
-
-    if (GTNetS_run_until_next_flow_completion(&metadata)) {
-      xbt_assert0(0,"Cannot run GTNetS simulation until next flow completion");
+  //  surf_action_network_GTNETS_t next_action = NULL;
+  xbt_swag_t running_actions = surf_network_model->states.running_action_set;
+
+  /* If there are no renning flows, just return */
+  if (time_to_next_flow_completion < 0.0) {
+    return;
+  }
+
+  /*KF: if delta == time_to_next_flow_completion, too. */
+  if (time_to_next_flow_completion <= delta) {  /* run until the first flow completes */
+    void **metadata;
+    int i, num_flows;
+
+    num_flows = 0;
+
+    if (gtnets_run_until_next_flow_completion(&metadata, &num_flows)) {
+      xbt_assert0(0,
+                  "Cannot run GTNetS simulation until next flow completion");
+    }
+    if (num_flows < 1) {
+      xbt_assert0(0,
+                  "GTNetS simulation couldn't find a flow that would complete");
     }
 
-    action = (surf_action_t)metadata;
+    xbt_swag_foreach(action, running_actions) {
+      DEBUG2("Action (%p) remains old value: %f", action,
+             action->generic_action.remains);
+      double remain = gtnets_get_flow_rx(action);
+      DEBUG1("Remain value returned by GTNetS : %f", remain);
+      //need to trust this remain value
+      if (remain == 0) {
+        action->generic_action.remains = 0;
+      } else {
+        action->generic_action.remains = action->generic_action.cost - remain;
+      }
+      DEBUG2("Action (%p) remains new value: %f", action,
+             action->generic_action.remains);
+    }
 
-    action->generic_action.remains = 0;
-    action->generic_action.finish =  now + time_to_next_flow_completion;
-    action->generic_action.finish =  SURF_ACTION_DONE;
-    /* TODO: Anything else here? */
-  } else {
-    if (GTNetS_run(delta)) {
-      xbt_assert0(0,"Cannot run GTNetS simulation");
+    for (i = 0; i < num_flows; i++) {
+      action = (surf_action_network_GTNETS_t) (metadata[i]);
+
+      action->generic_action.finish = now + time_to_next_flow_completion;
+      action_state_set((surf_action_t) action, SURF_ACTION_DONE);
+      DEBUG1("----> Action (%p) just terminated", action);
+    }
+
+
+  } else {                      /* run for a given number of seconds */
+    if (gtnets_run(delta)) {
+      xbt_assert0(0, "Cannot run GTNetS simulation");
     }
   }
-  
+
   return;
 }
 
 /* UNUSED HERE: no traces */
 static void update_resource_state(void *id,
-				  tmgr_trace_event_t event_type,
-				  double value)
+                                  tmgr_trace_event_t event_type,
+                                  double value, double date)
 {
+  xbt_assert0(0, "Cannot update model state for GTNetS simulation");
   return;
 }
 
 /* KF: Rate not supported */
-static surf_action_t communicate(void *src, void *dst, double size, double rate)
+/* Max durations are not supported */
+static surf_action_t communicate(const char *src_name, const char *dst_name,
+                                 int src, int dst, double size, double rate)
 {
   surf_action_network_GTNETS_t action = NULL;
-  network_card_GTNETS_t card_src = src;
-  network_card_GTNETS_t card_dst = dst;
-  int route_size = ROUTE_SIZE(card_src->id, card_dst->id);
-  network_link_GTNETS_t *route = ROUTE(card_src->id, card_dst->id);
-  int i;
-
-  xbt_assert2(route_size,"You're trying to send data from %s to %s but there is no connexion between these two cards.", card_src->name, card_dst->name);
-
-  action = xbt_new0(s_surf_action_network_GTNETS_t, 1);
-
-  action->generic_action.using = 1; 
-  action->generic_action.cost = size;
-  action->generic_action.remains = size;
-  action->generic_action.max_duration = NO_MAX_DURATION;
-  action->generic_action.start = surf_get_clock(); 
-  action->generic_action.finish = -1.0; 
-  action->generic_action.resource_type =
-      (surf_resource_t) surf_network_resource;
 
-  action->generic_action.state_set =
-      surf_network_resource->common_public->states.running_action_set;
+  DEBUG4("Setting flow src %d \"%s\", dst %d \"%s\"", src, src_name, dst, dst_name);
 
-  xbt_swag_insert(action, action->generic_action.state_set);
+  action =
+    surf_action_new(sizeof(s_surf_action_network_GTNETS_t), size,
+                    surf_network_model, 0);
 
   /* KF: Add a flow to the GTNets Simulation, associated to this action */
-  GTNetS_create_flow(src->id, dst->id, size, (void *)action);
+  if (gtnets_create_flow(src, dst, size, (void *) action) < 0) {
+    xbt_assert2(0, "Not route between host %s and host %s", src_name,
+                dst_name);
+  }
 
   return (surf_action_t) action;
 }
@@ -402,13 +510,13 @@ static surf_action_t communicate(void *src, void *dst, double size, double rate)
 /* Suspend a flow() */
 static void action_suspend(surf_action_t action)
 {
-  xbt_assert0(0,"action_suspend() not supported for the GTNets network model");
+  THROW_UNIMPLEMENTED;
 }
 
 /* Resume a flow() */
 static void action_resume(surf_action_t action)
 {
-  xbt_assert0(0,"action_resume() not supported for the GTNets network model");
+  THROW_UNIMPLEMENTED;
 }
 
 /* Test whether a flow is suspended */
@@ -419,112 +527,58 @@ static int action_is_suspended(surf_action_t action)
 
 static void finalize(void)
 {
-  int i,j;
-
-  xbt_dict_free(&network_card_set);
-  xbt_dict_free(&network_link_set);
-  xbt_swag_free(surf_network_resource->common_public->states.
-		ready_action_set);
-  xbt_swag_free(surf_network_resource->common_public->states.
-		running_action_set);
-  xbt_swag_free(surf_network_resource->common_public->states.
-		failed_action_set);
-  xbt_swag_free(surf_network_resource->common_public->states.
-		done_action_set);
-  free(surf_network_resource->common_public);
-  free(surf_network_resource->common_private);
-  free(surf_network_resource->extension_public);
-
-  free(surf_network_resource);
-  surf_network_resource = NULL;
-
-#if 0
-  for (i = 0; i < card_number; i++) 
-    for (j = 0; j < card_number; j++) 
-      free(ROUTE(i,j));
-  free(routing_table);
-  routing_table = NULL;
-  free(routing_table_size);
-  routing_table_size = NULL;
-  card_number = 0;
-#endif
+  xbt_dict_free(&surf_network_model->resource_set);
+
+  surf_model_exit(surf_network_model);
 
-  /* ADDED BY KF */
-  GTNetS_finalize();
-  /* END ADDITION */
+  free(surf_network_model);
+  surf_network_model = NULL;
+
+  gtnets_finalize();
 }
 
-static void surf_network_resource_init_internal(void)
+static void surf_network_model_init_internal(void)
 {
-  s_surf_action_t action;
-
-  surf_network_resource = xbt_new0(s_surf_network_resource_t, 1);
-
-  surf_network_resource->common_private =
-      xbt_new0(s_surf_resource_private_t, 1);
-  surf_network_resource->common_public =
-      xbt_new0(s_surf_resource_public_t, 1);
-  surf_network_resource->extension_public =
-      xbt_new0(s_surf_network_resource_extension_public_t, 1);
-
-  surf_network_resource->common_public->states.ready_action_set =
-      xbt_swag_new(xbt_swag_offset(action, state_hookup));
-  surf_network_resource->common_public->states.running_action_set =
-      xbt_swag_new(xbt_swag_offset(action, state_hookup));
-  surf_network_resource->common_public->states.failed_action_set =
-      xbt_swag_new(xbt_swag_offset(action, state_hookup));
-  surf_network_resource->common_public->states.done_action_set =
-      xbt_swag_new(xbt_swag_offset(action, state_hookup));
-
-  surf_network_resource->common_public->name_service = name_service;
-  surf_network_resource->common_public->get_resource_name =
-      get_resource_name;
-  surf_network_resource->common_public->action_get_state =
-      surf_action_get_state;
-  surf_network_resource->common_public->action_use = action_use;
-  surf_network_resource->common_public->action_free = action_free;
-  surf_network_resource->common_public->action_cancel = action_cancel;
-  surf_network_resource->common_public->action_recycle = action_recycle;
-  surf_network_resource->common_public->action_change_state =
-      action_change_state;
-  surf_network_resource->common_public->action_set_data = surf_action_set_data;
-  surf_network_resource->common_public->name = "network";
-
-  surf_network_resource->common_private->resource_used = resource_used;
-  surf_network_resource->common_private->share_resources = share_resources;
-  surf_network_resource->common_private->update_actions_state =
-      update_actions_state;
-  surf_network_resource->common_private->update_resource_state =
-      update_resource_state;
-  surf_network_resource->common_private->finalize = finalize;
-
-  surf_network_resource->common_public->suspend = action_suspend;
-  surf_network_resource->common_public->resume = action_resume;
-  surf_network_resource->common_public->is_suspended = action_is_suspended;
-
-  surf_network_resource->extension_public->communicate = communicate;
-
-  network_link_set = xbt_dict_new();
-  network_card_set = xbt_dict_new();
-
-  /* HC: I am assuming that this stays in for simulation of compute tasks */
-  xbt_assert0(maxmin_system, "surf_init has to be called first!");
+  surf_network_model = surf_model_init();
+
+  surf_network_model->name = "network GTNetS";
+  surf_network_model->action_unref = action_unref;
+  surf_network_model->action_cancel = action_cancel;
+  surf_network_model->action_recycle = action_recycle;
+  surf_network_model->action_state_set = action_state_set;
+  surf_network_model->get_remains = action_get_remains;
+
+  surf_network_model->model_private->resource_used = resource_used;
+  surf_network_model->model_private->share_resources = share_resources;
+  surf_network_model->model_private->update_actions_state =
+    update_actions_state;
+  surf_network_model->model_private->update_resource_state =
+    update_resource_state;
+  surf_network_model->model_private->finalize = finalize;
+
+  surf_network_model->suspend = action_suspend;
+  surf_network_model->resume = action_resume;
+  surf_network_model->is_suspended = action_is_suspended;
+
+  surf_network_model->extension.network.communicate = communicate;
 
   /* KF: Added the initialization for GTNetS interface */
-  if (GTNetS_initialize()) {
+  if (gtnets_initialize()) {
     xbt_assert0(0, "impossible to initialize GTNetS interface");
   }
-}
 
-/* HC: I put this prototype here for now but it will have to go in 
-   src/include/surf.h when it is functionnal. */
-void surf_network_resource_init_GTNETS(const char *filename);
+}
 
-void surf_network_resource_init_GTNETS(const char *filename)
+#ifdef HAVE_GTNETS
+void surf_network_model_init_GTNETS(const char *filename)
 {
-  if (surf_network_resource)
+  if (surf_network_model)
     return;
-  surf_network_resource_init_internal();
-  parse_file(filename);
-  xbt_dynar_push(resource_list, &surf_network_resource);
+  surf_network_model_init_internal();
+  define_callbacks(filename);
+  xbt_dynar_push(model_list, &surf_network_model);
+
+  update_model_description(surf_network_model_description,
+                           "GTNets", surf_network_model);
 }
+#endif