X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/0caecdf74ead0d19cc3b13eeb96dcd206243d71e..e1459ae1f3ca27cf74f3592980445151049defd1:/src/mc/LivenessChecker.cpp?ds=sidebyside diff --git a/src/mc/LivenessChecker.cpp b/src/mc/LivenessChecker.cpp index 2ea5b2285b..8dae669664 100644 --- a/src/mc/LivenessChecker.cpp +++ b/src/mc/LivenessChecker.cpp @@ -30,6 +30,7 @@ #include "src/mc/mc_safety.h" #include "src/mc/mc_exit.h" #include "src/mc/Transition.hpp" +#include "src/mc/Session.hpp" XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_liveness, mc, "Logging specific to algorithms for liveness properties verification"); @@ -93,7 +94,7 @@ static bool evaluate_label( } } -Pair::Pair() : num(++mc_stats->expanded_pairs) +Pair::Pair(unsigned long expanded_pairs) : num(expanded_pairs) {} Pair::~Pair() {} @@ -138,8 +139,8 @@ std::shared_ptr LivenessChecker::insertAcceptancePair(simgrid::mc:: explorationStack_.pop_back(); if (dot_output != nullptr) fprintf(dot_output, "\"%d\" -> \"%d\" [%s];\n", - initial_global_state->prev_pair, pair_test->num, - initial_global_state->prev_req.c_str()); + this->previousPair_, pair_test->num, + this->previousRequest_.c_str()); return nullptr; } @@ -158,9 +159,7 @@ 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; + this->previousPair_ = 0; std::shared_ptr> propos = this->getPropositionValues(); @@ -188,7 +187,7 @@ void LivenessChecker::replay() } /* Restore the initial state */ - simgrid::mc::restore_snapshot(initial_global_state->snapshot); + simgrid::mc::session->restoreInitialState(); /* Traverse the stack from the initial state and re-execute the transitions */ int depth = 1; @@ -219,13 +218,12 @@ void LivenessChecker::replay() state.get()); } - mc_model_checker->handle_simcall(state->transition); - mc_model_checker->wait_for_requests(); + this->getSession().execute(state->transition); } /* Update statistics */ - mc_stats->visited_pairs++; - mc_stats->executed_transitions++; + visitedPairsCount_++; + mc_model_checker->executed_transitions++; depth++; @@ -302,6 +300,14 @@ RecordTrace LivenessChecker::getRecordTrace() // override return res; } +void LivenessChecker::logState() // override +{ + Checker::logState(); + XBT_INFO("Expanded pairs = %lu", expandedPairsCount_); + XBT_INFO("Visited pairs = %lu", visitedPairsCount_); + XBT_INFO("Executed transitions = %lu", mc_model_checker->executed_transitions); +} + void LivenessChecker::showAcceptanceCycle(std::size_t depth) { XBT_INFO("*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*"); @@ -311,7 +317,7 @@ void LivenessChecker::showAcceptanceCycle(std::size_t depth) simgrid::mc::dumpRecordPath(); for (auto& s : this->getTextualTrace()) XBT_INFO("%s", s.c_str()); - MC_print_statistics(mc_stats); + simgrid::mc::session->logState(); XBT_INFO("Counter-example depth : %zd", depth); } @@ -361,8 +367,8 @@ int LivenessChecker::main(void) 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.c_str()); + this->previousPair_, visited_num, + this->previousRequest_.c_str()); fflush(dot_output); } XBT_DEBUG("Pair already visited (equal to pair %d), exploration on the current path stopped.", visited_num); @@ -375,14 +381,14 @@ int LivenessChecker::main(void) int req_num = current_pair->graph_state->transition.argument; if (dot_output != nullptr) { - if (initial_global_state->prev_pair != 0 && initial_global_state->prev_pair != current_pair->num) { + if (this->previousPair_ != 0 && this->previousPair_ != current_pair->num) { fprintf(dot_output, "\"%d\" -> \"%d\" [%s];\n", - initial_global_state->prev_pair, current_pair->num, - initial_global_state->prev_req.c_str()); - initial_global_state->prev_req.clear(); + this->previousPair_, current_pair->num, + this->previousRequest_.c_str()); + this->previousRequest_.clear(); } - initial_global_state->prev_pair = current_pair->num; - initial_global_state->prev_req = simgrid::mc::request_get_dot_output(req, req_num); + this->previousPair_ = current_pair->num; + this->previousRequest_ = simgrid::mc::request_get_dot_output(req, req_num); if (current_pair->search_cycle) fprintf(dot_output, "%d [shape=doublecircle];\n", current_pair->num); fflush(dot_output); @@ -392,10 +398,10 @@ int LivenessChecker::main(void) simgrid::mc::request_to_string( req, req_num, simgrid::mc::RequestType::simix).c_str()); - /* Update mc_stats */ - mc_stats->executed_transitions++; + /* Update stats */ + mc_model_checker->executed_transitions++; if (!current_pair->exploration_started) - mc_stats->visited_pairs++; + visitedPairsCount_++; /* Answer the request */ mc_model_checker->handle_simcall(current_pair->graph_state->transition); @@ -423,15 +429,15 @@ int LivenessChecker::main(void) } XBT_INFO("No property violation found."); - MC_print_statistics(mc_stats); + simgrid::mc::session->logState(); return SIMGRID_MC_EXIT_SUCCESS; } std::shared_ptr LivenessChecker::newPair(Pair* current_pair, xbt_automaton_state_t state, std::shared_ptr> propositions) { - std::shared_ptr next_pair = std::make_shared(); + std::shared_ptr next_pair = std::make_shared(++expandedPairsCount_); next_pair->automaton_state = state; - next_pair->graph_state = std::shared_ptr(MC_state_new()); + next_pair->graph_state = std::shared_ptr(MC_state_new(++expandedStatesCount_)); next_pair->atomic_propositions = std::move(propositions); if (current_pair) next_pair->depth = current_pair->depth + 1; @@ -477,16 +483,12 @@ int LivenessChecker::run() { XBT_INFO("Check the liveness property %s", _sg_mc_property_file); MC_automaton_load(_sg_mc_property_file); - mc_model_checker->wait_for_requests(); XBT_DEBUG("Starting the liveness algorithm"); - - /* Create the initial state */ - simgrid::mc::initial_global_state = std::unique_ptr(new s_mc_global_t()); - + simgrid::mc::session->initialize(); this->prepare(); + int res = this->main(); - simgrid::mc::initial_global_state = nullptr; return res; }