Skip to content

Commit eeb91bc

Browse files
committed
pmix/s2: fix srun native launch for pmi2
recent changes that broke native launch on cray using srun or aprun was also broke native launch using pmi2. This commit fixes this problem. Signed-off-by: Howard Pritchard <[email protected]>
1 parent bf1c863 commit eeb91bc

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

opal/mca/pmix/s2/pmix_s2.c

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* Copyright (c) 2007 The Trustees of Indiana University.
44
* All rights reserved.
55
* Copyright (c) 2011-2016 Cisco Systems, Inc. All rights reserved.
6-
* Copyright (c) 2011-2013 Los Alamos National Security, LLC. All
6+
* Copyright (c) 2011-2017 Los Alamos National Security, LLC. All
77
* rights reserved.
88
* Copyright (c) 2013-2017 Intel, Inc. All rights reserved.
99
* Copyright (c) 2014-2016 Research Organization for Information Science
@@ -175,6 +175,7 @@ static int s2_init(opal_list_t *ilist)
175175
opal_process_name_t wildcard_rank;
176176

177177
if (0 < pmix_init_count) {
178+
++pmix_init_count;
178179
return OPAL_SUCCESS;
179180
}
180181

@@ -425,13 +426,13 @@ static int s2_fini(void) {
425426

426427
if (0 == --pmix_init_count) {
427428
PMI2_Finalize();
428-
}
429-
if (NULL != pmix_kvs_name) {
430-
free(pmix_kvs_name);
431-
pmix_kvs_name = NULL;
432-
}
433-
if (NULL != s2_lranks) {
434-
free(s2_lranks);
429+
if (NULL != pmix_kvs_name) {
430+
free(pmix_kvs_name);
431+
pmix_kvs_name = NULL;
432+
}
433+
if (NULL != s2_lranks) {
434+
free(s2_lranks);
435+
}
435436
}
436437
return OPAL_SUCCESS;
437438
}

0 commit comments

Comments
 (0)