class PosixSynchro : public Synchro {
public:
- explicit PosixSynchro(Parmap<T>& parmap);
- ~PosixSynchro();
+ explicit PosixSynchro(Parmap<T>& parmap) : Synchro(parmap) {}
void master_signal() override;
void master_wait() override;
void worker_signal() override;
this->workers[0] = nullptr;
for (unsigned i = 1; i < num_workers; i++) {
- ThreadData* data = new ThreadData(*this, i);
+ auto* data = new ThreadData(*this, i);
this->workers[i] = new std::thread(worker_main, data);
/* Bind the worker to a core if possible */
XBT_CDEBUG(xbt_parmap, "New worker thread created");
/* Worker's main loop */
- while (1) {
+ while (true) {
round++; // New scheduling round
parmap.synchro->worker_wait(round);
if (parmap.destroying)
delete data;
}
-template <typename T> Parmap<T>::PosixSynchro::PosixSynchro(Parmap<T>& parmap) : Synchro(parmap)
-{
-}
-
-template <typename T> Parmap<T>::PosixSynchro::~PosixSynchro()
-{
-}
-
template <typename T> void Parmap<T>::PosixSynchro::master_signal()
{
std::unique_lock<std::mutex> lk(ready_mutex);
template <typename T> void Parmap<T>::PosixSynchro::master_wait()
{
std::unique_lock<std::mutex> lk(done_mutex);
- while (this->parmap.thread_counter < this->parmap.num_workers) {
- /* wait for all workers to be ready */
- done_cond.wait(lk);
- }
+ /* wait for all workers to be ready */
+ done_cond.wait(lk, [this]() { return this->parmap.thread_counter >= this->parmap.num_workers; });
}
template <typename T> void Parmap<T>::PosixSynchro::worker_signal()
{
std::unique_lock<std::mutex> lk(ready_mutex);
/* wait for more work */
- while (this->parmap.work_round != round) {
- ready_cond.wait(lk);
- }
+ ready_cond.wait(lk, [this, round]() { return this->parmap.work_round == round; });
}
#if HAVE_FUTEX_H