X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/ae719657fdc26201d71d3df9b919129e8c7aa554..080541387d2296c808397215b8c9401a9f97a1d5:/src/kernel/activity/ConditionVariableImpl.cpp diff --git a/src/kernel/activity/ConditionVariableImpl.cpp b/src/kernel/activity/ConditionVariableImpl.cpp index 6f95c23356..8dad63fe86 100644 --- a/src/kernel/activity/ConditionVariableImpl.cpp +++ b/src/kernel/activity/ConditionVariableImpl.cpp @@ -1,16 +1,15 @@ -/* Copyright (c) 2007-2021. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2007-2022. 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 "src/kernel/activity/ConditionVariableImpl.hpp" -#include "simgrid/Exception.hpp" #include "src/kernel/activity/MutexImpl.hpp" -#include "src/kernel/activity/SynchroRaw.hpp" -#include "src/mc/checker/SimcallObserver.hpp" +#include "src/kernel/activity/Synchro.hpp" +#include "src/kernel/actor/SimcallObserver.hpp" #include // std::isfinite -XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_condition, simix_synchro, "Condition variables"); +XBT_LOG_NEW_DEFAULT_SUBCATEGORY(ker_condition, ker_synchro, "Condition variables kernel-space implementation"); /********************************* Condition **********************************/ @@ -21,15 +20,14 @@ namespace activity { /** * @brief Signalizes a condition. * - * Signalizes a condition and wakes up a sleeping process. - * If there are no process sleeping, no action is done. + * Signalizes a condition and wakes up a sleeping actor. + * If there are no actor sleeping, no action is done. */ void ConditionVariableImpl::signal() { XBT_DEBUG("Signal condition %p", this); - /* If there are processes waiting for the condition choose one and try - to make it acquire the mutex */ + /* If there are actors waiting for the condition choose one and try to make it acquire the mutex */ if (not sleeping_.empty()) { auto& proc = sleeping_.front(); sleeping_.pop_front(); @@ -38,10 +36,10 @@ void ConditionVariableImpl::signal() proc.waiting_synchro_ = nullptr; /* Now transform the cond wait simcall into a mutex lock one */ - smx_simcall_t simcall = &proc.simcall_; - const auto* observer = dynamic_cast(simcall->observer_); + simix::Simcall* simcall = &proc.simcall_; + const auto* observer = dynamic_cast(simcall->observer_); xbt_assert(observer != nullptr); - observer->get_mutex()->lock(simcall->issuer_); + observer->get_mutex()->lock_async(simcall->issuer_)->wait_for(simcall->issuer_, -1); } XBT_OUT(); } @@ -49,8 +47,8 @@ void ConditionVariableImpl::signal() /** * @brief Broadcasts a condition. * - * Signal ALL processes waiting on a condition. - * If there are no process waiting, no action is done. + * Signal ALL actors waiting on a condition. + * If there are no actor waiting, no action is done. */ void ConditionVariableImpl::broadcast() { @@ -75,9 +73,9 @@ void ConditionVariableImpl::wait(smx_mutex_t mutex, double timeout, actor::Actor mutex->unlock(issuer); } - RawImplPtr synchro(new RawImpl([this, issuer]() { + SynchroImplPtr synchro(new SynchroImpl([this, issuer]() { this->remove_sleeping_actor(*issuer); - auto* observer = dynamic_cast(issuer->simcall_.observer_); + auto* observer = dynamic_cast(issuer->simcall_.observer_); xbt_assert(observer != nullptr); observer->set_result(true); })); @@ -87,12 +85,12 @@ void ConditionVariableImpl::wait(smx_mutex_t mutex, double timeout, actor::Actor } // boost::intrusive_ptr support: -void intrusive_ptr_add_ref(simgrid::kernel::activity::ConditionVariableImpl* cond) +void intrusive_ptr_add_ref(ConditionVariableImpl* cond) { cond->refcount_.fetch_add(1, std::memory_order_relaxed); } -void intrusive_ptr_release(simgrid::kernel::activity::ConditionVariableImpl* cond) +void intrusive_ptr_release(ConditionVariableImpl* cond) { if (cond->refcount_.fetch_sub(1, std::memory_order_release) == 1) { std::atomic_thread_fence(std::memory_order_acquire);