X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/002c591623abdaa8c94a3b434a2f179c35d29cb9..2558c8c0eb206ff353cb88672f5a90cd0e2562d3:/src/mc/mc_base.cpp diff --git a/src/mc/mc_base.cpp b/src/mc/mc_base.cpp index 162fd84426..d44192625b 100644 --- a/src/mc/mc_base.cpp +++ b/src/mc/mc_base.cpp @@ -13,6 +13,8 @@ #include "src/mc/mc_replay.hpp" #include "src/simix/smx_private.hpp" +#include + #if SIMGRID_HAVE_MC #include "src/mc/ModelChecker.hpp" #include "src/mc/remote/RemoteClient.hpp" @@ -72,7 +74,7 @@ bool actor_is_enabled(smx_actor_t actor) #if SIMGRID_HAVE_MC // If in the MCer, ask the client app since it has all the data if (mc_model_checker != nullptr) { - return mc_model_checker->process().actor_is_enabled(actor->pid_); + return mc_model_checker->process().actor_is_enabled(actor->get_pid()); } #endif @@ -96,16 +98,17 @@ bool actor_is_enabled(smx_actor_t actor) } /* On the other hand if it hasn't a timeout, check if the comm is ready.*/ else if (act->detached && act->src_actor_ == nullptr && - act->type == simgrid::kernel::activity::CommImpl::Type::READY) + act->type_ == simgrid::kernel::activity::CommImpl::Type::READY) return (act->dst_actor_ != nullptr); return (act->src_actor_ && act->dst_actor_); } case SIMCALL_COMM_WAITANY: { - xbt_dynar_t comms = simcall_comm_waitany__get__comms(req); - for (unsigned int index = 0; index < comms->used; ++index) { - simgrid::kernel::activity::CommImpl* act = xbt_dynar_get_as(comms, index, simgrid::kernel::activity::CommImpl*); - if (act->src_actor_ && act->dst_actor_) + simgrid::kernel::activity::CommImpl** comms = simcall_comm_waitany__get__comms(req); + size_t count = simcall_comm_waitany__get__count(req); + for (unsigned int index = 0; index < count; ++index) { + auto* comm = comms[index]; + if (comm->src_actor_ && comm->dst_actor_) return true; } return false; @@ -114,9 +117,9 @@ bool actor_is_enabled(smx_actor_t actor) case SIMCALL_MUTEX_LOCK: { smx_mutex_t mutex = simcall_mutex_lock__get__mutex(req); - if (mutex->owner == nullptr) + if (mutex->owner_ == nullptr) return true; - return mutex->owner->pid_ == req->issuer->pid_; + return mutex->owner_->get_pid() == req->issuer->get_pid(); } case SIMCALL_SEM_ACQUIRE: { @@ -159,28 +162,12 @@ bool request_is_visible(smx_simcall_t req) } } -static int prng_random(int min, int max) -{ - unsigned long output_size = ((unsigned long) max - (unsigned long) min) + 1; - unsigned long input_size = (unsigned long) RAND_MAX + 1; - unsigned long reject_size = input_size % output_size; - unsigned long accept_size = input_size - reject_size; // module*accept_size - - // Use rejection in order to avoid skew - unsigned long x; - do { -#ifndef _WIN32 - x = (unsigned long) random(); -#else - x = (unsigned long) rand(); -#endif - } while( x >= accept_size ); - return min + (x % output_size); -} - int simcall_HANDLER_mc_random(smx_simcall_t simcall, int min, int max) { - if (not MC_is_active() && MC_record_path.empty()) - return prng_random(min, max); + if (not MC_is_active() && MC_record_path.empty()) { + static std::default_random_engine rnd_engine; + std::uniform_int_distribution prng(min, max); + return prng(rnd_engine); + } return simcall->mc_value; }