Use only one ConditionVariable to sync UI

[stockfish] / src / thread.h

diff --git a/src/thread.h b/src/thread.h
index 8f930149f333e764d3525fc9d93fd8f4dba3798b..215b27c688b2823b11112a56e862645cd7c726d0 100644 (file)
--- 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 {
 const size_t MAX_SLAVES_PER_SPLITPOINT = 4;
 
 class Spinlock {

-  std::atomic_int _lock;
+
+  std::atomic_int lock;

 
 public:
 
 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
   }
               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
   SplitPoint* parentSplitPoint;
 
   // Shared variable data

-  Spinlock mutex;
+  Spinlock spinlock;

   std::bitset<MAX_THREADS> slavesMask;
   volatile bool allSlavesSearching;
   volatile uint64_t nodes;
   std::bitset<MAX_THREADS> slavesMask;
   volatile bool allSlavesSearching;
   volatile uint64_t nodes;


@@ -97,6 +98,7 @@ struct ThreadBase {
 
   std::thread nativeThread;
   Mutex mutex;
 
   std::thread nativeThread;
   Mutex mutex;

+  Spinlock spinlock;

   ConditionVariable sleepCondition;
   volatile bool exit = false;
 };
   ConditionVariable sleepCondition;
   volatile bool exit = false;
 };


@@ -127,7 +129,6 @@ struct Thread : public ThreadBase {
   SplitPoint* volatile activeSplitPoint;
   volatile size_t splitPointsSize;
   volatile bool searching;
   SplitPoint* volatile activeSplitPoint;
   volatile size_t splitPointsSize;
   volatile bool searching;

-  Spinlock allocMutex;

 };
 
 
 };
 
 


@@ -136,6 +137,7 @@ struct Thread : public ThreadBase {
 
 struct MainThread : public Thread {
   virtual void idle_loop();
 
 struct MainThread : public Thread {
   virtual void idle_loop();

+  void join();

   volatile bool thinking = true; // Avoid a race with start_thinking()
 };
 
   volatile bool thinking = true; // Avoid a race with start_thinking()
 };
 


@@ -161,11 +163,9 @@ struct ThreadPool : public std::vector<Thread*> {
   MainThread* main() { return static_cast<MainThread*>(at(0)); }
   void read_uci_options();
   Thread* available_slave(const SplitPoint* sp) const;
   MainThread* main() { return static_cast<MainThread*>(at(0)); }
   void read_uci_options();
   Thread* available_slave(const SplitPoint* sp) const;

-  void wait_for_think_finished();

   void start_thinking(const Position&, const Search::LimitsType&, Search::StateStackPtr&);
 
   Depth minimumSplitDepth;
   void start_thinking(const Position&, const Search::LimitsType&, Search::StateStackPtr&);
 
   Depth minimumSplitDepth;

-  ConditionVariable sleepCondition;

   TimerThread* timer;
 };
 
   TimerThread* timer;
 };