X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/e240a25a0e554e068cfac550da70f309459c568d..42d8a4dc6763b3a784a72cb6842dfd1f5115d2a0:/src/mc/mc_global.c diff --git a/src/mc/mc_global.c b/src/mc/mc_global.c index e909cc6042..dbdbc7cc6d 100644 --- a/src/mc/mc_global.c +++ b/src/mc/mc_global.c @@ -24,130 +24,8 @@ XBT_LOG_NEW_CATEGORY(mc, "All MC categories"); XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_global, mc, "Logging specific to MC (global)"); -/* Configuration support */ -e_mc_reduce_t mc_reduce_kind = e_mc_reduce_unset; - -int _sg_do_model_check = 0; -int _sg_mc_checkpoint = 0; -int _sg_mc_sparse_checkpoint = 0; -char *_sg_mc_property_file = NULL; -int _sg_mc_timeout = 0; -int _sg_mc_hash = 0; -int _sg_mc_max_depth = 1000; -int _sg_mc_visited = 0; -char *_sg_mc_dot_output_file = NULL; -int _sg_mc_comms_determinism = 0; -int _sg_mc_send_determinism = 0; -int _sg_mc_safety = 0; -int _sg_mc_liveness = 0; - int user_max_depth_reached = 0; -void _mc_cfg_cb_reduce(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a reduction strategy after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - char *val = xbt_cfg_get_string(_sg_cfg_set, name); - if (!strcasecmp(val, "none")) { - mc_reduce_kind = e_mc_reduce_none; - } else if (!strcasecmp(val, "dpor")) { - mc_reduce_kind = e_mc_reduce_dpor; - } else { - xbt_die("configuration option %s can only take 'none' or 'dpor' as a value", - name); - } -} - -void _mc_cfg_cb_checkpoint(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a checkpointing value after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_checkpoint = xbt_cfg_get_int(_sg_cfg_set, name); -} - -void _mc_cfg_cb_sparse_checkpoint(const char *name, int pos) { - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die("You are specifying a checkpointing value after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_sparse_checkpoint = xbt_cfg_get_int(_sg_cfg_set, name); -} - -void _mc_cfg_cb_property(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a property after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_property_file = xbt_cfg_get_string(_sg_cfg_set, name); -} - -void _mc_cfg_cb_timeout(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a value to enable/disable timeout for wait requests after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_timeout = xbt_cfg_get_boolean(_sg_cfg_set, name); -} - -void _mc_cfg_cb_hash(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a value to enable/disable the use of global hash to speedup state comparaison, but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_hash = xbt_cfg_get_boolean(_sg_cfg_set, name); -} - -void _mc_cfg_cb_max_depth(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a max depth value after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_max_depth = xbt_cfg_get_int(_sg_cfg_set, name); -} - -void _mc_cfg_cb_visited(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a number of stored visited states after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_visited = xbt_cfg_get_int(_sg_cfg_set, name); -} - -void _mc_cfg_cb_dot_output(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a file name for a dot output of graph state after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_dot_output_file = xbt_cfg_get_string(_sg_cfg_set, name); -} - -void _mc_cfg_cb_comms_determinism(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a value to enable/disable the detection of determinism in the communications schemes after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_comms_determinism = xbt_cfg_get_boolean(_sg_cfg_set, name); -} - -void _mc_cfg_cb_send_determinism(const char *name, int pos) -{ - if (_sg_cfg_init_status && !_sg_do_model_check) { - xbt_die - ("You are specifying a value to enable/disable the detection of send-determinism in the communications schemes after the initialization (through MSG_config?), but model-checking was not activated at config time (through --cfg=model-check:1). This won't work, sorry."); - } - _sg_mc_send_determinism = xbt_cfg_get_boolean(_sg_cfg_set, name); -} - /* MC global data structures */ mc_state_t mc_current_state = NULL; char mc_replay_mode = FALSE; @@ -294,6 +172,9 @@ void MC_init() MC_ignore_global_variable("maestro_stack_start"); MC_ignore_global_variable("maestro_stack_end"); MC_ignore_global_variable("smx_total_comms"); + MC_ignore_global_variable("communications_pattern"); + MC_ignore_global_variable("initial_communications_pattern"); + MC_ignore_global_variable("incomplete_communications_pattern"); if (MC_is_active()) { MC_ignore_global_variable("mc_diff_info"); @@ -441,7 +322,7 @@ void MC_exit(void) //xbt_abort(); } -int SIMIX_pre_mc_random(smx_simcall_t simcall, int min, int max) +int simcall_HANDLER_mc_random(smx_simcall_t simcall, int min, int max) { return simcall->mc_value; @@ -465,10 +346,10 @@ void MC_wait_for_requests(void) while (!xbt_dynar_is_empty(simix_global->process_to_run)) { SIMIX_process_runall(); - xbt_dynar_foreach(simix_global->process_that_ran, iter, process) { + xbt_dynar_foreach(simix_global->process_that_ran, iter, process) { req = &process->simcall; if (req->call != SIMCALL_NONE && !MC_request_is_visible(req)) - SIMIX_simcall_pre(req, 0); + SIMIX_simcall_handle(req, 0); } } } @@ -480,8 +361,7 @@ int MC_deadlock_check() if (xbt_swag_size(simix_global->process_list)) { deadlock = TRUE; xbt_swag_foreach(process, simix_global->process_list) { - if (process->simcall.call != SIMCALL_NONE - && MC_request_is_enabled(&process->simcall)) { + if (MC_request_is_enabled(&process->simcall)) { deadlock = FALSE; break; } @@ -490,6 +370,31 @@ int MC_deadlock_check() return deadlock; } +void mc_update_comm_pattern(mc_call_type call_type, smx_simcall_t req, int value, xbt_dynar_t pattern) { + switch(call_type) { + case MC_CALL_TYPE_NONE: + break; + case MC_CALL_TYPE_SEND: + case MC_CALL_TYPE_RECV: + get_comm_pattern(pattern, req, call_type); + break; + case MC_CALL_TYPE_WAIT: + { + smx_synchro_t current_comm = NULL; + if (call_type == MC_CALL_TYPE_WAIT) + current_comm = simcall_comm_wait__get__comm(req); + else + current_comm = xbt_dynar_get_as(simcall_comm_waitany__get__comms(req), value, smx_synchro_t); + // First wait only must be considered: + if (current_comm->comm.refcount == 1) + complete_comm_pattern(pattern, current_comm); + break; + } + default: + xbt_die("Unexpected call type %i", (int)call_type); + } +} + /** * \brief Re-executes from the state at position start all the transitions indicated by * a given model-checker stack. @@ -500,14 +405,12 @@ void MC_replay(xbt_fifo_t stack, int start) { int raw_mem = (mmalloc_get_current_heap() == mc_heap); - int value, i = 1, count = 1; + int value, i = 1, count = 1, j; char *req_str; smx_simcall_t req = NULL, saved_req = NULL; xbt_fifo_item_t item, start_item; mc_state_t state; smx_process_t process = NULL; - int comm_pattern = 0; - smx_action_t current_comm; XBT_DEBUG("**** Begin Replay ****"); @@ -529,7 +432,7 @@ void MC_replay(xbt_fifo_t stack, int start) MC_SET_MC_HEAP; - if ((mc_reduce_kind == e_mc_reduce_dpor) && !_sg_mc_comms_determinism && !_sg_mc_send_determinism) { + if (mc_reduce_kind == e_mc_reduce_dpor) { xbt_dict_reset(first_enabled_state); xbt_swag_foreach(process, simix_global->process_list) { if (MC_process_is_enabled(process)) { @@ -542,8 +445,12 @@ void MC_replay(xbt_fifo_t stack, int start) } if (_sg_mc_comms_determinism || _sg_mc_send_determinism) { - xbt_dynar_reset(communications_pattern); - xbt_dynar_reset(incomplete_communications_pattern); + for (j=0; jissuer->pid); - xbt_dict_remove(first_enabled_state, key); + if(xbt_dict_get_or_null(first_enabled_state, key)) + xbt_dict_remove(first_enabled_state, key); xbt_free(key); MC_SET_STD_HEAP; } @@ -578,46 +486,24 @@ void MC_replay(xbt_fifo_t stack, int start) } /* TODO : handle test and testany simcalls */ + mc_call_type call = MC_CALL_TYPE_NONE; if (_sg_mc_comms_determinism || _sg_mc_send_determinism) { - if (req->call == SIMCALL_COMM_ISEND) - comm_pattern = 1; - else if (req->call == SIMCALL_COMM_IRECV) - comm_pattern = 2; - else if (req->call == SIMCALL_COMM_WAIT) - comm_pattern = 3; - else if (req->call == SIMCALL_COMM_WAITANY) - comm_pattern = 4; + call = mc_get_call_type(req); } - SIMIX_simcall_pre(req, value); + SIMIX_simcall_handle(req, value); if (_sg_mc_comms_determinism || _sg_mc_send_determinism) { MC_SET_MC_HEAP; - if (comm_pattern == 1 || comm_pattern == 2) { - get_comm_pattern(communications_pattern, req, comm_pattern); - } else if (comm_pattern == 3 /* || comm_pattern == 4 */ ) { - current_comm = simcall_comm_wait__get__comm(req); - if (current_comm->comm.refcount == 1) { /* First wait only must be considered */ - complete_comm_pattern(communications_pattern, current_comm); - } - } else if (comm_pattern == 4 /* || comm_pattern == 4 */ ) { - current_comm = - xbt_dynar_get_as(simcall_comm_waitany__get__comms(req), value, - smx_action_t); - if (current_comm->comm.refcount == 1) { /* First wait only must be considered */ - complete_comm_pattern(communications_pattern, current_comm); - } - } + mc_update_comm_pattern(call, req, value, communications_pattern); MC_SET_STD_HEAP; - comm_pattern = 0; } - MC_wait_for_requests(); count++; - if ((mc_reduce_kind == e_mc_reduce_dpor) && !_sg_mc_comms_determinism && !_sg_mc_send_determinism) { + if (mc_reduce_kind == e_mc_reduce_dpor) { MC_SET_MC_HEAP; /* Insert in dict all enabled processes */ xbt_swag_foreach(process, simix_global->process_list) { @@ -655,12 +541,7 @@ void MC_replay_liveness(xbt_fifo_t stack, int all_stack) initial_global_state->raw_mem_set = (mmalloc_get_current_heap() == mc_heap); - int value; - char *req_str; - smx_simcall_t req = NULL, saved_req = NULL; xbt_fifo_item_t item; - mc_state_t state; - mc_pair_t pair; int depth = 1; XBT_DEBUG("**** Begin Replay ****"); @@ -673,14 +554,17 @@ void MC_replay_liveness(xbt_fifo_t stack, int all_stack) if (!initial_global_state->raw_mem_set) MC_SET_STD_HEAP; - if (all_stack) { - - item = xbt_fifo_get_last_item(stack); + /* Traverse the stack from the initial state and re-execute the transitions */ + for (item = xbt_fifo_get_last_item(stack); + all_stack ? depth <= xbt_fifo_size(stack) : item != xbt_fifo_get_first_item(stack); + item = xbt_fifo_get_prev_item(item)) { - while (depth <= xbt_fifo_size(stack)) { + mc_pair_t pair = (mc_pair_t) xbt_fifo_get_item_content(item); - pair = (mc_pair_t) xbt_fifo_get_item_content(item); - state = (mc_state_t) pair->graph_state; + mc_state_t state = (mc_state_t) pair->graph_state; + smx_simcall_t req = NULL, saved_req = NULL; + int value; + char *req_str; if (pair->requests > 0) { @@ -688,7 +572,7 @@ void MC_replay_liveness(xbt_fifo_t stack, int all_stack) //XBT_DEBUG("SavedReq->call %u", saved_req->call); if (saved_req != NULL) { - /* because we got a copy of the executed request, we have to fetch the + /* because we got a copy of the executed request, we have to fetch the real one, pointed by the request field of the issuer process */ req = &saved_req->issuer->simcall; //XBT_DEBUG("Req->call %u", req->call); @@ -702,60 +586,17 @@ void MC_replay_liveness(xbt_fifo_t stack, int all_stack) } - SIMIX_simcall_pre(req, value); + SIMIX_simcall_handle(req, value); MC_wait_for_requests(); } - depth++; - /* Update statistics */ mc_stats->visited_pairs++; mc_stats->executed_transitions++; - item = xbt_fifo_get_prev_item(item); - } - - } else { - - /* Traverse the stack from the initial state and re-execute the transitions */ - for (item = xbt_fifo_get_last_item(stack); - item != xbt_fifo_get_first_item(stack); - item = xbt_fifo_get_prev_item(item)) { - - pair = (mc_pair_t) xbt_fifo_get_item_content(item); - state = (mc_state_t) pair->graph_state; - - if (pair->requests > 0) { - - saved_req = MC_state_get_executed_request(state, &value); - //XBT_DEBUG("SavedReq->call %u", saved_req->call); - - if (saved_req != NULL) { - /* because we got a copy of the executed request, we have to fetch the - real one, pointed by the request field of the issuer process */ - req = &saved_req->issuer->simcall; - //XBT_DEBUG("Req->call %u", req->call); - - /* Debug information */ - if (XBT_LOG_ISENABLED(mc_global, xbt_log_priority_debug)) { - req_str = MC_request_to_string(req, value); - XBT_DEBUG("Replay (depth = %d) : %s (%p)", depth, req_str, state); - xbt_free(req_str); - } - - } - - SIMIX_simcall_pre(req, value); - MC_wait_for_requests(); - } - depth++; - /* Update statistics */ - mc_stats->visited_pairs++; - mc_stats->executed_transitions++; } - } XBT_DEBUG("**** End Replay ****"); @@ -885,6 +726,8 @@ void MC_dump_stack_liveness(xbt_fifo_t stack) void MC_print_statistics(mc_stats_t stats) { + xbt_mheap_t previous_heap = mmalloc_get_current_heap(); + if (stats->expanded_pairs == 0) { XBT_INFO("Expanded states = %lu", stats->expanded_states); XBT_INFO("Visited states = %lu", stats->visited_states); @@ -906,7 +749,7 @@ void MC_print_statistics(mc_stats_t stats) XBT_INFO("Send-deterministic : %s", !initial_global_state->send_deterministic ? "No" : "Yes"); } - MC_SET_STD_HEAP; + mmalloc_set_current_heap(previous_heap); } void MC_assert(int prop)