X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=src%2Fthread.cpp;h=cdb0d5412a501b781e4064815eacf255b358ef66;hb=dc3508d157118cbda500bb42595575105a4602e5;hp=c73f55660330545d300857b2874287fd54b21563;hpb=2ca142a5b4ca200c56cb99495ec51a804983d07d;p=stockfish diff --git a/src/thread.cpp b/src/thread.cpp index c73f5566..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 = 5 * ONE_PLY ; - while (size() < requested) push_back(new_thread());