Skip to content

Commit 3440b46

Browse files
author
Ralph Castain
authored
Merge pull request #2820 from rhc54/topic/async
Per f2f meeting: if async modex is given, default to no MPI init barr…
2 parents f4a8690 + a7b8190 commit 3440b46

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

ompi/runtime/ompi_mpi_init.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
* Copyright (c) 2008-2009 Sun Microsystems, Inc. All rights reserved.
1818
* Copyright (c) 2011 Sandia National Laboratories. All rights reserved.
1919
* Copyright (c) 2012-2013 Inria. All rights reserved.
20-
* Copyright (c) 2014-2016 Intel, Inc. All rights reserved.
20+
* Copyright (c) 2014-2017 Intel, Inc. All rights reserved.
2121
* Copyright (c) 2014-2016 Research Organization for Information Science
2222
* and Technology (RIST). All rights reserved.
2323
* Copyright (c) 2016 Mellanox Technologies Ltd. All rights reserved.
@@ -640,10 +640,10 @@ int ompi_mpi_init(int argc, char **argv, int requested, int *provided)
640640
* if data exchange is required. The modex occurs solely across procs
641641
* in our job. If a barrier is required, the "modex" function will
642642
* perform it internally */
643-
active = true;
644643
opal_pmix.commit();
645644
if (!opal_pmix_base_async_modex) {
646645
if (NULL != opal_pmix.fence_nb) {
646+
active = true;
647647
opal_pmix.fence_nb(NULL, opal_pmix_collect_all_data,
648648
fence_release, (void*)&active);
649649
OMPI_LAZY_WAIT_FOR_COMPLETION(active);

ompi/runtime/ompi_mpi_params.c

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
* Copyright (c) 2007-2015 Los Alamos National Security, LLC. All rights
1515
* reserved.
1616
* Copyright (c) 2013 NVIDIA Corporation. All rights reserved.
17-
* Copyright (c) 2013-2016 Intel, Inc. All rights reserved
17+
* Copyright (c) 2013-2017 Intel, Inc. All rights reserved.
1818
* Copyright (c) 2015 Mellanox Technologies, Inc.
1919
* All rights reserved.
2020
* Copyright (c) 2016 Research Organization for Information Science
@@ -37,6 +37,7 @@
3737
#include "ompi/runtime/params.h"
3838
#include "ompi/mca/rte/rte.h"
3939

40+
#include "opal/mca/pmix/base/base.h"
4041
#include "opal/util/argv.h"
4142
#include "opal/util/output.h"
4243
#include "opal/util/show_help.h"
@@ -286,7 +287,11 @@ int ompi_mpi_register_params(void)
286287
MCA_BASE_VAR_SCOPE_READONLY,
287288
&ompi_mpi_dynamics_enabled);
288289

289-
ompi_async_mpi_init = false;
290+
if (opal_pmix_base_async_modex) {
291+
ompi_async_mpi_init = true;
292+
} else {
293+
ompi_async_mpi_init = false;
294+
}
290295
(void) mca_base_var_register("ompi", "async", "mpi", "init",
291296
"Do not perform a barrier at the end of MPI_Init",
292297
MCA_BASE_VAR_TYPE_BOOL, NULL, 0, 0,

0 commit comments

Comments
 (0)