-/* Copyright (c) 2008-2014. The SimGrid Team.
- * All rights reserved. */
+/* Copyright (c) 2008-2017. 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 <assert.h>
-#include "../simix/smx_private.h"
-#include "xbt/fifo.h"
-#include "mc_state.h"
-#include "mc_request.h"
-#include "mc_private.h"
-#include "mc_comm_pattern.h"
-#include "mc_smx.h"
+#include <boost/range/algorithm.hpp>
+
+#include <xbt/log.h>
+#include <xbt/sysdep.h>
+
+#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"
using simgrid::mc::remote;
-extern "C" {
+XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_state, mc, "Logging specific to MC (state)");
-XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_state, mc,
- "Logging specific to MC (state)");
+namespace simgrid {
+namespace mc {
-/**
- * \brief Creates a state data structure used by the exploration algorithm
- */
-mc_state_t MC_state_new()
+State::State(unsigned long state_number)
{
- mc_state_t state = xbt_new0(s_mc_state_t, 1);
-
- state->max_pid = MC_smx_get_maxpid();
- state->proc_status = xbt_new0(s_mc_procstate_t, state->max_pid);
- state->system_state = NULL;
- state->num = ++mc_stats->expanded_states;
- state->in_visited_states = 0;
- state->incomplete_comm_pattern = NULL;
+ this->internal_comm.clear();
+ std::memset(&this->internal_req, 0, sizeof(this->internal_req));
+ std::memset(&this->executed_req, 0, sizeof(this->executed_req));
+
+ actorStates.resize(MC_smx_get_maxpid());
+ num = state_number;
/* Stateful model checking */
- if((_sg_mc_checkpoint > 0 && (mc_stats->expanded_states % _sg_mc_checkpoint == 0)) || _sg_mc_termination){
- state->system_state = 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);
+ if ((_sg_mc_checkpoint > 0 && (state_number % _sg_mc_checkpoint == 0)) || _sg_mc_termination) {
+ system_state = simgrid::mc::take_snapshot(num);
+ if (_sg_mc_comms_determinism || _sg_mc_send_determinism) {
+ MC_state_copy_incomplete_communications_pattern(this);
+ MC_state_copy_index_communications_pattern(this);
}
}
- return state;
}
-/**
- * \brief Deletes a state data structure
- * \param trans The state to be deleted
- */
-void MC_state_delete(mc_state_t state, int free_snapshot){
- if (state->system_state && free_snapshot){
- delete state->system_state;
- }
- if(_sg_mc_comms_determinism || _sg_mc_send_determinism){
- xbt_free(state->index_comm);
- xbt_free(state->incomplete_comm_pattern);
- }
- xbt_free(state->proc_status);
- xbt_free(state);
+std::size_t State::interleaveSize() const
+{
+ return boost::range::count_if(this->actorStates,
+ [](simgrid::mc::ProcessState const& p) { return p.isTodo(); });
}
-void MC_state_interleave_process(mc_state_t state, smx_process_t process)
+Transition State::getTransition() const
{
- assert(state);
- state->proc_status[process->pid].state = MC_INTERLEAVE;
- state->proc_status[process->pid].interleave_count = 0;
+ return this->transition;
}
-void MC_state_remove_interleave_process(mc_state_t state, smx_process_t process)
-{
- if (state->proc_status[process->pid].state == MC_INTERLEAVE)
- state->proc_status[process->pid].state = MC_DONE;
+}
}
-unsigned int MC_state_interleave_size(mc_state_t state)
+/* 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 ProcessState (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)
{
- unsigned int i, size = 0;
+ /* reset the outgoing transition */
+ simgrid::mc::ProcessState* procstate = &state->actorStates[actor->pid];
+ state->transition.pid = -1;
+ state->transition.argument = -1;
+ state->executed_req.call = SIMCALL_NONE;
+
+ if (!procstate->isTodo())
+ return nullptr; // Not considered by the checker algorithm
+ if (!simgrid::mc::actor_is_enabled(actor))
+ return nullptr; // Not executable in the application
+
+ smx_simcall_t req = nullptr;
+ switch (actor->simcall.call) {
+ case SIMCALL_COMM_WAITANY:
+ state->transition.argument = -1;
+ while (procstate->times_considered <
+ read_length(mc_model_checker->process(),
+ remote(simcall_comm_waitany__get__comms(&actor->simcall)))) {
+ if (simgrid::mc::request_is_enabled_by_idx(&actor->simcall,
+ procstate->times_considered++)) {
+ state->transition.argument = procstate->times_considered - 1;
+ break;
+ }
+ }
- for (i = 0; i < state->max_pid; i++) {
- if ((state->proc_status[i].state == MC_INTERLEAVE)
- || (state->proc_status[i].state == MC_MORE_INTERLEAVE))
- size++;
- }
+ if (procstate->times_considered >=
+ simgrid::mc::read_length(mc_model_checker->process(),
+ simgrid::mc::remote(simcall_comm_waitany__get__comms(&actor->simcall))))
+ procstate->setDone();
+ if (state->transition.argument != -1)
+ req = &actor->simcall;
+ break;
- return size;
-}
+ 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 - 1;
+ break;
+ }
-int MC_state_process_is_done(mc_state_t state, smx_process_t process)
-{
- return state->proc_status[process->pid].state == MC_DONE ? TRUE : FALSE;
-}
+ if (procstate->times_considered >=
+ simcall_comm_testany__get__count(&actor->simcall))
+ procstate->setDone();
-void MC_state_set_executed_request(mc_state_t state, smx_simcall_t req,
- int value)
-{
- state->executed_req = *req;
- state->req_num = value;
+ if (state->transition.argument != -1 || start_count == 0)
+ req = &actor->simcall;
+
+ break;
+ }
- smx_process_t process = NULL;
+ case SIMCALL_COMM_WAIT: {
+ simgrid::mc::RemotePtr<simgrid::kernel::activity::Comm> remote_act = remote(
+ static_cast<simgrid::kernel::activity::Comm*>(simcall_comm_wait__get__comm(&actor->simcall)));
+ simgrid::mc::Remote<simgrid::kernel::activity::Comm> temp_act;
+ mc_model_checker->process().read(temp_act, remote_act);
+ simgrid::kernel::activity::Comm* act = temp_act.getBuffer();
+ if (act->src_proc && act->dst_proc)
+ state->transition.argument = 0;
+ else if (act->src_proc == nullptr && act->type == SIMIX_COMM_READY
+ && act->detached == 1)
+ state->transition.argument = 0;
+ else
+ state->transition.argument = -1;
+ procstate->setDone();
+ req = &actor->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->setDone();
+ req = &actor->simcall;
+ break;
+ }
- /* 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. */
+ default:
+ procstate->setDone();
+ state->transition.argument = 0;
+ req = &actor->simcall;
+ break;
+ }
+ if (!req)
+ return nullptr;
+
+ state->transition.pid = actor->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:
+ case SIMCALL_COMM_WAITANY: {
state->internal_req.call = SIMCALL_COMM_WAIT;
- state->internal_req.issuer = req->issuer;
- MC_process_read_dynar_element(&mc_model_checker->process(),
- &state->internal_comm, simcall_comm_waitany__get__comms(req),
- value, sizeof(state->internal_comm));
- simcall_comm_wait__set__comm(&state->internal_req, &state->internal_comm);
+ smx_activity_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(
+ static_cast<simgrid::kernel::activity::Comm*>(remote_comm)));
+ simcall_comm_wait__set__comm(&state->internal_req, state->internal_comm.getBuffer());
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;
- if (value > 0)
- MC_process_read_dynar_element(&mc_model_checker->process(),
- &state->internal_comm, simcall_comm_testany__get__comms(req),
- value, sizeof(state->internal_comm));
+ if (state->transition.argument > 0) {
+ smx_activity_t 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(
+ static_cast<simgrid::kernel::activity::Comm*>(remote_comm)));
+ }
- simcall_comm_test__set__comm(&state->internal_req, &state->internal_comm);
- simcall_comm_test__set__result(&state->internal_req, value);
+ simcall_comm_test__set__comm(&state->internal_req, state->internal_comm.getBuffer());
+ 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);
+ simcall_comm_wait__set__comm(&state->executed_req, state->internal_comm.getBuffer());
+ simcall_comm_wait__set__comm(&state->internal_req, state->internal_comm.getBuffer());
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);
+ simcall_comm_test__set__comm(&state->executed_req, state->internal_comm.getBuffer());
+ simcall_comm_test__set__comm(&state->internal_req, state->internal_comm.getBuffer());
break;
- case SIMCALL_MC_RANDOM: {
- state->internal_req = *req;
- int random_max = simcall_mc_random__get__max(req);
- if (value != random_max) {
- MC_EACH_SIMIX_PROCESS(process,
- mc_procstate_t procstate = &state->proc_status[process->pid];
- const smx_process_t issuer = MC_smx_simcall_get_issuer(req);
- if (process->pid == issuer->pid) {
- procstate->state = MC_MORE_INTERLEAVE;
- break;
- }
- );
- }
- break;
- }
-
default:
- state->internal_req = *req;
+ /* No translation needed */
break;
}
-}
-
-smx_simcall_t MC_state_get_executed_request(mc_state_t state, int *value)
-{
- *value = state->req_num;
- return &state->executed_req;
-}
-smx_simcall_t MC_state_get_internal_request(mc_state_t state)
-{
- return &state->internal_req;
+ return req;
}
-static inline smx_simcall_t MC_state_get_request_for_process(
- mc_state_t state, int*value, smx_process_t process)
+smx_simcall_t MC_state_get_request(simgrid::mc::State* state)
{
- mc_procstate_t procstate = &state->proc_status[process->pid];
-
- if (procstate->state != MC_INTERLEAVE
- && procstate->state != MC_MORE_INTERLEAVE)
- return NULL;
- if (!MC_process_is_enabled(process))
- return NULL;
-
- switch (process->simcall.call) {
-
- case SIMCALL_COMM_WAITANY:
- *value = -1;
- while (procstate->interleave_count <
- MC_process_read_dynar_length(&mc_model_checker->process(),
- simcall_comm_waitany__get__comms(&process->simcall))) {
- if (MC_request_is_enabled_by_idx
- (&process->simcall, procstate->interleave_count++)) {
- *value = procstate->interleave_count - 1;
- break;
- }
- }
-
- if (procstate->interleave_count >=
- MC_process_read_dynar_length(&mc_model_checker->process(),
- simcall_comm_waitany__get__comms(&process->simcall)))
- procstate->state = MC_DONE;
-
- if (*value != -1)
- return &process->simcall;
-
- break;
-
- case SIMCALL_COMM_TESTANY: {
- unsigned start_count = procstate->interleave_count;
- *value = -1;
- while (procstate->interleave_count <
- MC_process_read_dynar_length(&mc_model_checker->process(),
- simcall_comm_testany__get__comms(&process->simcall))) {
- if (MC_request_is_enabled_by_idx
- (&process->simcall, procstate->interleave_count++)) {
- *value = procstate->interleave_count - 1;
- break;
- }
- }
-
- if (procstate->interleave_count >=
- MC_process_read_dynar_length(&mc_model_checker->process(),
- simcall_comm_testany__get__comms(&process->simcall)))
- procstate->state = MC_DONE;
-
- if (*value != -1 || start_count == 0)
- return &process->simcall;
-
- break;
- }
-
- 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) {
- *value = 0;
- } else {
- if (act.comm.src_proc == NULL && act.comm.type == SIMIX_COMM_READY
- && act.comm.detached == 1)
- *value = 0;
- else
- *value = -1;
- }
- procstate->state = MC_DONE;
- return &process->simcall;
- }
-
- case SIMCALL_MC_RANDOM:
- if (procstate->state == MC_INTERLEAVE)
- *value = simcall_mc_random__get__min(&process->simcall);
- else {
- if (state->req_num < simcall_mc_random__get__max(&process->simcall))
- *value = state->req_num + 1;
- }
- procstate->state = MC_DONE;
- return &process->simcall;
-
- default:
- procstate->state = MC_DONE;
- *value = 0;
- return &process->simcall;
- }
- return NULL;
-}
-
-smx_simcall_t MC_state_get_request(mc_state_t state, int *value)
-{
- smx_process_t process = NULL;
- MC_EACH_SIMIX_PROCESS(process,
- smx_simcall_t res = MC_state_get_request_for_process(state, value, process);
+ for (auto& actor : mc_model_checker->process().actors()) {
+ smx_simcall_t res = MC_state_get_request_for_process(state, actor.copy.getBuffer());
if (res)
return res;
- );
-
- return NULL;
-}
-
+ }
+ return nullptr;
}