X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/b8dc7c0693d2606e9cb9f21bcced90a5174bd503..2dab73b61a6ad4feb7d2d267bf5621c7b95926b3:/src/mc/mc_request.cpp diff --git a/src/mc/mc_request.cpp b/src/mc/mc_request.cpp index 539143d711..565aeec692 100644 --- a/src/mc/mc_request.cpp +++ b/src/mc/mc_request.cpp @@ -1,4 +1,4 @@ -/* Copyright (c) 2008-2019. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2008-2020. 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. */ @@ -8,12 +8,12 @@ #include "src/kernel/activity/CommImpl.hpp" #include "src/kernel/activity/MutexImpl.hpp" #include "src/mc/ModelChecker.hpp" +#include "src/mc/checker/SimcallInspector.hpp" #include "src/mc/mc_smx.hpp" using simgrid::mc::remote; -XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_request, mc, - "Logging specific to MC (request)"); +XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_request, mc, "Logging specific to MC (request)"); static char *pointer_to_string(void *pointer); static char *buff_size_to_string(size_t size); @@ -56,14 +56,12 @@ bool request_depend_asymmetric(smx_simcall_t r1, smx_simcall_t r2) if (r1->call_ == SIMCALL_COMM_IRECV && r2->call_ == SIMCALL_COMM_ISEND) return false; - // Those are internal requests, we do not need indirection - // because those objects are copies: - simgrid::kernel::activity::CommImpl* synchro1 = MC_get_comm(r1); - simgrid::kernel::activity::CommImpl* synchro2 = MC_get_comm(r2); + // Those are internal requests, we do not need indirection because those objects are copies: + const kernel::activity::CommImpl* synchro1 = MC_get_comm(r1); + const kernel::activity::CommImpl* synchro2 = MC_get_comm(r2); if ((r1->call_ == SIMCALL_COMM_ISEND || r1->call_ == SIMCALL_COMM_IRECV) && r2->call_ == SIMCALL_COMM_WAIT) { - - smx_mailbox_t mbox = MC_get_mbox(r1); + const kernel::activity::MailboxImpl* mbox = MC_get_mbox(r1); if (mbox != synchro2->mbox_cpy && simcall_comm_wait__get__timeout(r2) <= 0) @@ -82,13 +80,12 @@ bool request_depend_asymmetric(smx_simcall_t r1, smx_simcall_t r2) return false; } - /* FIXME: the following rule assumes that the result of the - * isend/irecv call is not stored in a buffer used in the + /* FIXME: the following rule assumes that the result of the isend/irecv call is not stored in a buffer used in the * test call. */ #if 0 if((r1->call == SIMCALL_COMM_ISEND || r1->call == SIMCALL_COMM_IRECV) - && r2->call == SIMCALL_COMM_TEST) - return false; + && r2->call == SIMCALL_COMM_TEST) + return false; #endif if (r1->call_ == SIMCALL_COMM_WAIT && (r2->call_ == SIMCALL_COMM_WAIT || r2->call_ == SIMCALL_COMM_TEST) && @@ -126,10 +123,9 @@ bool request_depend(smx_simcall_t req1, smx_simcall_t req2) if (req1->call_ != req2->call_) return request_depend_asymmetric(req1, req2) && request_depend_asymmetric(req2, req1); - // Those are internal requests, we do not need indirection - // because those objects are copies: - simgrid::kernel::activity::CommImpl* synchro1 = MC_get_comm(req1); - simgrid::kernel::activity::CommImpl* synchro2 = MC_get_comm(req2); + // Those are internal requests, we do not need indirection because those objects are copies: + const kernel::activity::CommImpl* synchro1 = MC_get_comm(req1); + const kernel::activity::CommImpl* synchro2 = MC_get_comm(req2); switch (req1->call_) { case SIMCALL_COMM_ISEND: @@ -149,12 +145,11 @@ bool request_depend(smx_simcall_t req1, smx_simcall_t req2) } } -} -} +} // namespace mc +} // namespace simgrid static char *pointer_to_string(void *pointer) { - if (XBT_LOG_ISENABLED(mc_request, xbt_log_priority_verbose)) return bprintf("%p", pointer); @@ -163,7 +158,6 @@ static char *pointer_to_string(void *pointer) static char *buff_size_to_string(size_t buff_size) { - if (XBT_LOG_ISENABLED(mc_request, xbt_log_priority_verbose)) return bprintf("%zu", buff_size); @@ -175,8 +169,8 @@ std::string simgrid::mc::request_to_string(smx_simcall_t req, int value, simgrid { xbt_assert(mc_model_checker != nullptr, "Must be called from MCer"); - if (req->transition_ != nullptr) - return req->transition_->to_string(); + if (req->inspector_ != nullptr) + return req->inspector_->to_string(); bool use_remote_comm = true; switch(request_type) { @@ -197,7 +191,6 @@ std::string simgrid::mc::request_to_string(smx_simcall_t req, int value, simgrid smx_actor_t issuer = MC_smx_simcall_get_issuer(req); switch (req->call_) { - case SIMCALL_COMM_ISEND: { type = "iSend"; char* p = pointer_to_string(simcall_comm_isend__get__src_buff(req)); @@ -216,7 +209,7 @@ std::string simgrid::mc::request_to_string(smx_simcall_t req, int value, simgrid size_t* remote_size = simcall_comm_irecv__get__dst_buff_size(req); size_t size = 0; if (remote_size) - mc_model_checker->process().read_bytes(&size, sizeof(size), remote(remote_size)); + mc_model_checker->get_remote_simulation().read_bytes(&size, sizeof(size), remote(remote_size)); type = "iRecv"; char* p = pointer_to_string(simcall_comm_irecv__get__dst_buff(req)); @@ -244,16 +237,18 @@ std::string simgrid::mc::request_to_string(smx_simcall_t req, int value, simgrid p = pointer_to_string(remote_act); simgrid::mc::Remote temp_synchro; - simgrid::kernel::activity::CommImpl* act; + const simgrid::kernel::activity::CommImpl* act; if (use_remote_comm) { - mc_model_checker->process().read(temp_synchro, - remote(static_cast(remote_act))); + mc_model_checker->get_remote_simulation().read( + temp_synchro, remote(static_cast(remote_act))); act = temp_synchro.get_buffer(); } else act = remote_act; - smx_actor_t src_proc = mc_model_checker->process().resolve_actor(simgrid::mc::remote(act->src_actor_.get())); - smx_actor_t dst_proc = mc_model_checker->process().resolve_actor(simgrid::mc::remote(act->dst_actor_.get())); + smx_actor_t src_proc = + mc_model_checker->get_remote_simulation().resolve_actor(simgrid::mc::remote(act->src_actor_.get())); + smx_actor_t dst_proc = + mc_model_checker->get_remote_simulation().resolve_actor(simgrid::mc::remote(act->dst_actor_.get())); args = bprintf("comm=%s [(%ld)%s (%s)-> (%ld)%s (%s)]", p, src_proc ? src_proc->get_pid() : 0, src_proc ? MC_smx_actor_get_host_name(src_proc) : "", src_proc ? MC_smx_actor_get_name(src_proc) : "", dst_proc ? dst_proc->get_pid() : 0, @@ -268,10 +263,10 @@ std::string simgrid::mc::request_to_string(smx_simcall_t req, int value, simgrid simgrid::kernel::activity::CommImpl* remote_act = static_cast(simcall_comm_test__getraw__comm(req)); simgrid::mc::Remote temp_synchro; - simgrid::kernel::activity::CommImpl* act; + const simgrid::kernel::activity::CommImpl* act; if (use_remote_comm) { - mc_model_checker->process().read(temp_synchro, - remote(static_cast(remote_act))); + mc_model_checker->get_remote_simulation().read( + temp_synchro, remote(static_cast(remote_act))); act = temp_synchro.get_buffer(); } else act = remote_act; @@ -285,8 +280,10 @@ std::string simgrid::mc::request_to_string(smx_simcall_t req, int value, simgrid type = "Test TRUE"; p = pointer_to_string(remote_act); - smx_actor_t src_proc = mc_model_checker->process().resolve_actor(simgrid::mc::remote(act->src_actor_.get())); - smx_actor_t dst_proc = mc_model_checker->process().resolve_actor(simgrid::mc::remote(act->dst_actor_.get())); + smx_actor_t src_proc = + mc_model_checker->get_remote_simulation().resolve_actor(simgrid::mc::remote(act->src_actor_.get())); + smx_actor_t dst_proc = + mc_model_checker->get_remote_simulation().resolve_actor(simgrid::mc::remote(act->dst_actor_.get())); args = bprintf("comm=%s [(%ld)%s (%s) -> (%ld)%s (%s)]", p, src_proc->get_pid(), MC_smx_actor_get_name(src_proc), MC_smx_actor_get_host_name(src_proc), dst_proc->get_pid(), MC_smx_actor_get_name(dst_proc), MC_smx_actor_get_host_name(dst_proc)); @@ -300,7 +297,8 @@ std::string simgrid::mc::request_to_string(smx_simcall_t req, int value, simgrid size_t count = simcall_comm_waitany__get__count(req); if (count > 0) { simgrid::kernel::activity::CommImpl* remote_sync; - remote_sync = mc_model_checker->process().read(remote(simcall_comm_waitany__get__comms(req) + value)); + remote_sync = + mc_model_checker->get_remote_simulation().read(remote(simcall_comm_waitany__get__comms(req) + value)); char* p = pointer_to_string(remote_sync); args = bprintf("comm=%s (%d of %zu)", p, value + 1, count); xbt_free(p); @@ -327,14 +325,14 @@ std::string simgrid::mc::request_to_string(smx_simcall_t req, int value, simgrid type = "Mutex TRYLOCK"; simgrid::mc::Remote mutex; - mc_model_checker->process().read_bytes(mutex.get_buffer(), sizeof(mutex), - remote(req->call_ == SIMCALL_MUTEX_LOCK - ? simcall_mutex_lock__get__mutex(req) - : simcall_mutex_trylock__get__mutex(req))); + mc_model_checker->get_remote_simulation().read_bytes(mutex.get_buffer(), sizeof(mutex), + remote(req->call_ == SIMCALL_MUTEX_LOCK + ? simcall_mutex_lock__get__mutex(req) + : simcall_mutex_trylock__get__mutex(req))); args = bprintf("locked = %d, owner = %d, sleeping = n/a", mutex.get_buffer()->is_locked(), - mutex.get_buffer()->owner_ != nullptr - ? (int)mc_model_checker->process() - .resolve_actor(simgrid::mc::remote(mutex.get_buffer()->owner_)) + mutex.get_buffer()->get_owner() != nullptr + ? (int)mc_model_checker->get_remote_simulation() + .resolve_actor(simgrid::mc::remote(mutex.get_buffer()->get_owner())) ->get_pid() : -1); break; @@ -367,47 +365,34 @@ namespace mc { bool request_is_enabled_by_idx(smx_simcall_t req, unsigned int idx) { - simgrid::kernel::activity::CommImpl* remote_act = nullptr; + kernel::activity::CommImpl* remote_act = nullptr; switch (req->call_) { - case SIMCALL_COMM_WAIT: /* FIXME: check also that src and dst processes are not suspended */ remote_act = simcall_comm_wait__getraw__comm(req); break; case SIMCALL_COMM_WAITANY: - remote_act = mc_model_checker->process().read(remote(simcall_comm_testany__get__comms(req) + idx)); + remote_act = mc_model_checker->get_remote_simulation().read(remote(simcall_comm_testany__get__comms(req) + idx)); break; case SIMCALL_COMM_TESTANY: - remote_act = mc_model_checker->process().read(remote(simcall_comm_testany__get__comms(req) + idx)); + remote_act = mc_model_checker->get_remote_simulation().read(remote(simcall_comm_testany__get__comms(req) + idx)); break; default: return true; } - simgrid::mc::Remote temp_comm; - mc_model_checker->process().read(temp_comm, remote(remote_act)); - simgrid::kernel::activity::CommImpl* comm = temp_comm.get_buffer(); + Remote temp_comm; + mc_model_checker->get_remote_simulation().read(temp_comm, remote(remote_act)); + const kernel::activity::CommImpl* comm = temp_comm.get_buffer(); return comm->src_actor_.get() && comm->dst_actor_.get(); } - static const char* colors[] = { - "blue", - "red", - "green3", - "goldenrod", - "brown", - "purple", - "magenta", - "turquoise4", - "gray25", - "forestgreen", - "hotpink", - "lightblue", - "tan", + "blue", "red", "green3", "goldenrod", "brown", "purple", "magenta", + "turquoise4", "gray25", "forestgreen", "hotpink", "lightblue", "tan", }; static inline const char* get_color(int id) @@ -420,71 +405,70 @@ std::string request_get_dot_output(smx_simcall_t req, int value) const smx_actor_t issuer = MC_smx_simcall_get_issuer(req); const char* color = get_color(issuer->get_pid() - 1); - if (req->transition_ != nullptr) + if (req->inspector_ != nullptr) return simgrid::xbt::string_printf("label = \"%s\", color = %s, fontcolor = %s", - req->transition_->dot_label().c_str(), color, color); + req->inspector_->dot_label().c_str(), color, color); std::string label; switch (req->call_) { case SIMCALL_COMM_ISEND: if (issuer->get_host()) - label = simgrid::xbt::string_printf("[(%ld)%s] iSend", issuer->get_pid(), MC_smx_actor_get_host_name(issuer)); + label = xbt::string_printf("[(%ld)%s] iSend", issuer->get_pid(), MC_smx_actor_get_host_name(issuer)); else label = bprintf("[(%ld)] iSend", issuer->get_pid()); break; case SIMCALL_COMM_IRECV: if (issuer->get_host()) - label = simgrid::xbt::string_printf("[(%ld)%s] iRecv", issuer->get_pid(), MC_smx_actor_get_host_name(issuer)); + label = xbt::string_printf("[(%ld)%s] iRecv", issuer->get_pid(), MC_smx_actor_get_host_name(issuer)); else - label = simgrid::xbt::string_printf("[(%ld)] iRecv", issuer->get_pid()); + label = xbt::string_printf("[(%ld)] iRecv", issuer->get_pid()); break; case SIMCALL_COMM_WAIT: if (value == -1) { if (issuer->get_host()) - label = simgrid::xbt::string_printf("[(%ld)%s] WaitTimeout", issuer->get_pid(), - MC_smx_actor_get_host_name(issuer)); + label = xbt::string_printf("[(%ld)%s] WaitTimeout", issuer->get_pid(), MC_smx_actor_get_host_name(issuer)); else - label = simgrid::xbt::string_printf("[(%ld)] WaitTimeout", issuer->get_pid()); + label = xbt::string_printf("[(%ld)] WaitTimeout", issuer->get_pid()); } else { - simgrid::kernel::activity::ActivityImpl* remote_act = simcall_comm_wait__getraw__comm(req); - simgrid::mc::Remote temp_comm; - mc_model_checker->process().read(temp_comm, - remote(static_cast(remote_act))); - simgrid::kernel::activity::CommImpl* comm = temp_comm.get_buffer(); - - smx_actor_t src_proc = mc_model_checker->process().resolve_actor(simgrid::mc::remote(comm->src_actor_.get())); - smx_actor_t dst_proc = mc_model_checker->process().resolve_actor(simgrid::mc::remote(comm->dst_actor_.get())); + kernel::activity::ActivityImpl* remote_act = simcall_comm_wait__getraw__comm(req); + Remote temp_comm; + mc_model_checker->get_remote_simulation().read(temp_comm, + remote(static_cast(remote_act))); + const kernel::activity::CommImpl* comm = temp_comm.get_buffer(); + + const kernel::actor::ActorImpl* src_proc = + mc_model_checker->get_remote_simulation().resolve_actor(mc::remote(comm->src_actor_.get())); + const kernel::actor::ActorImpl* dst_proc = + mc_model_checker->get_remote_simulation().resolve_actor(mc::remote(comm->dst_actor_.get())); if (issuer->get_host()) - label = simgrid::xbt::string_printf("[(%ld)%s] Wait [(%ld)->(%ld)]", issuer->get_pid(), - MC_smx_actor_get_host_name(issuer), src_proc ? src_proc->get_pid() : 0, - dst_proc ? dst_proc->get_pid() : 0); + label = + xbt::string_printf("[(%ld)%s] Wait [(%ld)->(%ld)]", issuer->get_pid(), MC_smx_actor_get_host_name(issuer), + src_proc ? src_proc->get_pid() : 0, dst_proc ? dst_proc->get_pid() : 0); else - label = simgrid::xbt::string_printf("[(%ld)] Wait [(%ld)->(%ld)]", issuer->get_pid(), - src_proc ? src_proc->get_pid() : 0, dst_proc ? dst_proc->get_pid() : 0); + label = xbt::string_printf("[(%ld)] Wait [(%ld)->(%ld)]", issuer->get_pid(), + src_proc ? src_proc->get_pid() : 0, dst_proc ? dst_proc->get_pid() : 0); } break; case SIMCALL_COMM_TEST: { - simgrid::kernel::activity::ActivityImpl* remote_act = simcall_comm_test__getraw__comm(req); - simgrid::mc::Remote temp_comm; - mc_model_checker->process().read(temp_comm, - remote(static_cast(remote_act))); - simgrid::kernel::activity::CommImpl* comm = temp_comm.get_buffer(); + kernel::activity::ActivityImpl* remote_act = simcall_comm_test__getraw__comm(req); + Remote temp_comm; + mc_model_checker->get_remote_simulation().read(temp_comm, + remote(static_cast(remote_act))); + const kernel::activity::CommImpl* comm = temp_comm.get_buffer(); if (comm->src_actor_.get() == nullptr || comm->dst_actor_.get() == nullptr) { if (issuer->get_host()) - label = simgrid::xbt::string_printf("[(%ld)%s] Test FALSE", issuer->get_pid(), - MC_smx_actor_get_host_name(issuer)); + label = xbt::string_printf("[(%ld)%s] Test FALSE", issuer->get_pid(), MC_smx_actor_get_host_name(issuer)); else label = bprintf("[(%ld)] Test FALSE", issuer->get_pid()); } else { if (issuer->get_host()) - label = - simgrid::xbt::string_printf("[(%ld)%s] Test TRUE", issuer->get_pid(), MC_smx_actor_get_host_name(issuer)); + label = xbt::string_printf("[(%ld)%s] Test TRUE", issuer->get_pid(), MC_smx_actor_get_host_name(issuer)); else - label = simgrid::xbt::string_printf("[(%ld)] Test TRUE", issuer->get_pid()); + label = xbt::string_printf("[(%ld)] Test TRUE", issuer->get_pid()); } break; } @@ -492,55 +476,52 @@ std::string request_get_dot_output(smx_simcall_t req, int value) case SIMCALL_COMM_WAITANY: { size_t comms_size = simcall_comm_waitany__get__count(req); if (issuer->get_host()) - label = simgrid::xbt::string_printf("[(%ld)%s] WaitAny [%d of %zu]", issuer->get_pid(), - MC_smx_actor_get_host_name(issuer), value + 1, comms_size); + label = xbt::string_printf("[(%ld)%s] WaitAny [%d of %zu]", issuer->get_pid(), + MC_smx_actor_get_host_name(issuer), value + 1, comms_size); else - label = simgrid::xbt::string_printf("[(%ld)] WaitAny [%d of %zu]", issuer->get_pid(), value + 1, comms_size); + label = xbt::string_printf("[(%ld)] WaitAny [%d of %zu]", issuer->get_pid(), value + 1, comms_size); break; } case SIMCALL_COMM_TESTANY: if (value == -1) { if (issuer->get_host()) - label = simgrid::xbt::string_printf("[(%ld)%s] TestAny FALSE", issuer->get_pid(), - MC_smx_actor_get_host_name(issuer)); + label = xbt::string_printf("[(%ld)%s] TestAny FALSE", issuer->get_pid(), MC_smx_actor_get_host_name(issuer)); else - label = simgrid::xbt::string_printf("[(%ld)] TestAny FALSE", issuer->get_pid()); + label = xbt::string_printf("[(%ld)] TestAny FALSE", issuer->get_pid()); } else { if (issuer->get_host()) - label = simgrid::xbt::string_printf("[(%ld)%s] TestAny TRUE [%d of %lu]", issuer->get_pid(), - MC_smx_actor_get_host_name(issuer), value + 1, - simcall_comm_testany__get__count(req)); + label = + xbt::string_printf("[(%ld)%s] TestAny TRUE [%d of %lu]", issuer->get_pid(), + MC_smx_actor_get_host_name(issuer), value + 1, simcall_comm_testany__get__count(req)); else - label = simgrid::xbt::string_printf("[(%ld)] TestAny TRUE [%d of %lu]", issuer->get_pid(), value + 1, - simcall_comm_testany__get__count(req)); + label = xbt::string_printf("[(%ld)] TestAny TRUE [%d of %lu]", issuer->get_pid(), value + 1, + simcall_comm_testany__get__count(req)); } break; case SIMCALL_MUTEX_TRYLOCK: - label = simgrid::xbt::string_printf("[(%ld)] Mutex TRYLOCK", issuer->get_pid()); + label = xbt::string_printf("[(%ld)] Mutex TRYLOCK", issuer->get_pid()); break; case SIMCALL_MUTEX_LOCK: - label = simgrid::xbt::string_printf("[(%ld)] Mutex LOCK", issuer->get_pid()); + label = xbt::string_printf("[(%ld)] Mutex LOCK", issuer->get_pid()); break; case SIMCALL_MC_RANDOM: if (issuer->get_host()) - label = simgrid::xbt::string_printf("[(%ld)%s] MC_RANDOM (%d)", issuer->get_pid(), - MC_smx_actor_get_host_name(issuer), value); + label = xbt::string_printf("[(%ld)%s] MC_RANDOM (%d)", issuer->get_pid(), MC_smx_actor_get_host_name(issuer), + value); else - label = simgrid::xbt::string_printf("[(%ld)] MC_RANDOM (%d)", issuer->get_pid(), value); + label = xbt::string_printf("[(%ld)] MC_RANDOM (%d)", issuer->get_pid(), value); break; default: THROW_UNIMPLEMENTED; } - return simgrid::xbt::string_printf( - "label = \"%s\", color = %s, fontcolor = %s", label.c_str(), - color, color); + return xbt::string_printf("label = \"%s\", color = %s, fontcolor = %s", label.c_str(), color, color); } -} -} +} // namespace mc +} // namespace simgrid