Skip to content

Commit a091da7

Browse files
committed
cm: fix selection priority
This patch removes a priority check that disables cm if the previous pml had higher priority. The check was incorrect as coded and is unnecessary as we finalize all but one pml anyway. Fixes open-mpi#1035 Signed-off-by: Nathan Hjelm <[email protected]> (cherry picked from open-mpi/ompi@2fd176a)
1 parent ada7bed commit a091da7

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

ompi/mca/pml/cm/pml_cm_component.c

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -143,11 +143,6 @@ mca_pml_cm_component_init(int* priority,
143143
{
144144
int ret;
145145

146-
if((*priority) > ompi_pml_cm.default_priority) {
147-
*priority = ompi_pml_cm.default_priority;
148-
return NULL;
149-
}
150-
*priority = ompi_pml_cm.default_priority;
151146
opal_output_verbose( 10, 0,
152147
"in cm pml priority is %d\n", *priority);
153148
/* find a useable MTL */
@@ -166,9 +161,10 @@ mca_pml_cm_component_init(int* priority,
166161
* may still select PML/ob1 (BTLs) or PML/cm (MTLs) if preferable for the app/site.
167162
*/
168163
*priority = 30;
164+
} else {
165+
*priority = ompi_pml_cm.default_priority;
169166
}
170167

171-
172168
if (ompi_mtl->mtl_flags & MCA_MTL_BASE_FLAG_REQUIRE_WORLD) {
173169
ompi_pml_cm.super.pml_flags |= MCA_PML_BASE_FLAG_REQUIRE_WORLD;
174170
}

0 commit comments

Comments
 (0)