Skip to content

Commit

Permalink
Make more explicit how we only alter test.default_num_cpus_per_core o…
Browse files Browse the repository at this point in the history
…n hyperthreading-enabled systems
  • Loading branch information
Caspar van Leeuwen committed Jun 28, 2024
1 parent eba036e commit 09b4bc5
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion eessi/testsuite/hooks.py
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,9 @@ def assign_tasks_per_compute_unit(test: rfm.RegressionTest, compute_unit: str, n
if SCALES[test.scale].get('num_cpus_per_node') is not None and compute_unit != COMPUTE_UNIT[HWTHREAD]:
check_proc_attribute_defined(test, 'num_cpus_per_core')
num_cpus_per_core = test.current_partition.processor.num_cpus_per_core
test.default_num_cpus_per_node = test.default_num_cpus_per_node * num_cpus_per_core
# On a hyperthreading system?
if num_cpus_per_core > 1:
test.default_num_cpus_per_node = test.default_num_cpus_per_node * num_cpus_per_core

if FEATURES[GPU] in test.current_partition.features:
_assign_default_num_gpus_per_node(test)
Expand Down

0 comments on commit 09b4bc5

Please sign in to comment.