Skip to content
This repository was archived by the owner on Jul 17, 2025. It is now read-only.

Commit 33538fe

Browse files
committed
actually use max not min
Signed-off-by: Reto Achermann <achreto@gmail.com>
1 parent 056cf17 commit 33538fe

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

kernel/tests/s11_rackscale_benchmarks.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -624,7 +624,7 @@ fn rackscale_memcached_benchmark(transport: RackscaleTransport) {
624624
test.run_dhcpd_for_baseline = true;
625625

626626
if !is_smoke {
627-
test.shmem_size = std::cmp::min(
627+
test.shmem_size = std::cmp::max(
628628
MEMCACHED_MEM_SIZE_MB * 8,
629629
testutils::helpers::SHMEM_SIZE * 4,
630630
);
@@ -653,7 +653,7 @@ fn rackscale_memcached_benchmark(transport: RackscaleTransport) {
653653
// Memory must also be divisible by number of nodes, which could be 1, 2, 3, or 4
654654
// memory = result of this function / num_clients - shmem_size
655655
(8192
656-
+ std::cmp::min(
656+
+ std::cmp::max(
657657
MEMCACHED_MEM_SIZE_MB * 8,
658658
testutils::helpers::SHMEM_SIZE * 4,
659659
))
@@ -1305,7 +1305,7 @@ fn s11_rackscale_memcached_benchmark_sharded_nros() {
13051305
test.is_multi_node = true;
13061306

13071307
if !is_smoke {
1308-
test.shmem_size = std::cmp::min(
1308+
test.shmem_size = std::cmp::max(
13091309
MEMCACHED_MEM_SIZE_MB * 8,
13101310
testutils::helpers::SHMEM_SIZE * 4,
13111311
);
@@ -1333,7 +1333,7 @@ fn s11_rackscale_memcached_benchmark_sharded_nros() {
13331333
} else {
13341334
// Memory must also be divisible by number of nodes, which could be 1, 2, 3, or 4
13351335
(8192
1336-
+ std::cmp::min(
1336+
+ std::cmp::max(
13371337
MEMCACHED_MEM_SIZE_MB * 8,
13381338
testutils::helpers::SHMEM_SIZE * 4,
13391339
))

0 commit comments

Comments
 (0)