X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/6f64fa47ccef54748112a4f1910633ffd53f1a79..7215139acf8ac34d2b9cc0de4baba96fbbd64171:/src/mc/LivenessChecker.cpp diff --git a/src/mc/LivenessChecker.cpp b/src/mc/LivenessChecker.cpp index 40ad38df1e..c1ceaabb8c 100644 --- a/src/mc/LivenessChecker.cpp +++ b/src/mc/LivenessChecker.cpp @@ -157,33 +157,16 @@ void LivenessChecker::removeAcceptancePair(int pair_num) void LivenessChecker::prepare(void) { mc_model_checker->wait_for_requests(); - initial_global_state->snapshot = simgrid::mc::take_snapshot(0); initial_global_state->prev_pair = 0; + // For each initial state of the property automaton, push a + // (application_state, automaton_state) pair to the exploration stack: unsigned int cursor = 0; xbt_automaton_state_t automaton_state; - - xbt_dynar_foreach(simgrid::mc::property_automaton->states, cursor, automaton_state) { - if (automaton_state->type == -1) { /* Initial automaton state */ - - std::shared_ptr initial_pair = std::make_shared(); - initial_pair->automaton_state = automaton_state; - initial_pair->graph_state = std::shared_ptr(MC_state_new()); - initial_pair->atomic_propositions = this->getPropositionValues(); - initial_pair->depth = 1; - - /* Get enabled processes and insert them in the interleave set of the graph_state */ - for (auto& p : mc_model_checker->process().simix_processes()) - if (simgrid::mc::process_is_enabled(&p.copy)) - MC_state_interleave_process(initial_pair->graph_state.get(), &p.copy); - - initial_pair->requests = MC_state_interleave_size(initial_pair->graph_state.get()); - initial_pair->search_cycle = false; - - livenessStack_.push_back(std::move(initial_pair)); - } - } + xbt_dynar_foreach(simgrid::mc::property_automaton->states, cursor, automaton_state) + if (automaton_state->type == -1) + livenessStack_.push_back(this->newPair(nullptr, automaton_state)); } @@ -353,11 +336,7 @@ std::vector LivenessChecker::getTextualTrace() // override int LivenessChecker::main(void) { - int visited_num = -1; - while (!livenessStack_.empty()){ - - /* Get current pair */ std::shared_ptr current_pair = livenessStack_.back(); /* Update current state in buchi automaton */ @@ -373,97 +352,77 @@ int LivenessChecker::main(void) continue; } - std::shared_ptr reached_pair; - if (current_pair->automaton_state->type == 1 && !current_pair->exploration_started - && (reached_pair = this->insertAcceptancePair(current_pair.get())) == nullptr) { - this->showAcceptanceCycle(current_pair->depth); - return SIMGRID_MC_EXIT_LIVENESS; + std::shared_ptr reached_pair; + if (current_pair->automaton_state->type == 1 && !current_pair->exploration_started + && (reached_pair = this->insertAcceptancePair(current_pair.get())) == nullptr) { + this->showAcceptanceCycle(current_pair->depth); + return SIMGRID_MC_EXIT_LIVENESS; + } + + /* Pair already visited ? stop the exploration on the current path */ + int visited_num = -1; + if ((!current_pair->exploration_started) + && (visited_num = this->insertVisitedPair( + reached_pair, current_pair.get())) != -1) { + if (dot_output != nullptr){ + fprintf(dot_output, "\"%d\" -> \"%d\" [%s];\n", initial_global_state->prev_pair, visited_num, initial_global_state->prev_req); + fflush(dot_output); + } + XBT_DEBUG("Pair already visited (equal to pair %d), exploration on the current path stopped.", visited_num); + current_pair->requests = 0; + this->backtrack(); + continue; + } + + int value; + smx_simcall_t req = MC_state_get_request(current_pair->graph_state.get(), &value); + + if (dot_output != nullptr) { + if (initial_global_state->prev_pair != 0 && initial_global_state->prev_pair != current_pair->num) { + fprintf(dot_output, "\"%d\" -> \"%d\" [%s];\n", initial_global_state->prev_pair, current_pair->num, initial_global_state->prev_req); + xbt_free(initial_global_state->prev_req); } + initial_global_state->prev_pair = current_pair->num; + initial_global_state->prev_req = simgrid::mc::request_get_dot_output(req, value); + if (current_pair->search_cycle) + fprintf(dot_output, "%d [shape=doublecircle];\n", current_pair->num); + fflush(dot_output); + } - /* Pair already visited ? stop the exploration on the current path */ - if ((!current_pair->exploration_started) - && (visited_num = this->insertVisitedPair( - reached_pair, current_pair.get())) != -1) { + char* req_str = simgrid::mc::request_to_string(req, value, simgrid::mc::RequestType::simix); + XBT_DEBUG("Execute: %s", req_str); + xbt_free(req_str); - if (dot_output != nullptr){ - fprintf(dot_output, "\"%d\" -> \"%d\" [%s];\n", initial_global_state->prev_pair, visited_num, initial_global_state->prev_req); - fflush(dot_output); - } + /* Set request as executed */ + MC_state_set_executed_request(current_pair->graph_state.get(), req, value); + + /* Update mc_stats */ + mc_stats->executed_transitions++; + if (!current_pair->exploration_started) + mc_stats->visited_pairs++; - XBT_DEBUG("Pair already visited (equal to pair %d), exploration on the current path stopped.", visited_num); - current_pair->requests = 0; - this->backtrack(); - - }else{ - - int value; - smx_simcall_t req = MC_state_get_request(current_pair->graph_state.get(), &value); - - if (dot_output != nullptr) { - if (initial_global_state->prev_pair != 0 && initial_global_state->prev_pair != current_pair->num) { - fprintf(dot_output, "\"%d\" -> \"%d\" [%s];\n", initial_global_state->prev_pair, current_pair->num, initial_global_state->prev_req); - xbt_free(initial_global_state->prev_req); - } - initial_global_state->prev_pair = current_pair->num; - initial_global_state->prev_req = simgrid::mc::request_get_dot_output(req, value); - if (current_pair->search_cycle) - fprintf(dot_output, "%d [shape=doublecircle];\n", current_pair->num); - fflush(dot_output); - } - - char* req_str = simgrid::mc::request_to_string(req, value, simgrid::mc::RequestType::simix); - XBT_DEBUG("Execute: %s", req_str); - xbt_free(req_str); - - /* Set request as executed */ - MC_state_set_executed_request(current_pair->graph_state.get(), req, value); - - /* Update mc_stats */ - mc_stats->executed_transitions++; - if (!current_pair->exploration_started) - mc_stats->visited_pairs++; - - /* Answer the request */ - simgrid::mc::handle_simcall(req, value); - - /* Wait for requests (schedules processes) */ - mc_model_checker->wait_for_requests(); - - current_pair->requests--; - current_pair->exploration_started = true; - - /* Get values of atomic propositions (variables used in the property formula) */ - std::vector prop_values = this->getPropositionValues(); - - /* Evaluate enabled/true transitions in automaton according to atomic propositions values and create new pairs */ - int cursor = xbt_dynar_length(current_pair->automaton_state->out) - 1; - while (cursor >= 0) { - xbt_automaton_transition_t transition_succ = (xbt_automaton_transition_t)xbt_dynar_get_as(current_pair->automaton_state->out, cursor, xbt_automaton_transition_t); - if (evaluate_label(transition_succ->label, prop_values)) { - std::shared_ptr next_pair = std::make_shared(); - next_pair->graph_state = std::shared_ptr(MC_state_new()); - next_pair->automaton_state = transition_succ->dst; - next_pair->atomic_propositions = this->getPropositionValues(); - next_pair->depth = current_pair->depth + 1; - /* Get enabled processes and insert them in the interleave set of the next graph_state */ - for (auto& p : mc_model_checker->process().simix_processes()) - if (simgrid::mc::process_is_enabled(&p.copy)) - MC_state_interleave_process(next_pair->graph_state.get(), &p.copy); - - next_pair->requests = MC_state_interleave_size(next_pair->graph_state.get()); - - /* FIXME : get search_cycle value for each acceptant state */ - if (next_pair->automaton_state->type == 1 || current_pair->search_cycle) - next_pair->search_cycle = true; - - /* Add new pair to the exploration stack */ - livenessStack_.push_back(std::move(next_pair)); - - } - cursor--; - } - - } /* End of visited_pair test */ + /* Answer the request */ + simgrid::mc::handle_simcall(req, value); + + /* Wait for requests (schedules processes) */ + mc_model_checker->wait_for_requests(); + + current_pair->requests--; + current_pair->exploration_started = true; + + /* Get values of atomic propositions (variables used in the property formula) */ + std::vector prop_values = this->getPropositionValues(); + + // For each enabled transition in the property automaton, push a + // (application_state, automaton_state) pair to the exploration stack: + int cursor = xbt_dynar_length(current_pair->automaton_state->out) - 1; + while (cursor >= 0) { + xbt_automaton_transition_t transition_succ = (xbt_automaton_transition_t)xbt_dynar_get_as(current_pair->automaton_state->out, cursor, xbt_automaton_transition_t); + if (evaluate_label(transition_succ->label, prop_values)) + livenessStack_.push_back(this->newPair( + current_pair.get(), transition_succ->dst)); + cursor--; + } } @@ -472,6 +431,30 @@ int LivenessChecker::main(void) return SIMGRID_MC_EXIT_SUCCESS; } +std::shared_ptr LivenessChecker::newPair(Pair* current_pair, xbt_automaton_state_t state) +{ + std::shared_ptr next_pair = std::make_shared(); + next_pair->automaton_state = state; + next_pair->graph_state = std::shared_ptr(MC_state_new()); + next_pair->atomic_propositions = this->getPropositionValues(); + if (current_pair) + next_pair->depth = current_pair->depth + 1; + else + next_pair->depth = 1; + /* Get enabled processes and insert them in the interleave set of the next graph_state */ + for (auto& p : mc_model_checker->process().simix_processes()) + if (simgrid::mc::process_is_enabled(&p.copy)) + MC_state_interleave_process(next_pair->graph_state.get(), &p.copy); + next_pair->requests = MC_state_interleave_size(next_pair->graph_state.get()); + /* FIXME : get search_cycle value for each acceptant state */ + if (next_pair->automaton_state->type == 1 || + (current_pair && current_pair->search_cycle)) + next_pair->search_cycle = true; + else + next_pair->search_cycle = false; + return std::move(next_pair); +} + void LivenessChecker::backtrack() { /* Traverse the stack backwards until a pair with a non empty interleave