Skip to content

Commit df19408

Browse files
committed
Merge pull request #1591 from hjelmn/rcache_update
rcache: fix leave_pinned failure path
2 parents f0f3383 + 25a97af commit df19408

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

opal/mca/rcache/base/help-rcache-base.txt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,17 +22,17 @@ Open MPI will disable any transports that are attempting to use the
2222
leave pinned functionality; your job may still run, but may fall back
2323
to a slower network transport (such as TCP).
2424

25-
Mpool name: %s
26-
Process: %s
27-
Local host: %s
25+
rcache name: %s
26+
Process: %s
27+
Local host: %s
2828
#
2929
[cannot deregister in-use memory]
3030
Open MPI intercepted a call to free memory that is still being used by
3131
an ongoing MPI communication. This usually reflects an error in the
3232
MPI application; it may signify memory corruption. Open MPI will now
3333
abort your job.
3434

35-
Mpool name: %s
35+
rcache name: %s
3636
Local host: %s
3737
Buffer address: %p
3838
Buffer size: %lu

opal/mca/rcache/base/rcache_base_create.c

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
* University of Stuttgart. All rights reserved.
1111
* Copyright (c) 2004-2005 The Regents of the University of California.
1212
* All rights reserved.
13-
* Copyright (c) 2015 Los Alamos National Security, LLC. All rights
13+
* Copyright (c) 2015-2016 Los Alamos National Security, LLC. All rights
1414
* reserved.
1515
* $COPYRIGHT$
1616
*
@@ -59,17 +59,15 @@ mca_rcache_base_module_t* mca_rcache_base_module_create (const char* name, void
5959
actually). This is a hook available for memory manager hooks
6060
without good initialization routine support */
6161
(void) mca_base_framework_open (&opal_memory_base_framework, 0);
62-
}
6362

64-
if (opal_leave_pinned != 0 || opal_leave_pinned_pipeline) {
6563
if ((OPAL_MEMORY_FREE_SUPPORT | OPAL_MEMORY_MUNMAP_SUPPORT) ==
6664
((OPAL_MEMORY_FREE_SUPPORT | OPAL_MEMORY_MUNMAP_SUPPORT) &
6765
opal_mem_hooks_support_level())) {
6866
if (-1 == opal_leave_pinned) {
6967
opal_leave_pinned = !opal_leave_pinned_pipeline;
7068
}
7169
opal_mem_hooks_register_release(mca_rcache_base_mem_cb, NULL);
72-
} else {
70+
} else if (1 == opal_leave_pinned || opal_leave_pinned_pipeline) {
7371
opal_show_help("help-rcache-base.txt", "leave pinned failed",
7472
true, name, OPAL_NAME_PRINT(OPAL_PROC_MY_NAME),
7573
opal_proc_local_get()->proc_hostname);

0 commit comments

Comments
 (0)