X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;ds=sidebyside;f=src%2Fthread.cpp;h=cdb0d5412a501b781e4064815eacf255b358ef66;hb=ca3835857435590865fd41aac9bf4c1cda76dfc3;hp=2fd3c7aa9e37fbfc24b579ddd2f89d33dbb6d983;hpb=5d1b92e8f9836e1d403bcf60653dcf6b059c8720;p=stockfish diff --git a/src/thread.cpp b/src/thread.cpp index 2fd3c7aa..cdb0d541 100644 --- a/src/thread.cpp +++ b/src/thread.cpp @@ -200,21 +200,22 @@ void Thread::split(Position& pos, Stack* ss, Value alpha, Value beta, Value* bes assert(!searching); assert(!activePosition); - searching = true; - // We have returned from the idle loop, which means that all threads are // finished. Note that decreasing splitPointsSize must be done under lock // protection to avoid a race with Thread::can_join(). - sp.spinlock.acquire(); + spinlock.acquire(); + searching = true; --splitPointsSize; activeSplitPoint = sp.parentSplitPoint; activePosition = &pos; + + spinlock.release(); + + // Split point data cannot be changed now, so no need to lock protect pos.set_nodes_searched(pos.nodes_searched() + sp.nodes); *bestMove = sp.bestMove; *bestValue = sp.bestValue; - - sp.spinlock.release(); } @@ -321,10 +322,6 @@ void ThreadPool::read_uci_options() { assert(requested > 0); - // If zero (default) then set best minimum split depth automatically - if (!minimumSplitDepth) - minimumSplitDepth = requested < 8 ? 4 * ONE_PLY : 7 * ONE_PLY; - while (size() < requested) push_back(new_thread());