-
Notifications
You must be signed in to change notification settings - Fork 934
Enable the PMIx notification callback system and fix debugger attach #1358
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
Conversation
|
@jsquyres @ggouaillardet @hjelmn @hppritcha Can someone please verify that this solves the debugger attach at launch problem? |
|
@rhc54 Did this not work? |
|
@jsquyres I think I accidentally deleted the branch and then immediately restored it. It should work, but I was asking that you test it with a debugger |
|
I actually don't have Totalview. :-( I can confirm that this still works for DDT, though. @hppritcha Do you have Totalview? Or should we ask our friends at TV to test for us? |
|
I will give this a try on one of the LANL clusters today.
|
|
I did try to run with totalview yesterday - and thought I'd posted here. I hit a bunch of issues almost certainly not related to this PR. I'd say go ahead and check this in (after rebase). @hjelmn and I will check to see whether we have a site specific issue with our totalview install. |
…supported by the pmix120 component, which is not selected by default. All other components will ignore error registration requests, and thus do not support debugger attach when launched via mpirun. Note that direct launched applications will support such attachment, but may not do so in a scalable fashion. Fixes ##1225
Enable the PMIx notification callback system and fix debugger attach
|
Nathan and I worked more on this today. Things look good with ddt. unfortunately, with totalveiw things still don't seem to work. We get this error when trying to use the mpirun --debug option: |
|
Asked @rhc54 about the message and he indicated it is a stale debug message. Should be cleaned out. |
…_proc_t v2.x: oshmem: get rid of oshmem_proc_t and use ompi_proc_t instead
This currently is only supported by the pmix120 component, which is not selected by default. All other components will ignore error registration requests, and thus do not support debugger attach when launched via mpirun. Note that direct launched applications will support such attachment, but may not do so in a scalable fashion.
Fixes ##1225