Skip to content

Conversation

@rustam-cpp
Copy link
Contributor

STC (10+0.1 th1) was accepted:
LLR: 2.95 (-2.94,2.94) <0.00,2.00>
Total: 75712 W: 19701 L: 19326 D: 36685
Ptnml(0-2): 254, 8738, 19513, 9081, 270
https://tests.stockfishchess.org/tests/view/68e286d5fa806e2e8393d160

LTC (60+0.6 th1) was accepted:
LLR: 2.96 (-2.94,2.94) <0.50,2.50>
Total: 108492 W: 28068 L: 27604 D: 52820
Ptnml(0-2): 60, 11639, 30390, 12091, 66
https://tests.stockfishchess.org/tests/view/68e3e564a017f472e763dac0

bench 2128316

@anematode
Copy link
Contributor

interesting, I'm getting a pretty dramatic decrease in nodes per second (~4%) with this branch. Maybe someone else can corroborate? If so that means the increased size of the pawn history table is worth a lot, but also that it might be good to somehow try to optimize it for cache locality or density.

@rustam-cpp
Copy link
Contributor Author

interesting, I'm getting a pretty dramatic decrease in nodes per second (~4%) with this branch. Maybe someone else can corroborate? If so that means the increased size of the pawn history table is worth a lot, but also that it might be good to somehow try to optimize it for cache locality or density.

maybe i can test a decrease in NPS? if so, say pls, how to do it

@AliceRoselia
Copy link
Contributor

maybe i can test a decrease in NPS? if so, say pls, how to do it

Use the speedtest function and compare the before and after.

@rustam-cpp
Copy link
Contributor Author

image image the results may be noisy, because while the these tests run, my processor was running a lichess bot :) so idk are these results true of not

@Torom
Copy link
Contributor

Torom commented Oct 9, 2025

I think you tested both times the patch. Both times it shows the commit 3ba08ea6 as the version.

And yes, to ensure that the test is as accurate as possible, nothing CPU-intensive should be running on the system at the same time.

@rustam-cpp
Copy link
Contributor Author

I think you tested both times the patch. Both times it shows the commit 3ba08ea6 as the version.

And yes, to ensure that the test is as accurate as possible, nothing CPU-intensive should be running on the system at the same time.

ok, i'll rerun tests, but if you think it's the same version, why the results are different?

@rustam-cpp
Copy link
Contributor Author

image image yeah, i think it's true, exactly 4%

@Torom
Copy link
Contributor

Torom commented Oct 9, 2025

master

Version                    : Stockfish dev-20251007-e18ed795
Compiled by                : g++ (GNUC) 15.2.1 on Linux
Compilation architecture   : x86-64-bmi2
Compilation settings       : 64bit BMI2 AVX2 SSE41 SSSE3 SSE2 POPCNT
Compiler __VERSION__ macro : 15.2.1 20250813
Large pages                : yes
User invocation            : speedtest 
Filled invocation          : speedtest 8 1024 150
Available processors       : 0-7
Thread count               : 8
Thread binding             : none
TT size [MiB]              : 1024
Hash max, avg [per mille]  : 
    single search          : 29, 14
    single game            : 443, 293
Total nodes searched       : 569655536
Total search time [s]      : 153.622
Nodes/second               : 3708163
...
Nodes/second               : 3791878
...
Nodes/second               : 3746873

patch

Version                    : Stockfish dev-20251008-3ba08ea6
Compiled by                : g++ (GNUC) 15.2.1 on Linux
Compilation architecture   : x86-64-bmi2
Compilation settings       : 64bit BMI2 AVX2 SSE41 SSSE3 SSE2 POPCNT
Compiler __VERSION__ macro : 15.2.1 20250813
Large pages                : yes
User invocation            : speedtest 
Filled invocation          : speedtest 8 1024 150
Available processors       : 0-7
Thread count               : 8
Thread binding             : none
TT size [MiB]              : 1024
Hash max, avg [per mille]  : 
    single search          : 31, 14
    single game            : 454, 289
Total nodes searched       : 576534583
Total search time [s]      : 153.614
Nodes/second               : 3753138
...
Nodes/second               : 3742318
...
Nodes/second               : 3722030

no slowdown here

@rustam-cpp
Copy link
Contributor Author

what is label "gainer"?

@vondele
Copy link
Member

vondele commented Oct 11, 2025

what is label "gainer"?

the best label :-)
Gains Elo

@anematode
Copy link
Contributor

I think I might have just measured the wrong binary... looks good to me now :P

@rustam-cpp
Copy link
Contributor Author

Master:

Stockfish dev-20251007-e18ed795 by the Stockfish developers (see AUTHORS file)
speedtest
info string Using 12 threads
Warmup position 3/3
Position 258/258
===========================
Version                    : Stockfish dev-20251007-e18ed795
Compiled by                : g++ (GNUC) 15.1.1 on Linux
Compilation architecture   : x86-64-avx2
Compilation settings       : 64bit AVX2 SSE41 SSSE3 SSE2 POPCNT
Compiler __VERSION__ macro : 15.1.1 20250425
Large pages                : yes
User invocation            : speedtest 
Filled invocation          : speedtest 12 1536 150
Available processors       : 0-11
Thread count               : 12
Thread binding             : none
TT size [MiB]              : 1536
Hash max, avg [per mille]  : 
    single search          : 20, 7
    single game            : 260, 163
Total nodes searched       : 522005728
Total search time [s]      : 153.759
Nodes/second               : 3394960

Patch:

Stockfish dev-20251008-3ba08ea6 by the Stockfish developers (see AUTHORS file)
speedtest
info string Using 12 threads
Warmup position 3/3
Position 258/258
===========================
Version                    : Stockfish dev-20251008-3ba08ea6
Compiled by                : g++ (GNUC) 15.1.1 on Linux
Compilation architecture   : x86-64-avx2
Compilation settings       : 64bit AVX2 SSE41 SSSE3 SSE2 POPCNT
Compiler __VERSION__ macro : 15.1.1 20250425
Large pages                : yes
User invocation            : speedtest 
Filled invocation          : speedtest 12 1536 150
Available processors       : 0-11
Thread count               : 12
Thread binding             : none
TT size [MiB]              : 1536
Hash max, avg [per mille]  : 
    single search          : 24, 8
    single game            : 258, 169
Total nodes searched       : 523034820
Total search time [s]      : 153.785
Nodes/second               : 3401078

i tried second time

ppigazzini pushed a commit to official-stockfish/fishtest that referenced this pull request Oct 12, 2025
a pending SF patch requires an update of the memory estimate

official-stockfish/Stockfish#6350
@vondele vondele added the to be merged Will be merged shortly label Oct 14, 2025
@vondele vondele closed this in 63d449d Oct 14, 2025
vondele pushed a commit that referenced this pull request Oct 14, 2025
As the worker data is quite large (28MB after #6350) we can make use of huge pages as a speedup.

prior to #6350

STC passed elo gaining bounds:
LLR: 2.95 (-2.94,2.94) <0.00,2.00>
Total: 166272 W: 43479 L: 42993 D: 79800
Ptnml(0-2): 540, 17598, 46365, 18102, 531
https://tests.stockfishchess.org/tests/view/68e9f3c0d323fd15c04e3ba4

Tested the speedup on a large machine with speedtest:
==== master ====
Average (over 20):  288644510
==== largePageWorker ====
Average (over 20):  292082422

Test after #6350:
==== rustam-cpp-testPR ====
Average (over 20):  291035351
==== rustam-cpp-testPR-pages ====
Average (over 20):  291937367

#6359

No functional change
MinetaS pushed a commit to MinetaS/Stockfish that referenced this pull request Oct 21, 2025
As the worker data is quite large (28MB after official-stockfish#6350) we can make use of huge pages as a speedup.

prior to official-stockfish#6350

STC passed elo gaining bounds:
LLR: 2.95 (-2.94,2.94) <0.00,2.00>
Total: 166272 W: 43479 L: 42993 D: 79800
Ptnml(0-2): 540, 17598, 46365, 18102, 531
https://tests.stockfishchess.org/tests/view/68e9f3c0d323fd15c04e3ba4

Tested the speedup on a large machine with speedtest:
==== master ====
Average (over 20):  288644510
==== largePageWorker ====
Average (over 20):  292082422

Test after official-stockfish#6350:
==== rustam-cpp-testPR ====
Average (over 20):  291035351
==== rustam-cpp-testPR-pages ====
Average (over 20):  291937367

official-stockfish#6359

No functional change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀 gainer to be merged Will be merged shortly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants