From c2581d0e335ef03e6c0b7715d66a43ea4f7ce499 Mon Sep 17 00:00:00 2001 From: Joshua Hursey Date: Mon, 18 Mar 2019 21:50:21 -0500 Subject: [PATCH] Do not force 'hash' gds on direct modex * Forcing the 'hash' gds component should not be necessary any more. Signed-off-by: Joshua Hursey --- opal/mca/pmix/ext3x/ext3x_client.c | 6 +----- opal/mca/pmix/ext3x/ext3x_server_south.c | 6 +----- 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/opal/mca/pmix/ext3x/ext3x_client.c b/opal/mca/pmix/ext3x/ext3x_client.c index 5a87ab5f1b..617f20936b 100644 --- a/opal/mca/pmix/ext3x/ext3x_client.c +++ b/opal/mca/pmix/ext3x/ext3x_client.c @@ -11,6 +11,7 @@ * Copyright (c) 2018 The University of Tennessee and The University * of Tennessee Research Foundation. All rights * reserved. + * Copyright (c) 2019 IBM Corporation. All rights reserved. * $COPYRIGHT$ * * Additional copyrights may follow @@ -102,11 +103,6 @@ int ext3x_client_init(opal_list_t *ilist) ninfo = 0; } - /* check for direct modex use-case */ - if (opal_pmix_base_async_modex && !opal_pmix_collect_all_data) { - opal_setenv("PMIX_MCA_gds", "hash", true, &environ); - } - OPAL_PMIX_RELEASE_THREAD(&opal_pmix_base.lock); rc = PMIx_Init(&mca_pmix_ext3x_component.myproc, pinfo, ninfo); if (NULL != pinfo) { diff --git a/opal/mca/pmix/ext3x/ext3x_server_south.c b/opal/mca/pmix/ext3x/ext3x_server_south.c index 440dc8fa9e..2991aca6c2 100644 --- a/opal/mca/pmix/ext3x/ext3x_server_south.c +++ b/opal/mca/pmix/ext3x/ext3x_server_south.c @@ -12,6 +12,7 @@ * Copyright (c) 2018 The University of Tennessee and The University * of Tennessee Research Foundation. All rights * reserved. + * Copyright (c) 2019 IBM Corporation. All rights reserved. * $COPYRIGHT$ * * Additional copyrights may follow @@ -133,11 +134,6 @@ int ext3x_server_init(opal_pmix_server_module_t *module, } } - /* check for direct modex use-case */ - if (opal_pmix_base_async_modex && !opal_pmix_collect_all_data) { - opal_setenv("PMIX_MCA_gds", "hash", true, &environ); - } - /* insert ourselves into our list of jobids - it will be the * first, and so we'll check it first */ job = OBJ_NEW(opal_ext3x_jobid_trkr_t);