Skip to content

Commit cdbf8c6

Browse files
committed
partitions: make it work with sessions
even when configuring with --enable-mca-dso. The partition framework open/close was not being properly managed prior to this patch, leading to segfaults upon reinitialization if the app starts another session after closing a previous one and the Open MPI was configured with --enable-mca-dso. Related to #12887 Signed-off-by: Howard Pritchard <howardp@lanl.gov>
1 parent c393881 commit cdbf8c6

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

ompi/instance/instance.c

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ static mca_base_framework_t *ompi_framework_dependencies[] = {
115115
&ompi_hook_base_framework, &ompi_op_base_framework,
116116
&opal_allocator_base_framework, &opal_rcache_base_framework, &opal_mpool_base_framework, &opal_smsc_base_framework,
117117
&ompi_bml_base_framework, &ompi_pml_base_framework, &ompi_coll_base_framework,
118-
&ompi_osc_base_framework, NULL,
118+
&ompi_osc_base_framework, &ompi_part_base_framework, NULL,
119119
};
120120

121121
static mca_base_framework_t *ompi_lazy_frameworks[] = {
@@ -657,11 +657,7 @@ static int ompi_mpi_instance_init_common (int argc, char **argv)
657657
return ompi_instance_print_error ("ompi_win_init() failed", ret);
658658
}
659659

660-
/* initialize partcomm */
661-
if (OMPI_SUCCESS != (ret = mca_base_framework_open(&ompi_part_base_framework, 0))) {
662-
return ompi_instance_print_error ("mca_part_base_select() failed", ret);
663-
}
664-
660+
/* select part component to use */
665661
if (OMPI_SUCCESS != (ret = mca_part_base_select (true, true))) {
666662
return ompi_instance_print_error ("mca_part_base_select() failed", ret);
667663
}

0 commit comments

Comments
 (0)