Skip to content

Commit 3fa7aab

Browse files
committed
fix srun latency, change default yield_when_idle=0
This changes the default to 0, to avoid yields during progress in srun. In mpirun, ompi_mpi_yield_when_idle is set to 1 if oversubscribed otherwise 0. But the default is 1 though, and it is used in srun. Now srun and mpirun have the same latency in non-oversubscribed cases. Signed-off-by: Piotr Lesnicki <[email protected]>
1 parent d954167 commit 3fa7aab

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

ompi/runtime/ompi_mpi_params.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ bool ompi_mpi_keep_fqdn_hostnames = false;
6262
bool ompi_have_sparse_group_storage = OPAL_INT_TO_BOOL(OMPI_GROUP_SPARSE);
6363
bool ompi_use_sparse_group_storage = OPAL_INT_TO_BOOL(OMPI_GROUP_SPARSE);
6464

65-
bool ompi_mpi_yield_when_idle = true;
65+
bool ompi_mpi_yield_when_idle = false;
6666
int ompi_mpi_event_tick_rate = -1;
6767
char *ompi_mpi_show_mca_params_string = NULL;
6868
bool ompi_mpi_have_sparse_group_storage = !!(OMPI_GROUP_SPARSE);
@@ -107,7 +107,7 @@ int ompi_mpi_register_params(void)
107107
*/
108108
/* JMS: Need ORTE data here -- set this to 0 when
109109
exactly/under-subscribed, or 1 when oversubscribed */
110-
ompi_mpi_yield_when_idle = true;
110+
ompi_mpi_yield_when_idle = false;
111111
(void) mca_base_var_register("ompi", "mpi", NULL, "yield_when_idle",
112112
"Yield the processor when waiting for MPI communication (for MPI processes, will default to 1 when oversubscribing nodes)",
113113
MCA_BASE_VAR_TYPE_BOOL, NULL, 0, 0,

0 commit comments

Comments
 (0)