From 5be3d98d175e22b182bbef3a3085c0828a3baf77 Mon Sep 17 00:00:00 2001 From: Joona Kiiski Date: Fri, 24 Jul 2009 17:34:41 +0300 Subject: [PATCH] Do not adjust Minimum Split Depth automatically Currently minimum split depth is set automatically to 6 when number of CPUs is more than 4. I believe this is a bad idea since for example my quad (4CPU with hyperthreading) is detected as 8CPU computer. I've manually lowered down the number of Threads, but so far I have played all games with Minimum Split Depth set to 6! Since 4CPU computers with hyperthreading are quite common and 8 CPU computers extremely rear (I expect we can get a direct jump to 16 or 32 cores), this automatic adjusting is likely to do more harm than good. Add a note in Readme.txt, so that those rear 8CPU owners can manually tweak the "Minimum Split Depth" parameter Signed-off-by: Marco Costalba --- Readme.txt | 4 +++- src/ucioption.cpp | 9 --------- 2 files changed, 3 insertions(+), 10 deletions(-) diff --git a/Readme.txt b/Readme.txt index 1a371398..09216abc 100644 --- a/Readme.txt +++ b/Readme.txt @@ -14,7 +14,9 @@ number of CPUs on your computer and set the number of search threads accordingly, but please be aware that the detection is not always correct. It is therefore recommended to inspect the value of the "Threads" UCI parameter, and to make sure it equals the number of CPU -cores on your computer. +cores on your computer. If you are using more than four threads, it +is recommended to raise the value of "Minimum Split Depth" UCI parameter +to 6. 2. Files diff --git a/src/ucioption.cpp b/src/ucioption.cpp index e25877fe..57264166 100644 --- a/src/ucioption.cpp +++ b/src/ucioption.cpp @@ -194,15 +194,6 @@ void init_uci_options() { options["Threads"].defaultValue = stringify(Min(cpu_count(), 7)); options["Threads"].currentValue = stringify(Min(cpu_count(), 7)); - - // Increase the minimum split depth when the number of CPUs is big. - // It would probably be better to let this depend on the number of threads - // instead. - if (cpu_count() > 4) - { - options["Minimum Split Depth"].defaultValue = "6"; - options["Minimum Split Depth"].currentValue = "6"; - } } -- 2.39.2