Add platform specific dynamic library loading #119
Conversation
include/fusilli/support/dllib.h
Outdated
| // Use dlmopen with LM_ID_NEWLM to load in a new namespace for isolation. | ||
| handle_ = dlmopen(LM_ID_NEWLM, path.c_str(), RTLD_LAZY | RTLD_LOCAL); |
There was a problem hiding this comment.
Was it intentional to not check if the library is already loaded and reuse if so? The previous code was doing that (RTLD_NOLOAD) but the new code always creates a new namespace via LM_ID_NEWLM. Just checking because this could lead to duplicate copies of the library in different namespaces.
There was a problem hiding this comment.
So the idea I had was to support reloading the same library multiple times. It avoids having to manage separately loaded version (e.g. something load the libIreeCompiler.so for its own implementation now having duplicate ownership). One of the problems with core dlopen is that if another took uses the library it could get into an illegal state (because of self counting destroy).
There was a problem hiding this comment.
Got it. Maybe elaborate the reasoning in the comments for reference.
rsuderman
force-pushed
the
dylib
branch
2 times, most recently
from
9d34c1f to
bc9b249
Compare
sjain-stanford
left a comment
sjain-stanford
left a comment
There was a problem hiding this comment.
Nice restructuring to make the library interface platform agnostic. LGTM modulo few comments.
include/fusilli/support/dllib.h
Outdated
| // Use dlmopen with LM_ID_NEWLM to load in a new namespace for isolation. | ||
| handle_ = dlmopen(LM_ID_NEWLM, path.c_str(), RTLD_LAZY | RTLD_LOCAL); |
There was a problem hiding this comment.
Got it. Maybe elaborate the reasoning in the comments for reference.
Signed-off-by: Rob Suderman <rob.suderman@gmail.com>
Signed-off-by: Rob Suderman <rob.suderman@gmail.com>
Signed-off-by: Rob Suderman <rob.suderman@gmail.com>
Signed-off-by: Rob Suderman <rob.suderman@gmail.com>
2 participants
Windows differs from Linux in how it loads from dynamic libraries. Adding a support shim to select between platforms allows the libraries to be loaded platform independently.