LBRYCommunity/lbrycrd
1
0
Fork 0
Code Issues 29 Pull requests 1 Projects Releases 32 Packages Wiki Activity

Cap nAttempts penalty at 8 and switch to pow instead of a division loop.

Browse source 
On hosts that had spent some time with a failed internet connection their
 nAttempts penalty was going through the roof (e.g. thousands for all peers)
 and as a result the connect search was pegging the CPU and failing to get
 more than a 4 connections after days of running (because it was taking so
 long per try).
This commit is contained in:
Gregory Maxwell 2015-04-19 11:47:56 -07:00
parent 8f955b9661
commit a784f90c98
1 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/addrman.cpp
View file

@ -67,9 +67,8 @@ double CAddrInfo::GetChance(int64_t nNow) const
if (nSinceLastTry < 60 * 10) if (nSinceLastTry < 60 * 10)
fChance *= 0.01; fChance *= 0.01;
// deprioritize 50% after each failed attempt // deprioritize 66% after each failed attempt, but at most 1/28th to avoid the search taking forever or overly penalizing outages.
for (int n = 0; n < nAttempts; n++) fChance *= pow(0.66, min(nAttempts, 8));
fChance /= 1.5;
return fChance; return fChance;
} }