From: Joona Kiiski Date: Sat, 14 Mar 2015 21:20:11 +0000 (+0000) Subject: Fully yielding locks, no spinning X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=a7381d5e8159a08d0df7e8312bdbea8fc084da5c;ds=sidebyside Fully yielding locks, no spinning 7 threads: ELO: 2.00 +-2.7 (95%) LOS: 92.4% Total: 20000 W: 3276 L: 3161 D: 13563 There is no functional change in single thread mode Resolves #304 --- diff --git a/src/thread.h b/src/thread.h index ee7fb061..e880b01c 100644 --- a/src/thread.h +++ b/src/thread.h @@ -48,8 +48,8 @@ public: 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 + while (lock.load(std::memory_order_relaxed) <= 0) + std::this_thread::yield(); // Be nice to hyperthreading } void release() { lock.store(1, std::memory_order_release); } };