Skip to content

Commit 04f9d39

Browse files
committed
ompi: ompi_mpi_init(): do not export threading level to modex.
For some of our configuration this flag increases per-process contribution by ~20% while it is not being used currently. The consumer of this flag was communicator ID calculation logic, but it was changed in 0bf06de. Signed-off-by: Artem Polyakov <[email protected]> (cherry picked from commit b601dd5)
1 parent a891a76 commit 04f9d39

File tree

1 file changed

+0
-16
lines changed

1 file changed

+0
-16
lines changed

ompi/runtime/ompi_mpi_init.c

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -383,10 +383,6 @@ int ompi_mpi_init(int argc, char **argv, int requested, int *provided)
383383

384384
OMPI_TIMING_INIT(32);
385385

386-
/* bitflag of the thread level support provided. To be used
387-
* for the modex in order to work in heterogeneous environments. */
388-
uint8_t threadlevel_bf;
389-
390386
ompi_hook_base_mpi_init_top(argc, argv, requested, provided);
391387

392388
/* Ensure that we were not already initialized or finalized.
@@ -551,18 +547,6 @@ int ompi_mpi_init(int argc, char **argv, int requested, int *provided)
551547
goto error;
552548
}
553549

554-
/* determine the bitflag belonging to the threadlevel_support provided */
555-
memset ( &threadlevel_bf, 0, sizeof(uint8_t));
556-
OMPI_THREADLEVEL_SET_BITFLAG ( ompi_mpi_thread_provided, threadlevel_bf );
557-
558-
/* add this bitflag to the modex */
559-
OPAL_MODEX_SEND_STRING(ret, OPAL_PMIX_GLOBAL,
560-
"MPI_THREAD_LEVEL", &threadlevel_bf, sizeof(uint8_t));
561-
if (OPAL_SUCCESS != ret) {
562-
error = "ompi_mpi_init: modex send thread level";
563-
goto error;
564-
}
565-
566550
/* initialize datatypes. This step should be done early as it will
567551
* create the local convertor and local arch used in the proc
568552
* init.

0 commit comments

Comments
 (0)