X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fthread.h;h=7932ad45f6b0e967229b6557f69858a3abd94fcf;hp=8f930149f333e764d3525fc9d93fd8f4dba3798b;hb=13d4df95cd4c56c0e730328648de54d9ed8bd81e;hpb=bae4679de2db252764610f8dc0dfce125fac461e diff --git a/src/thread.h b/src/thread.h index 8f930149..7932ad45 100644 --- a/src/thread.h +++ b/src/thread.h @@ -41,16 +41,17 @@ const size_t MAX_SPLITPOINTS_PER_THREAD = 8; const size_t MAX_SLAVES_PER_SPLITPOINT = 4; class Spinlock { - std::atomic_int _lock; + + std::atomic_int lock; public: - Spinlock() { _lock = 1; } // Init here to workaround a bug with MSVC 2013 - void lock() { - while (_lock.fetch_sub(1, std::memory_order_acquire) != 1) - for (int cnt = 0; _lock.load(std::memory_order_relaxed) <= 0; ++cnt) + Spinlock() { lock = 1; } // Init here to workaround a bug with MSVC 2013 + void acquire() { + while (lock.fetch_sub(1, std::memory_order_acquire) != 1) + for (int cnt = 0; lock.load(std::memory_order_relaxed) <= 0; ++cnt) if (cnt >= 10000) std::this_thread::yield(); // Be nice to hyperthreading } - void unlock() { _lock.store(1, std::memory_order_release); } + void release() { lock.store(1, std::memory_order_release); } }; @@ -73,7 +74,7 @@ struct SplitPoint { SplitPoint* parentSplitPoint; // Shared variable data - Spinlock mutex; + Spinlock spinlock; std::bitset slavesMask; volatile bool allSlavesSearching; volatile uint64_t nodes; @@ -97,6 +98,7 @@ struct ThreadBase { std::thread nativeThread; Mutex mutex; + Spinlock spinlock; ConditionVariable sleepCondition; volatile bool exit = false; }; @@ -127,7 +129,6 @@ struct Thread : public ThreadBase { SplitPoint* volatile activeSplitPoint; volatile size_t splitPointsSize; volatile bool searching; - Spinlock allocMutex; };