X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/f58e6bec42872e9b11d157fcbef97473667ed272..0abc9e8c70d2f3c5c98ed3429d4b630683f77c22:/src/mc/checker/LivenessChecker.cpp diff --git a/src/mc/checker/LivenessChecker.cpp b/src/mc/checker/LivenessChecker.cpp index c82522e619..e9a2fe2cda 100644 --- a/src/mc/checker/LivenessChecker.cpp +++ b/src/mc/checker/LivenessChecker.cpp @@ -54,12 +54,9 @@ static bool evaluate_label(const xbt_automaton_exp_label* l, std::vector co case xbt_automaton_exp_label::AUT_NOT: return not evaluate_label(l->u.exp_not, values); case xbt_automaton_exp_label::AUT_PREDICAT:{ - unsigned int cursor = 0; - xbt_automaton_propositional_symbol_t p = nullptr; - xbt_dynar_foreach(simgrid::mc::property_automaton->propositional_symbols, cursor, p) { - if (std::strcmp(xbt_automaton_propositional_symbol_get_name(p), l->u.predicat) == 0) - return values[cursor] != 0; - } + auto cursor = mcapi::get().compare_automaton_exp_lable(l, values); + if(cursor >= 0) + return values[cursor] != 0; xbt_die("Missing predicate"); break; } @@ -75,11 +72,7 @@ Pair::Pair(unsigned long expanded_pairs) : num(expanded_pairs) std::shared_ptr> LivenessChecker::get_proposition_values() const { - std::vector values; - unsigned int cursor = 0; - xbt_automaton_propositional_symbol_t ps = nullptr; - xbt_dynar_foreach (mc::property_automaton->propositional_symbols, cursor, ps) - values.push_back(xbt_automaton_propositional_symbol_evaluate(ps)); + auto values = mcapi::get().automaton_propositional_symbol_evaluate(); return std::make_shared>(std::move(values)); } @@ -88,13 +81,13 @@ std::shared_ptr LivenessChecker::insert_acceptance_pair(simgrid::mc auto new_pair = std::make_shared(pair->num, pair->automaton_state, pair->atomic_propositions, pair->graph_state); - auto res = boost::range::equal_range(acceptance_pairs_, new_pair.get(), DerefAndCompareByActorsCountAndUsedHeap()); + auto res = boost::range::equal_range(acceptance_pairs_, new_pair.get(), mcapi::get().compare_pair()); if (pair->search_cycle) for (auto i = res.first; i != res.second; ++i) { std::shared_ptr const& pair_test = *i; - if (xbt_automaton_state_compare(pair_test->automaton_state, new_pair->automaton_state) != 0 || + if (mcapi::get().automaton_state_compare(pair_test->automaton_state, new_pair->automaton_state) != 0 || *(pair_test->atomic_propositions) != *(new_pair->atomic_propositions) || - not snapshot_equal(pair_test->graph_state->system_state_.get(), new_pair->graph_state->system_state_.get())) + not mcapi::get().snapshot_equal(pair_test->graph_state->system_state_.get(), new_pair->graph_state->system_state_.get())) continue; XBT_INFO("Pair %d already reached (equal to pair %d) !", new_pair->num, pair_test->num); exploration_stack_.pop_back(); @@ -154,7 +147,7 @@ void LivenessChecker::replay() /* Debug information */ XBT_DEBUG("Replay (depth = %d) : %s (%p)", depth, - request_to_string(req, req_num, simgrid::mc::RequestType::simix).c_str(), state.get()); + mcapi::get().request_to_string(req, req_num, simgrid::mc::RequestType::simix).c_str(), state.get()); this->get_session().execute(state->transition_); } @@ -180,13 +173,13 @@ int LivenessChecker::insert_visited_pair(std::shared_ptr visited_pa visited_pair = std::make_shared(pair->num, pair->automaton_state, pair->atomic_propositions, pair->graph_state); - auto range = boost::range::equal_range(visited_pairs_, visited_pair.get(), DerefAndCompareByActorsCountAndUsedHeap()); + auto range = boost::range::equal_range(visited_pairs_, visited_pair.get(), mcapi::get().compare_pair()); for (auto i = range.first; i != range.second; ++i) { const VisitedPair* pair_test = i->get(); - if (xbt_automaton_state_compare(pair_test->automaton_state, visited_pair->automaton_state) != 0 || + if (mcapi::get().automaton_state_compare(pair_test->automaton_state, visited_pair->automaton_state) != 0 || *(pair_test->atomic_propositions) != *(visited_pair->atomic_propositions) || - not snapshot_equal(pair_test->graph_state->system_state_.get(), visited_pair->graph_state->system_state_.get())) + not mcapi::get().snapshot_equal(pair_test->graph_state->system_state_.get(), visited_pair->graph_state->system_state_.get())) continue; if (pair_test->other_num == -1) visited_pair->other_num = pair_test->num; @@ -243,7 +236,7 @@ void LivenessChecker::show_acceptance_cycle(std::size_t depth) XBT_INFO("Counter-example that violates formula:"); for (auto const& s : this->get_textual_trace()) XBT_INFO(" %s", s.c_str()); - mcapi::get().mc_dump_record_path(); + mcapi::get().dump_record_path(); mcapi::get().log_state(); XBT_INFO("Counter-example depth: %zu", depth); } @@ -255,7 +248,7 @@ std::vector LivenessChecker::get_textual_trace() // override int req_num = pair->graph_state->transition_.argument_; smx_simcall_t req = &pair->graph_state->executed_req_; if (req->call_ != simix::Simcall::NONE) - trace.push_back(request_to_string(req, req_num, RequestType::executed)); + trace.push_back(mcapi::get().request_to_string(req, req_num, RequestType::executed)); } return trace; } @@ -274,8 +267,9 @@ std::shared_ptr LivenessChecker::create_pair(const Pair* current_pair, xbt else next_pair->depth = 1; /* Get enabled actors and insert them in the interleave set of the next graph_state */ - for (auto& actor : mc_model_checker->get_remote_simulation().actors()) - if (mc::actor_is_enabled(actor.copy.get_buffer())) + auto actors = mcapi::get().get_actors(); + for (auto& actor : actors) + if (mcapi::get().actor_is_enabled(actor.copy.get_buffer()->get_pid())) next_pair->graph_state->add_interleaving_set(actor.copy.get_buffer()); next_pair->requests = next_pair->graph_state->interleave_size(); /* FIXME : get search_cycle value for each accepting state */ @@ -311,10 +305,10 @@ void LivenessChecker::backtrack() void LivenessChecker::run() { XBT_INFO("Check the liveness property %s", _sg_mc_property_file.get().c_str()); - MC_automaton_load(_sg_mc_property_file.get().c_str()); + mcapi::get().automaton_load(_sg_mc_property_file.get().c_str()); XBT_DEBUG("Starting the liveness algorithm"); - mc::session->initialize(); + mcapi::get().session_initialize(); /* Initialize */ this->previous_pair_ = 0; @@ -323,18 +317,18 @@ void LivenessChecker::run() // 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 (mc::property_automaton->states, cursor, automaton_state) + auto automaton_stack = mcapi::get().get_automaton_state(); + std::for_each(automaton_stack.begin(), automaton_stack.end(), [&](xbt_automaton_state_t const& automaton_state) { if (automaton_state->type == -1) exploration_stack_.push_back(this->create_pair(nullptr, automaton_state, propos)); + }); /* Actually run the double DFS search for counter-examples */ while (not exploration_stack_.empty()) { std::shared_ptr current_pair = exploration_stack_.back(); /* Update current state in buchi automaton */ - mc::property_automaton->current_state = current_pair->automaton_state; + mcapi::get().set_property_automaton(current_pair->automaton_state); XBT_DEBUG( "********************* ( Depth = %d, search_cycle = %d, interleave size = %zu, pair_num = %d, requests = %d)", @@ -381,24 +375,25 @@ void LivenessChecker::run() this->previous_request_.clear(); } this->previous_pair_ = current_pair->num; - this->previous_request_ = request_get_dot_output(req, req_num); + this->previous_request_ = mcapi::get().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); } - XBT_DEBUG("Execute: %s", request_to_string(req, req_num, RequestType::simix).c_str()); + XBT_DEBUG("Execute: %s", mcapi::get().request_to_string(req, req_num, RequestType::simix).c_str()); /* Update stats */ - mc_model_checker->executed_transitions++; + mcapi::get().mc_inc_executed_trans(); + if (not current_pair->exploration_started) visited_pairs_count_++; /* Answer the request */ - mc_model_checker->handle_simcall(current_pair->graph_state->transition_); + mcapi::get().handle_simcall(current_pair->graph_state->transition_); /* Wait for requests (schedules processes) */ - mc_model_checker->wait_for_requests(); + mcapi::get().mc_wait_for_requests(); current_pair->requests--; current_pair->exploration_started = true; @@ -408,16 +403,17 @@ void LivenessChecker::run() // For each enabled transition in the property automaton, push a // (application_state, automaton_state) pair to the exploration stack: - for (int i = xbt_dynar_length(current_pair->automaton_state->out) - 1; i >= 0; i--) { + for (int i = mcapi::get().get_dynar_length(current_pair->automaton_state->out) - 1; i >= 0; i--) { const xbt_automaton_transition* transition_succ = xbt_dynar_get_as(current_pair->automaton_state->out, i, xbt_automaton_transition_t); - if (evaluate_label(transition_succ->label, *prop_values)) + auto transition_succ_label = mcapi::get().get_automaton_transition_label(current_pair->automaton_state->out, i); + if (evaluate_label(transition_succ_label, *prop_values)) exploration_stack_.push_back(this->create_pair(current_pair.get(), transition_succ->dst, prop_values)); - } + } } XBT_INFO("No property violation found."); - mc::session->log_state(); + mcapi::get().log_state(); } Checker* createLivenessChecker(Session& s)