Skip to content

Commit

Permalink
wreck/openmpi.lua: drop LD_LIBRARY_PATH, add TMPDIR
Browse files Browse the repository at this point in the history
The new openmpi Flux component uses FLUX_PMI_LIBRARY_PATH
at runtime to dlopen our PMI library, so LD_LIBRARY_PATH
does not need to be set for it to work right in the
default configuration.  Therefore, don't change it
in the openmpi lua script.

Do set TMPDIR to be the rank-specific directory so that
shared memory segment names do not collide when
there are multiple Flux ranks on a (real) node launching
an MPI program with multiple MPI ranks on a (flux) node.
  • Loading branch information
garlick committed Dec 15, 2016
1 parent d4430bd commit 03dbebe
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions src/modules/wreck/lua.d/openmpi.lua
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,9 @@ local dirname = require 'flux.posix'.dirname
function rexecd_init ()
local env = wreck.environ
local f = wreck.flux
local libpmi = f:getattr ('conf.pmi_library_path')
local ldpath = dirname (libpmi)
local rankdir = f:getattr ('scratch-directory-rank')

if (env['LD_LIBRARY_PATH'] ~= nil) then
ldpath = ldpath..':'..env['LD_LIBRARY_PATH']
end
env['LD_LIBRARY_PATH'] = ldpath
env['TMPDIR'] = rankdir
end

-- vi: ts=4 sw=4 expandtab

0 comments on commit 03dbebe

Please sign in to comment.