-
Notifications
You must be signed in to change notification settings - Fork 21
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
Intel OpenMP breaks Gotcha #70
Comments
@DavidPoliakoff -- I just went back to this, and failed to reproduce it. Do you still have a reproducer for this issue? |
We are attempting to use gotcha for our memory wrappers and have encountered a problem that shows up with Intel 16.0 compilers but does not show up with Intel 20 compilers. With the older Intel compiled application dgemm, we see an abort in gotcha:
With the same tool and an Intel 20 compiled dgemm there is no abort and we get the expected results. While we were debugging the issue we also thought it was related to the OpenMP runtime. |
@mplegendre Our abort (above) only occurs when the MKL libraries are included in the executable. We are still trying to use gotcha but with older Intel compilers it aborts. Do you have any suggestions on how to debug? Are there flags to turn on to help with debugging?
Looks like the pointer gotcha is going to free points to 0. Thanks |
During the release process we discussed this issue with @daboehme, it was first reported in #40.
The problem is that Intel's runtimes use many mechanisms that break Gotcha, dlsym with RTLD_NEXT, and even (I believe) GOT rewriting. Fixing this could take multiple steps
Step 1) Function better with dlsym and RTLD_NEXT
Step 2) (If the problem still exists) Use some awareness when Intel OpenMP libraries are present. Perhaps, if Intel OpenMP is there, we wait until we see them do a rewrite before rewriting GOTs.
Closer to the time it comes to fix this I might ping some people at Intel, this seems like the kind of bug which we could iteratively cause for one another as we update code, I'd like to avoid that.
The text was updated successfully, but these errors were encountered: