Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace use of libdl and LD_PRELOAD with GOTCHA #43

Open
ilumsden opened this issue Sep 7, 2023 · 2 comments
Open

Replace use of libdl and LD_PRELOAD with GOTCHA #43

ilumsden opened this issue Sep 7, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@ilumsden
Copy link
Collaborator

ilumsden commented Sep 7, 2023

In the wrapper interface, we should move away from the super error-prone LD_PRELOAD approach towards something that's more powerful and less likely to cause odd issues. One option is LLNL's GOTCHA tool: https://github.com/LLNL/GOTCHA

@JaeseungYeom
Copy link
Contributor

We can discuss this. However, I am wondering how transparent the interface will be.

@ilumsden
Copy link
Collaborator Author

@JaeseungYeom Gotcha acts like a more fully featured, drop in replacement for libdl. As a result, no changes would appear in user code due to Gotcha. The only potential difference would come from how users "activate" the interception. Depending on how we add Gotcha support, users would either continue to use LD_PRELOAD with dyad_wrapper.so, or they would just link dyad_wrapper.so into their code. All the actual symbol interception still happens fully transparently with no required changes to user code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants