kate/BadFish
1
0
Fork 0
mirror of https://github.com/sockspls/badfish synced 2025-04-30 08:43:09 +00:00
Code Issues Releases Wiki Activity

Move beta counter variables to the per-thread data

Browse source 
This should reduce concurrent accessing in SMP case.

Suggestion by Tord Romstad.

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
This commit is contained in:
Marco Costalba 2009-05-16 12:22:40 +01:00
parent 436fa5c8fa
commit a75aa6035b
2 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/search.cpp
View file

@ -72,7 +72,8 @@ namespace {
// Apart for the first one that has its score, following moves
// normally have score -VALUE_INFINITE, so are ordered according
// to the number of beta cutoffs occurred under their subtree during
// the last iteration.
// the last iteration. The counters are per thread variables to avoid
// concurrent accessing under SMP case.
struct BetaCounterType {
@ -80,8 +81,6 @@ namespace {
void clear();
void add(Color us, Depth d, int threadID);
void read(Color us, int64_t& our, int64_t& their);
int64_t hits[THREAD_MAX][2];
};
@ -1894,13 +1893,13 @@ namespace {
void BetaCounterType::clear() {
for (int i = 0; i < THREAD_MAX; i++)
hits[i][WHITE] = hits[i][BLACK] = 0ULL;
Threads[i].betaCutOffs[WHITE] = Threads[i].betaCutOffs[BLACK] = 0ULL;
}
void BetaCounterType::add(Color us, Depth d, int threadID) {
// Weighted count based on depth
hits[threadID][us] += int(d);
Threads[threadID].betaCutOffs[us] += unsigned(d);
}
void BetaCounterType::read(Color us, int64_t& our, int64_t& their) {
@ -1908,8 +1907,8 @@ namespace {
our = their = 0UL;
for (int i = 0; i < THREAD_MAX; i++)
{
our += hits[i][us];
their += hits[i][opposite_color(us)];
our += Threads[i].betaCutOffs[us];
their += Threads[i].betaCutOffs[opposite_color(us)];
}
}

3
src/thread.h
View file

@ -66,13 +66,14 @@ struct Thread {
SplitPoint *splitPoint;
int activeSplitPoints;
uint64_t nodes;
uint64_t betaCutOffs[2];
bool failHighPly1;
volatile bool stop;
volatile bool running;
volatile bool idle;
volatile bool workIsWaiting;
volatile bool printCurrentLine;
unsigned char pad[64];
unsigned char pad[64]; // set some distance among local data for each thread
};