Fully yielding locks, no spinning

zamar · zamar · commit a7381d5e8159 · 2015-03-24T21:34:19.000Z
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
@@ -48,8 +48,8 @@ class Spinlock {
   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); }
 };
