X-Git-Url: http://bilbo.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/1865e94dfee342781e137d9e7e2aa1e423ac0169..5ac15923176130dc8e7274efb80fe108cc4bed02:/src/s4u/s4u_ConditionVariable.cpp diff --git a/src/s4u/s4u_ConditionVariable.cpp b/src/s4u/s4u_ConditionVariable.cpp index e43d0e9ff3..cfced69bdc 100644 --- a/src/s4u/s4u_ConditionVariable.cpp +++ b/src/s4u/s4u_ConditionVariable.cpp @@ -8,7 +8,8 @@ #include "simgrid/s4u/ConditionVariable.hpp" #include "simgrid/simix.h" #include "src/kernel/activity/ConditionVariableImpl.hpp" -#include "src/mc/checker/SimcallObserver.hpp" +#include "src/kernel/activity/MutexImpl.hpp" +#include "src/kernel/actor/SimcallObserver.hpp" #include "xbt/log.hpp" #include @@ -30,16 +31,16 @@ ConditionVariablePtr ConditionVariable::create() void ConditionVariable::wait(MutexPtr lock) { kernel::actor::ActorImpl* issuer = kernel::actor::ActorImpl::self(); - mc::ConditionWaitSimcall observer{issuer, pimpl_, lock->pimpl_}; - kernel::actor::simcall_blocking( + kernel::actor::ConditionWaitSimcall observer{issuer, pimpl_, lock->pimpl_}; + kernel::actor::simcall_blocking( [&observer] { observer.get_cond()->wait(observer.get_mutex(), -1.0, observer.get_issuer()); }, &observer); } void ConditionVariable::wait(const std::unique_lock& lock) { kernel::actor::ActorImpl* issuer = kernel::actor::ActorImpl::self(); - mc::ConditionWaitSimcall observer{issuer, pimpl_, lock.mutex()->pimpl_}; - kernel::actor::simcall_blocking( + kernel::actor::ConditionWaitSimcall observer{issuer, pimpl_, lock.mutex()->pimpl_}; + kernel::actor::simcall_blocking( [&observer] { observer.get_cond()->wait(observer.get_mutex(), -1.0, observer.get_issuer()); }, &observer); } @@ -49,7 +50,12 @@ std::cv_status s4u::ConditionVariable::wait_for(const std::unique_lock& l if (timeout < 0) timeout = 0.0; - if (simcall_cond_wait_timeout(pimpl_, lock.mutex()->pimpl_, timeout)) { + kernel::actor::ActorImpl* issuer = kernel::actor::ActorImpl::self(); + kernel::actor::ConditionWaitSimcall observer{issuer, pimpl_, lock.mutex()->pimpl_, timeout}; + bool timed_out = kernel::actor::simcall_blocking( + [&observer] { observer.get_cond()->wait(observer.get_mutex(), observer.get_timeout(), observer.get_issuer()); }, + &observer); + if (timed_out) { // If we reached the timeout, we have to take the lock again: lock.mutex()->lock(); return std::cv_status::timeout; @@ -60,7 +66,7 @@ std::cv_status s4u::ConditionVariable::wait_for(const std::unique_lock& l std::cv_status ConditionVariable::wait_until(const std::unique_lock& lock, double timeout_time) { - double now = SIMIX_get_clock(); + double now = Engine::get_clock(); double timeout; if (timeout_time < now) timeout = 0.0; @@ -109,7 +115,7 @@ void sg_cond_wait(sg_cond_t cond, sg_mutex_t mutex) int sg_cond_wait_for(sg_cond_t cond, sg_mutex_t mutex, double delay) { std::unique_lock lock(*mutex); - return cond->wait_for(lock, delay) == std::cv_status::timeout ? 1 : 0; + return cond->wait_for(lock, delay) == std::cv_status::timeout; } void sg_cond_notify_one(sg_cond_t cond)