X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fthread.cpp;h=faa3a5716aa1570945c5dcaea7cb345ca0e75455;hp=6fefd58a792b9225d7a121bf102e38e18a8850fc;hb=5a2d525048ca5eec1d5d0effc8868cb2fdc701bb;hpb=5e18b81e870dd44dd8fadab1a4e428d79e6b09d9 diff --git a/src/thread.cpp b/src/thread.cpp index 6fefd58a..faa3a571 100644 --- a/src/thread.cpp +++ b/src/thread.cpp @@ -34,10 +34,14 @@ namespace { extern "C" { // start_routine() is the C function which is called when a new thread // is launched. It is a wrapper to member function pointed by start_fn. - long start_routine(Thread* th) { (th->*(th->start_fn))(); return 0; } + long start_routine(Thread* th) { -} } + Threads.set_this_thread(th); // Save pointer into thread local storage + (th->*(th->start_fn))(); + return 0; + } +} } // Thread c'tor starts a newly-created thread of execution that will call // the idle loop function pointed by start_fn going immediately to sleep. @@ -48,7 +52,7 @@ Thread::Thread(Fn fn) { maxPly = splitPointsCnt = 0; curSplitPoint = NULL; start_fn = fn; - threadID = Threads.size(); + idx = Threads.size(); do_sleep = (fn != &Thread::main_loop); // Avoid a race with start_searching() @@ -60,7 +64,7 @@ Thread::Thread(Fn fn) { if (!thread_create(handle, start_routine, this)) { - std::cerr << "Failed to create thread number " << threadID << std::endl; + std::cerr << "Failed to create thread number " << idx << std::endl; ::exit(EXIT_FAILURE); } } @@ -173,13 +177,13 @@ bool Thread::cutoff_occurred() const { // Thread::is_available_to() checks whether the thread is available to help the -// thread with threadID "master" at a split point. An obvious requirement is that -// thread must be idle. With more than two threads, this is not sufficient: If -// the thread is the master of some active split point, it is only available as a -// slave to the threads which are busy searching the split point at the top of -// "slave"'s split point stack (the "helpful master concept" in YBWC terminology). +// thread 'master' at a split point. An obvious requirement is that thread must +// be idle. With more than two threads, this is not sufficient: If the thread is +// the master of some active split point, it is only available as a slave to the +// slaves which are busy searching the split point at the top of slaves split +// point stack (the "helpful master concept" in YBWC terminology). -bool Thread::is_available_to(int master) const { +bool Thread::is_available_to(Thread* master) const { if (is_searching) return false; @@ -190,7 +194,7 @@ bool Thread::is_available_to(int master) const { // No active split points means that the thread is available as a slave for any // other thread otherwise apply the "helpful master" concept if possible. - return !spCnt || (splitPoints[spCnt - 1].slavesMask & (1ULL << master)); + return !spCnt || (splitPoints[spCnt - 1].slavesMask & (1ULL << master->idx)); } @@ -201,10 +205,12 @@ bool Thread::is_available_to(int master) const { void ThreadsManager::init() { + tls_init(tlsKey); cond_init(sleepCond); lock_init(splitLock); timer = new Thread(&Thread::timer_loop); threads.push_back(new Thread(&Thread::main_loop)); + set_this_thread(main_thread()); // Use main thread's resources read_uci_options(); } @@ -219,6 +225,7 @@ ThreadsManager::~ThreadsManager() { delete timer; lock_destroy(splitLock); cond_destroy(sleepCond); + tls_destroy(tlsKey); } @@ -275,11 +282,9 @@ void ThreadsManager::sleep() const { // available_slave_exists() tries to find an idle thread which is available as -// a slave for the thread with threadID 'master'. +// a slave for the thread 'master'. -bool ThreadsManager::available_slave_exists(int master) const { - - assert(master >= 0 && master < size()); +bool ThreadsManager::available_slave_exists(Thread* master) const { for (int i = 0; i < size(); i++) if (threads[i]->is_available_to(master)) @@ -309,19 +314,18 @@ Value ThreadsManager::split(Position& pos, Stack* ss, Value alpha, Value beta, assert(beta <= VALUE_INFINITE); assert(depth > DEPTH_ZERO); - int master = pos.thread(); - Thread& masterThread = *threads[master]; + Thread* master = this_thread(); - if (masterThread.splitPointsCnt >= MAX_SPLITPOINTS_PER_THREAD) + if (master->splitPointsCnt >= MAX_SPLITPOINTS_PER_THREAD) return bestValue; // Pick the next available split point from the split point stack - SplitPoint* sp = &masterThread.splitPoints[masterThread.splitPointsCnt++]; + SplitPoint* sp = &master->splitPoints[master->splitPointsCnt++]; - sp->parent = masterThread.curSplitPoint; + sp->parent = master->curSplitPoint; sp->master = master; sp->cutoff = false; - sp->slavesMask = 1ULL << master; + sp->slavesMask = 1ULL << master->idx; sp->depth = depth; sp->bestMove = *bestMove; sp->threatMove = threatMove; @@ -335,9 +339,9 @@ Value ThreadsManager::split(Position& pos, Stack* ss, Value alpha, Value beta, sp->nodes = 0; sp->ss = ss; - assert(masterThread.is_searching); + assert(master->is_searching); - masterThread.curSplitPoint = sp; + master->curSplitPoint = sp; int slavesCnt = 0; // Try to allocate available threads and ask them to start searching setting @@ -370,11 +374,11 @@ Value ThreadsManager::split(Position& pos, Stack* ss, Value alpha, Value beta, // their work at this split point. if (slavesCnt || Fake) { - masterThread.idle_loop(sp); + master->idle_loop(sp); // In helpful master concept a master can help only a sub-tree of its split // point, and because here is all finished is not possible master is booked. - assert(!masterThread.is_searching); + assert(!master->is_searching); } // We have returned from the idle loop, which means that all threads are @@ -383,9 +387,9 @@ Value ThreadsManager::split(Position& pos, Stack* ss, Value alpha, Value beta, lock_grab(sp->lock); // To protect sp->nodes lock_grab(splitLock); - masterThread.is_searching = true; - masterThread.splitPointsCnt--; - masterThread.curSplitPoint = sp->parent; + master->is_searching = true; + master->splitPointsCnt--; + master->curSplitPoint = sp->parent; pos.set_nodes_searched(pos.nodes_searched() + sp->nodes); *bestMove = sp->bestMove; @@ -417,11 +421,11 @@ void ThreadsManager::set_timer(int msec) { void ThreadsManager::wait_for_search_finished() { - Thread* main = threads[0]; - lock_grab(main->sleepLock); - cond_signal(main->sleepCond); // In case is waiting for stop or ponderhit - while (!main->do_sleep) cond_wait(sleepCond, main->sleepLock); - lock_release(main->sleepLock); + Thread* t = main_thread(); + lock_grab(t->sleepLock); + cond_signal(t->sleepCond); // In case is waiting for stop or ponderhit + while (!t->do_sleep) cond_wait(sleepCond, t->sleepLock); + lock_release(t->sleepLock); } @@ -429,20 +433,22 @@ void ThreadsManager::wait_for_search_finished() { // main_loop() so to start a new search, then returns immediately. void ThreadsManager::start_searching(const Position& pos, const LimitsType& limits, - const std::set& searchMoves) { + const std::vector& searchMoves) { wait_for_search_finished(); + SearchTime.restart(); // As early as possible + Signals.stopOnPonderhit = Signals.firstRootMove = false; Signals.stop = Signals.failedLowAtRoot = false; - RootPosition.copy(pos, 0); + RootPosition = pos; Limits = limits; RootMoves.clear(); for (MoveList ml(pos); !ml.end(); ++ml) - if (searchMoves.empty() || searchMoves.count(ml.move())) + if (searchMoves.empty() || count(searchMoves.begin(), searchMoves.end(), ml.move())) RootMoves.push_back(RootMove(ml.move())); - threads[0]->do_sleep = false; - threads[0]->wake_up(); + main_thread()->do_sleep = false; + main_thread()->wake_up(); }