X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/0f76ec2ccc54b00c9f370b3cc5267f67ce76f8a7..38fad1a54be8df52b153c7f9ca4c40ced5dba8c9:/src/surf/network.c diff --git a/src/surf/network.c b/src/surf/network.c index 22d234aa76..202c2c613f 100644 --- a/src/surf/network.c +++ b/src/surf/network.c @@ -6,6 +6,7 @@ * under the terms of the license (GNU LGPL) which comes with this package. */ #include "network_private.h" +#include "xbt/log.h" #define SG_TCP_CTE_GAMMA 20000.0 @@ -17,12 +18,10 @@ surf_network_resource_t surf_network_resource = NULL; static xbt_dict_t network_link_set = NULL; xbt_dict_t network_card_set = NULL; -static int card_number = 0; -static network_link_CM02_t **routing_table = NULL; -static int *routing_table_size = NULL; - -#define ROUTE(i,j) routing_table[(i)+(j)*card_number] -#define ROUTE_SIZE(i,j) routing_table_size[(i)+(j)*card_number] +int card_number = 0; +network_link_CM02_t **routing_table = NULL; +int *routing_table_size = NULL; +static network_link_CM02_t loopback = NULL; static void create_routing_table(void) { @@ -32,8 +31,8 @@ static void create_routing_table(void) static void network_link_free(void *nw_link) { - xbt_free(((network_link_CM02_t)nw_link)->name); - xbt_free(nw_link); + free(((network_link_CM02_t)nw_link)->name); + free(nw_link); } static network_link_CM02_t network_link_new(char *name, @@ -43,7 +42,8 @@ static network_link_CM02_t network_link_new(char *name, tmgr_trace_t lat_trace, e_surf_network_link_state_t state_initial, - tmgr_trace_t state_trace) + tmgr_trace_t state_trace, + e_surf_network_link_sharing_policy_t policy) { network_link_CM02_t nw_link = xbt_new0(s_network_link_CM02_t, 1); @@ -66,6 +66,9 @@ static network_link_CM02_t network_link_new(char *name, nw_link->constraint = lmm_constraint_new(maxmin_system, nw_link, nw_link->bw_current); + if(policy == SURF_NETWORK_LINK_FATPIPE) + lmm_constraint_shared(nw_link->constraint); + xbt_dict_set(network_link_set, name, nw_link, network_link_free); return nw_link; @@ -73,15 +76,13 @@ static network_link_CM02_t network_link_new(char *name, static void network_card_free(void *nw_card) { - xbt_free(((network_card_CM02_t)nw_card)->name); - xbt_free(nw_card); + free(((network_card_CM02_t)nw_card)->name); + free(nw_card); } static int network_card_new(const char *card_name) { - network_card_CM02_t card = NULL; - - xbt_dict_get(network_card_set, card_name, (void *) &card); + network_card_CM02_t card = xbt_dict_get_or_null(network_card_set, card_name); if (!card) { card = xbt_new0(s_network_card_CM02_t, 1); @@ -100,10 +101,11 @@ static void route_new(int src_id, int dst_id, char **links, int nb_link) ROUTE_SIZE(src_id, dst_id) = nb_link; link_list = (ROUTE(src_id, dst_id) = xbt_new0(network_link_CM02_t, nb_link)); for (i = 0; i < nb_link; i++) { - xbt_dict_get(network_link_set, links[i], (void *) &(link_list[i])); - xbt_free(links[i]); + link_list[i] = xbt_dict_get_or_null(network_link_set, links[i]); + xbt_assert1(link_list[i],"Invalid link name \"%s\"",links[i]); + free(links[i]); } - xbt_free(links); + free(links); } static void parse_network_link(void) @@ -114,6 +116,7 @@ static void parse_network_link(void) double lat_initial; tmgr_trace_t lat_trace; e_surf_network_link_state_t state_initial = SURF_NETWORK_LINK_ON; + e_surf_network_link_sharing_policy_t policy_initial = SURF_NETWORK_LINK_SHARED; tmgr_trace_t state_trace; name = xbt_strdup(A_network_link_name); @@ -124,15 +127,22 @@ static void parse_network_link(void) xbt_assert0((A_network_link_state==A_network_link_state_ON)|| (A_network_link_state==A_network_link_state_OFF), - "Invalid state") + "Invalid state"); if (A_network_link_state==A_network_link_state_ON) state_initial = SURF_NETWORK_LINK_ON; - if (A_network_link_state==A_network_link_state_OFF) + else if (A_network_link_state==A_network_link_state_OFF) state_initial = SURF_NETWORK_LINK_OFF; + + if (A_network_link_sharing_policy==A_network_link_sharing_policy_SHARED) + policy_initial = SURF_NETWORK_LINK_SHARED; + else if (A_network_link_sharing_policy==A_network_link_sharing_policy_FATPIPE) + policy_initial = SURF_NETWORK_LINK_FATPIPE; + surf_parse_get_trace(&state_trace,A_network_link_state_file); network_link_new(name, bw_initial, bw_trace, - lat_initial, lat_trace, state_initial, state_trace); + lat_initial, lat_trace, state_initial, state_trace, + policy_initial); } static int nb_link = 0; @@ -162,6 +172,8 @@ static void parse_route_set_route(void) static void parse_file(const char *file) { + int i; + /* Figuring out the network links */ surf_parse_reset_parser(); ETag_network_link_fun=parse_network_link; @@ -186,14 +198,25 @@ static void parse_file(const char *file) surf_parse_open(file); xbt_assert1((!surf_parse()),"Parse error in %s",file); surf_parse_close(); + + /* Adding loopback if needed */ + + for (i = 0; i < card_number; i++) + if(!ROUTE_SIZE(i,i)) { + if(!loopback) + loopback = network_link_new(xbt_strdup("__MSG_loopback__"), + 498.00, NULL, 0.000015, NULL, + SURF_NETWORK_LINK_ON, NULL, + SURF_NETWORK_LINK_FATPIPE); + ROUTE_SIZE(i,i)=1; + ROUTE(i,i) = xbt_new0(network_link_CM02_t, 1); + ROUTE(i,i)[0] = loopback; + } } static void *name_service(const char *name) { - network_card_CM02_t card = NULL; - - xbt_dict_get(network_card_set, name, (void *) &card); - + network_card_CM02_t card = xbt_dict_get_or_null(network_card_set, name); return card; } @@ -208,14 +231,22 @@ static int resource_used(void *resource_id) ((network_link_CM02_t) resource_id)->constraint); } -static void action_free(surf_action_t action) +static int action_free(surf_action_t action) { - xbt_swag_remove(action, action->state_set); - if(((surf_action_network_CM02_t)action)->variable) - lmm_variable_free(maxmin_system, ((surf_action_network_CM02_t)action)->variable); - xbt_free(action); + action->using--; + if(!action->using) { + xbt_swag_remove(action, action->state_set); + if(((surf_action_network_CM02_t)action)->variable) + lmm_variable_free(maxmin_system, ((surf_action_network_CM02_t)action)->variable); + free(action); + return 1; + } + return 0; +} - return; +static void action_use(surf_action_t action) +{ + action->using++; } static void action_cancel(surf_action_t action) @@ -231,11 +262,11 @@ static void action_recycle(surf_action_t action) static void action_change_state(surf_action_t action, e_surf_action_state_t state) { - if((state==SURF_ACTION_DONE) || (state==SURF_ACTION_FAILED)) - if(((surf_action_network_CM02_t)action)->variable) { - lmm_variable_disable(maxmin_system, ((surf_action_network_CM02_t)action)->variable); - ((surf_action_network_CM02_t)action)->variable = NULL; - } +/* if((state==SURF_ACTION_DONE) || (state==SURF_ACTION_FAILED)) */ +/* if(((surf_action_network_CM02_t)action)->variable) { */ +/* lmm_variable_disable(maxmin_system, ((surf_action_network_CM02_t)action)->variable); */ +/* ((surf_action_network_CM02_t)action)->variable = NULL; */ +/* } */ surf_action_change_state(action, state); return; @@ -266,8 +297,10 @@ static void update_actions_state(double now, double delta) surf_action_network_CM02_t next_action = NULL; xbt_swag_t running_actions = surf_network_resource->common_public->states.running_action_set; + /* xbt_swag_t failed_actions = surf_network_resource->common_public->states.failed_action_set; + */ xbt_swag_foreach_safe(action, next_action, running_actions) { deltap = delta; @@ -368,8 +401,11 @@ static surf_action_t communicate(void *src, void *dst, double size, double rate) network_link_CM02_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_CM02_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; @@ -377,7 +413,8 @@ static surf_action_t communicate(void *src, void *dst, double size, double rate) action->generic_action.finish = -1.0; action->generic_action.resource_type = (surf_resource_t) surf_network_resource; - + action->suspended = 0; /* Should be useless because of the + calloc but it seems to help valgrind... */ action->generic_action.state_set = surf_network_resource->common_public->states.running_action_set; @@ -396,15 +433,18 @@ static surf_action_t communicate(void *src, void *dst, double size, double rate) action->variable = lmm_variable_new(maxmin_system, action, 1.0, -1.0, route_size); - if(action->rate<0) - lmm_update_variable_bound(maxmin_system, action->variable, - SG_TCP_CTE_GAMMA / action->lat_current); - else - lmm_update_variable_bound(maxmin_system, action->variable, - min(action->rate,SG_TCP_CTE_GAMMA / action->lat_current)); - - if(route_size == 0) { - action_change_state((surf_action_t) action, SURF_ACTION_DONE); + if(action->rate<0) { + if(action->lat_current>0) + lmm_update_variable_bound(maxmin_system, action->variable, + SG_TCP_CTE_GAMMA / action->lat_current); + else + lmm_update_variable_bound(maxmin_system, action->variable, -1.0); + } else { + if(action->lat_current>0) + lmm_update_variable_bound(maxmin_system, action->variable, + min(action->rate,SG_TCP_CTE_GAMMA / action->lat_current)); + else + lmm_update_variable_bound(maxmin_system, action->variable, action->rate); } for (i = 0; i < route_size; i++) @@ -432,6 +472,11 @@ static int action_is_suspended(surf_action_t action) return ((surf_action_network_CM02_t) action)->suspended; } +static void action_set_max_duration(surf_action_t action, double duration) +{ + action->max_duration = duration; +} + static void finalize(void) { int i,j; @@ -446,19 +491,20 @@ static void finalize(void) failed_action_set); xbt_swag_free(surf_network_resource->common_public->states. done_action_set); - xbt_free(surf_network_resource->common_public); - xbt_free(surf_network_resource->common_private); - xbt_free(surf_network_resource->extension_public); + free(surf_network_resource->common_public); + free(surf_network_resource->common_private); + free(surf_network_resource->extension_public); - xbt_free(surf_network_resource); + free(surf_network_resource); surf_network_resource = NULL; + loopback = NULL; for (i = 0; i < card_number; i++) for (j = 0; j < card_number; j++) - xbt_free(ROUTE(i,j)); - xbt_free(routing_table); + free(ROUTE(i,j)); + free(routing_table); routing_table = NULL; - xbt_free(routing_table_size); + free(routing_table_size); routing_table_size = NULL; card_number = 0; } @@ -491,6 +537,7 @@ static void surf_network_resource_init_internal(void) surf_network_resource->common_public->action_get_state = surf_action_get_state; surf_network_resource->common_public->action_free = action_free; + surf_network_resource->common_public->action_use = action_use; 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 = @@ -509,6 +556,7 @@ static void surf_network_resource_init_internal(void) 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_cpu_resource->common_public->set_max_duration = action_set_max_duration; surf_network_resource->extension_public->communicate = communicate;