X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/6d6ee3cd05fd81d6eca0b61ad21bf2912dad51b2..d36e9a7ac2032c4fddcb236753b99c6573129893:/src/mc/mc_state.cpp diff --git a/src/mc/mc_state.cpp b/src/mc/mc_state.cpp index 50df4a3f11..88254304b8 100644 --- a/src/mc/mc_state.cpp +++ b/src/mc/mc_state.cpp @@ -1,224 +1,202 @@ -/* Copyright (c) 2008-2015. The SimGrid Team. - * All rights reserved. */ +/* Copyright (c) 2008-2019. The SimGrid Team. All rights reserved. */ /* This program is free software; you can redistribute it and/or modify it * under the terms of the license (GNU LGPL) which comes with this package. */ -#include +#include "src/mc/mc_state.hpp" +#include "src/mc/mc_comm_pattern.hpp" +#include "src/mc/mc_config.hpp" +#include "src/mc/mc_request.hpp" +#include "src/mc/mc_smx.hpp" -#include - -#include -#include - -#include "src/simix/smx_private.h" -#include "src/mc/mc_state.h" -#include "src/mc/mc_request.h" -#include "src/mc/mc_private.h" -#include "src/mc/mc_comm_pattern.h" -#include "src/mc/mc_smx.h" -#include "src/mc/mc_xbt.hpp" -#include "src/mc/Transition.hpp" +#include using simgrid::mc::remote; -XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_state, mc, - "Logging specific to MC (state)"); - -/** - * \brief Creates a state data structure used by the exploration algorithm - */ -simgrid::mc::State* MC_state_new(unsigned long state_number) -{ - simgrid::mc::State* state = new simgrid::mc::State(); - state->processStates.resize(MC_smx_get_maxpid()); - state->num = state_number; - /* Stateful model checking */ - if((_sg_mc_checkpoint > 0 && (state_number % _sg_mc_checkpoint == 0)) || _sg_mc_termination){ - state->system_state = simgrid::mc::take_snapshot(state->num); - if(_sg_mc_comms_determinism || _sg_mc_send_determinism){ - MC_state_copy_incomplete_communications_pattern(state); - MC_state_copy_index_communications_pattern(state); - } - } - return state; -} +XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_state, mc, "Logging specific to MC (state)"); namespace simgrid { namespace mc { -State::State() +State::State(unsigned long state_number) : num_(state_number) { - std::memset(&this->internal_comm, 0, sizeof(this->internal_comm)); + this->internal_comm.clear(); std::memset(&this->internal_req, 0, sizeof(this->internal_req)); - std::memset(&this->executed_req, 0, sizeof(this->executed_req)); + std::memset(&this->executed_req_, 0, sizeof(this->executed_req_)); + + actor_states_.resize(MC_smx_get_maxpid()); + /* Stateful model checking */ + if ((_sg_mc_checkpoint > 0 && (state_number % _sg_mc_checkpoint == 0)) || _sg_mc_termination) { + system_state = std::make_shared(num_); + if (_sg_mc_comms_determinism || _sg_mc_send_determinism) { + MC_state_copy_incomplete_communications_pattern(this); + MC_state_copy_index_communications_pattern(this); + } + } } -std::size_t State::interleaveSize() const +std::size_t State::interleave_size() const { - return std::count_if(this->processStates.begin(), this->processStates.end(), - [](simgrid::mc::ProcessState const& state) { return state.isToInterleave(); }); + return boost::range::count_if(this->actor_states_, [](simgrid::mc::ActorState const& a) { return a.is_todo(); }); } -Transition State::getTransition() const +Transition State::get_transition() const { - return this->transition; + return this->transition_; } } } -static inline smx_simcall_t MC_state_get_request_for_process( - simgrid::mc::State* state, smx_process_t process) +/* Search an enabled transition for the given process. + * + * This can be seen as an iterator returning the next transition of the process. + * + * We only consider the processes that are both + * - marked "to be interleaved" in their ActorState (controlled by the checker algorithm). + * - which simcall can currently be executed (like a comm where the other partner is already known) + * Once we returned the last enabled transition of a process, it is marked done. + * + * Things can get muddled with the WAITANY and TESTANY simcalls, that are rewritten on the fly to a bunch of WAIT + * (resp TEST) transitions using the transition.argument field to remember what was the last returned sub-transition. + */ +static inline smx_simcall_t MC_state_get_request_for_process(simgrid::mc::State* state, smx_actor_t actor) { - simgrid::mc::ProcessState* procstate = &state->processStates[process->pid]; - state->transition.pid = -1; - state->transition.argument = -1; + /* reset the outgoing transition */ + simgrid::mc::ActorState* procstate = &state->actor_states_[actor->get_pid()]; + state->transition_.pid_ = -1; + state->transition_.argument_ = -1; + state->executed_req_.call_ = SIMCALL_NONE; - if (!procstate->isToInterleave()) - return nullptr; - if (!simgrid::mc::process_is_enabled(process)) - return nullptr; + if (not simgrid::mc::actor_is_enabled(actor)) + return nullptr; // Not executable in the application smx_simcall_t req = nullptr; - switch (process->simcall.call) { - case SIMCALL_COMM_WAITANY: - state->transition.argument = -1; - while (procstate->interleave_count < - read_length(mc_model_checker->process(), - remote(simcall_comm_waitany__get__comms(&process->simcall)))) { - if (simgrid::mc::request_is_enabled_by_idx(&process->simcall, - procstate->interleave_count++)) { - state->transition.argument = procstate->interleave_count - 1; - break; - } + switch (actor->simcall.call_) { + case SIMCALL_COMM_WAITANY: + state->transition_.argument_ = -1; + while (procstate->times_considered < simcall_comm_waitany__get__count(&actor->simcall)) { + if (simgrid::mc::request_is_enabled_by_idx(&actor->simcall, procstate->times_considered)) { + state->transition_.argument_ = procstate->times_considered; + ++procstate->times_considered; + break; } - - if (procstate->interleave_count >= - simgrid::mc::read_length(mc_model_checker->process(), - simgrid::mc::remote(simcall_comm_waitany__get__comms(&process->simcall)))) - procstate->setDone(); - if (state->transition.argument != -1) - req = &process->simcall; - break; - - case SIMCALL_COMM_TESTANY: { - unsigned start_count = procstate->interleave_count; - state->transition.argument = -1; - while (procstate->interleave_count < - read_length(mc_model_checker->process(), - remote(simcall_comm_testany__get__comms(&process->simcall)))) - if (simgrid::mc::request_is_enabled_by_idx(&process->simcall, - procstate->interleave_count++)) { - state->transition.argument = procstate->interleave_count - 1; - break; - } - - if (procstate->interleave_count >= - read_length(mc_model_checker->process(), - remote(simcall_comm_testany__get__comms(&process->simcall)))) - procstate->setDone(); - - if (state->transition.argument != -1 || start_count == 0) - req = &process->simcall; - - break; + ++procstate->times_considered; } - case SIMCALL_COMM_WAIT: { - smx_synchro_t remote_act = simcall_comm_wait__get__comm(&process->simcall); - s_smx_synchro_t act; - mc_model_checker->process().read_bytes( - &act, sizeof(act), remote(remote_act)); - if (act.comm.src_proc && act.comm.dst_proc) - state->transition.argument = 0; - else if (act.comm.src_proc == nullptr && act.comm.type == SIMIX_COMM_READY - && act.comm.detached == 1) - state->transition.argument = 0; - else - state->transition.argument = -1; - procstate->setDone(); - req = &process->simcall; - break; + if (procstate->times_considered >= simcall_comm_waitany__get__count(&actor->simcall)) + procstate->set_done(); + if (state->transition_.argument_ != -1) + req = &actor->simcall; + break; + + case SIMCALL_COMM_TESTANY: { + unsigned start_count = procstate->times_considered; + state->transition_.argument_ = -1; + while (procstate->times_considered < simcall_comm_testany__get__count(&actor->simcall)) { + if (simgrid::mc::request_is_enabled_by_idx(&actor->simcall, procstate->times_considered)) { + state->transition_.argument_ = procstate->times_considered; + ++procstate->times_considered; + break; + } + ++procstate->times_considered; } - case SIMCALL_MC_RANDOM: { - int min_value = simcall_mc_random__get__min(&process->simcall); - state->transition.argument = procstate->interleave_count + min_value; - procstate->interleave_count++; - if (state->transition.argument == simcall_mc_random__get__max(&process->simcall)) - procstate->setDone(); - req = &process->simcall; - break; - } + if (procstate->times_considered >= simcall_comm_testany__get__count(&actor->simcall)) + procstate->set_done(); + + if (state->transition_.argument_ != -1 || start_count == 0) + req = &actor->simcall; + + break; + } + + case SIMCALL_COMM_WAIT: { + simgrid::mc::RemotePtr remote_act = + remote(static_cast(simcall_comm_wait__getraw__comm(&actor->simcall))); + simgrid::mc::Remote temp_act; + mc_model_checker->process().read(temp_act, remote_act); + simgrid::kernel::activity::CommImpl* act = temp_act.get_buffer(); + if (act->src_actor_.get() && act->dst_actor_.get()) + state->transition_.argument_ = 0; // OK + else if (act->src_actor_.get() == nullptr && act->type_ == simgrid::kernel::activity::CommImpl::Type::READY && + act->detached()) + state->transition_.argument_ = 0; // OK + else + state->transition_.argument_ = -1; // timeout + procstate->set_done(); + req = &actor->simcall; + break; + } - default: - procstate->setDone(); - state->transition.argument = 0; - req = &process->simcall; - break; + case SIMCALL_MC_RANDOM: { + int min_value = simcall_mc_random__get__min(&actor->simcall); + state->transition_.argument_ = procstate->times_considered + min_value; + procstate->times_considered++; + if (state->transition_.argument_ == simcall_mc_random__get__max(&actor->simcall)) + procstate->set_done(); + req = &actor->simcall; + break; + } + + default: + procstate->set_done(); + state->transition_.argument_ = 0; + req = &actor->simcall; + break; } - if (!req) + if (not req) return nullptr; + state->transition_.pid_ = actor->get_pid(); + state->executed_req_ = *req; // Fetch the data of the request and translate it: + state->internal_req = *req; + + /* The waitany and testany request are transformed into a wait or test request over the corresponding communication + * action so it can be treated later by the dependence function. */ + switch (req->call_) { + case SIMCALL_COMM_WAITANY: { + state->internal_req.call_ = SIMCALL_COMM_WAIT; + simgrid::kernel::activity::CommImpl* remote_comm; + remote_comm = mc_model_checker->process().read( + remote(simcall_comm_waitany__get__comms(req) + state->transition_.argument_)); + mc_model_checker->process().read(state->internal_comm, remote(remote_comm)); + simcall_comm_wait__set__comm(&state->internal_req, state->internal_comm.get_buffer()); + simcall_comm_wait__set__timeout(&state->internal_req, 0); + break; + } - state->transition.pid = process->pid; - - state->executed_req = *req; - - /* The waitany and testany request are transformed into a wait or test request - * over the corresponding communication action so it can be treated later by - * the dependence function. */ - switch (req->call) { - case SIMCALL_COMM_WAITANY: { - state->internal_req.call = SIMCALL_COMM_WAIT; - state->internal_req.issuer = req->issuer; - smx_synchro_t remote_comm; - read_element(mc_model_checker->process(), - &remote_comm, remote(simcall_comm_waitany__get__comms(req)), - state->transition.argument, sizeof(remote_comm)); - mc_model_checker->process().read(&state->internal_comm, remote(remote_comm)); - simcall_comm_wait__set__comm(&state->internal_req, &state->internal_comm); - simcall_comm_wait__set__timeout(&state->internal_req, 0); - break; - } - - case SIMCALL_COMM_TESTANY: - state->internal_req.call = SIMCALL_COMM_TEST; - state->internal_req.issuer = req->issuer; + case SIMCALL_COMM_TESTANY: + state->internal_req.call_ = SIMCALL_COMM_TEST; - if (state->transition.argument > 0) { - smx_synchro_t remote_comm; - read_element(mc_model_checker->process(), - &remote_comm, remote(simcall_comm_testany__get__comms(req)), - state->transition.argument, sizeof(remote_comm)); - mc_model_checker->process().read(&state->internal_comm, remote(remote_comm)); - } + if (state->transition_.argument_ > 0) { + simgrid::kernel::activity::CommImpl* remote_comm = mc_model_checker->process().read( + remote(simcall_comm_testany__get__comms(req) + state->transition_.argument_)); + mc_model_checker->process().read(state->internal_comm, remote(remote_comm)); + } - simcall_comm_test__set__comm(&state->internal_req, &state->internal_comm); - simcall_comm_test__set__result(&state->internal_req, state->transition.argument); - break; - - case SIMCALL_COMM_WAIT: - state->internal_req = *req; - mc_model_checker->process().read_bytes(&state->internal_comm , - sizeof(state->internal_comm), remote(simcall_comm_wait__get__comm(req))); - simcall_comm_wait__set__comm(&state->executed_req, &state->internal_comm); - simcall_comm_wait__set__comm(&state->internal_req, &state->internal_comm); - break; - - case SIMCALL_COMM_TEST: - state->internal_req = *req; - mc_model_checker->process().read_bytes(&state->internal_comm, - sizeof(state->internal_comm), remote(simcall_comm_test__get__comm(req))); - simcall_comm_test__set__comm(&state->executed_req, &state->internal_comm); - simcall_comm_test__set__comm(&state->internal_req, &state->internal_comm); - break; - - default: - state->internal_req = *req; - break; + simcall_comm_test__set__comm(&state->internal_req, state->internal_comm.get_buffer()); + simcall_comm_test__set__result(&state->internal_req, state->transition_.argument_); + break; + + case SIMCALL_COMM_WAIT: + mc_model_checker->process().read_bytes(&state->internal_comm, sizeof(state->internal_comm), + remote(simcall_comm_wait__getraw__comm(req))); + simcall_comm_wait__set__comm(&state->executed_req_, state->internal_comm.get_buffer()); + simcall_comm_wait__set__comm(&state->internal_req, state->internal_comm.get_buffer()); + break; + + case SIMCALL_COMM_TEST: + mc_model_checker->process().read_bytes(&state->internal_comm, sizeof(state->internal_comm), + remote(simcall_comm_test__getraw__comm(req))); + simcall_comm_test__set__comm(&state->executed_req_, state->internal_comm.get_buffer()); + simcall_comm_test__set__comm(&state->internal_req, state->internal_comm.get_buffer()); + break; + + default: + /* No translation needed */ + break; } return req; @@ -226,8 +204,12 @@ static inline smx_simcall_t MC_state_get_request_for_process( smx_simcall_t MC_state_get_request(simgrid::mc::State* state) { - for (auto& p : mc_model_checker->process().simix_processes()) { - smx_simcall_t res = MC_state_get_request_for_process(state, &p.copy); + for (auto& actor : mc_model_checker->process().actors()) { + /* Only consider the actors that were marked as interleaving by the checker algorithm */ + if (not state->actor_states_[actor.copy.get_buffer()->get_pid()].is_todo()) + continue; + + smx_simcall_t res = MC_state_get_request_for_process(state, actor.copy.get_buffer()); if (res) return res; }