-/* Copyright (c) 2007-2017. The SimGrid Team. All rights reserved. */
+/* Copyright (c) 2007-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. */
#ifndef SIMGRID_MC_STATE_HPP
#define SIMGRID_MC_STATE_HPP
-#include <list>
-#include <memory>
-
+#include "src/kernel/activity/CommImpl.hpp"
#include "src/mc/Transition.hpp"
-#include "src/mc/mc_record.hpp"
-#include "src/mc/mc_snapshot.hpp"
+#include "src/mc/sosp/Snapshot.hpp"
namespace simgrid {
namespace mc {
enum class PatternCommunicationType {
- none = 0,
- send = 1,
+ none = 0,
+ send = 1,
receive = 2,
};
int num = 0;
simgrid::kernel::activity::CommImpl* comm_addr;
PatternCommunicationType type = PatternCommunicationType::send;
- unsigned long src_proc = 0;
- unsigned long dst_proc = 0;
- const char *src_host = nullptr;
- const char *dst_host = nullptr;
+ unsigned long src_proc = 0;
+ unsigned long dst_proc = 0;
+ const char* src_host = nullptr;
+ const char* dst_host = nullptr;
std::string rdv;
std::vector<char> data;
- int tag = 0;
+ int tag = 0;
int index = 0;
- PatternCommunication()
- {
- std::memset(&comm_addr, 0, sizeof(comm_addr));
- }
+ PatternCommunication() { std::memset(&comm_addr, 0, sizeof(comm_addr)); }
PatternCommunication dup() const
{
simgrid::mc::PatternCommunication res;
// num?
res.comm_addr = this->comm_addr;
- res.type = this->type;
+ res.type = this->type;
// src_proc?
// dst_proc?
res.dst_proc = this->dst_proc;
res.dst_host = this->dst_host;
- res.rdv = this->rdv;
- res.data = this->data;
+ res.rdv = this->rdv;
+ res.data = this->data;
// tag?
res.index = this->index;
return res;
}
-
};
-/* On every state, each process has an entry of the following type.
- * This represents both the process and its transition because
- * a process cannot have more than one enabled transition at a given time.
+/* On every state, each actor has an entry of the following type.
+ * This represents both the actor and its transition because
+ * an actor cannot have more than one enabled transition at a given time.
*/
-class ProcessState {
- /* Possible exploration status of a process transition in a state.
- * Either the checker did not consider the transition, or it was considered and to do, or considered and done.
+class ActorState {
+ /* Possible exploration status of a actor transition in a state.
+ * Either the checker did not consider the transition, or it was considered and still to do, or considered and done.
*/
enum class InterleavingType {
- /** This process transition is not considered by the checker (yet?) */
+ /** This actor transition is not considered by the checker (yet?) */
disabled = 0,
- /** The checker algorithm decided that this process transitions should be done at some point */
+ /** The checker algorithm decided that this actor transitions should be done at some point */
todo,
/** The checker algorithm decided that this should be done, but it was done in the meanwhile */
done,
/** Exploration control information */
InterleavingType state = InterleavingType::disabled;
+
public:
/** Number of times that the process was considered to be executed */
// TODO, make this private
unsigned int times_considered = 0;
- bool isDisabled() const {
- return this->state == InterleavingType::disabled;
- }
- bool isDone() const {
- return this->state == InterleavingType::done;
- }
- bool isTodo() const {
- return this->state == InterleavingType::todo;
- }
+ bool is_disabled() const { return this->state == InterleavingType::disabled; }
+ bool is_done() const { return this->state == InterleavingType::done; }
+ bool is_todo() const { return this->state == InterleavingType::todo; }
/** Mark that we should try executing this process at some point in the future of the checker algorithm */
- void consider() {
+ void consider()
+ {
this->state = InterleavingType::todo;
this->times_considered = 0;
}
- void setDone() {
- this->state = InterleavingType::done;
- }
+ void set_done() { this->state = InterleavingType::done; }
};
-/* A node in the exploration graph (kind-of)
- */
+/* A node in the exploration graph (kind-of) */
class XBT_PRIVATE State {
public:
/** Sequential state number (used for debugging) */
- int num = 0;
+ int num_ = 0;
/** State's exploration status by process */
- std::vector<ProcessState> actorStates;
+ std::vector<ActorState> actor_states_;
- Transition transition;
+ Transition transition_;
/** The simcall which was executed, going out of that state */
- s_smx_simcall_t executed_req;
+ s_smx_simcall executed_req_;
/* Internal translation of the executed_req simcall
*
* SIMCALL_COMM_TESTANY is translated to a SIMCALL_COMM_TEST
* and SIMCALL_COMM_WAITANY to a SIMCALL_COMM_WAIT.
*/
- s_smx_simcall_t internal_req;
+ s_smx_simcall internal_req;
/* Can be used as a copy of the remote synchro object */
simgrid::mc::Remote<simgrid::kernel::activity::CommImpl> internal_comm;
std::shared_ptr<simgrid::mc::Snapshot> system_state;
// For CommunicationDeterminismChecker
- std::vector<std::vector<simgrid::mc::PatternCommunication>> incomplete_comm_pattern;
- std::vector<unsigned> communicationIndices;
+ std::vector<std::vector<simgrid::mc::PatternCommunication>> incomplete_comm_pattern_;
+ std::vector<unsigned> communication_indices_;
- State(unsigned long state_number);
+ explicit State(unsigned long state_number);
- std::size_t interleaveSize() const;
- void addInterleavingSet(smx_actor_t actor) { this->actorStates[actor->pid].consider(); }
- Transition getTransition() const;
+ std::size_t interleave_size() const;
+ void add_interleaving_set(smx_actor_t actor) { this->actor_states_[actor->get_pid()].consider(); }
+ Transition get_transition() const;
};
-
}
}
-XBT_PRIVATE smx_simcall_t MC_state_get_request(simgrid::mc::State* state);
+XBT_PRIVATE smx_simcall_t MC_state_choose_request(simgrid::mc::State* state);
#endif