-
Notifications
You must be signed in to change notification settings - Fork 5
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
lc_finalize in v2 #9
Comments
Right would be good to do it, i took it as low priority since the job is
going away anyway so process manager will clean them up(hopefully)
You can add a pmi_barrier, to wait for all to arrive before exit.
…On Mon, Mar 2, 2020 at 2:03 PM Omri Mor ***@***.***> wrote:
lc_finalize in the v2 branch is currently a no-op. I'm running into
issues where one process exits before others and so I get a SIGPIPE when
others try to write to stdout or stderr (I think PMI is forwarding this
over a pipe to rank 0 or something).
Also, we should probably just generally clean up when finalizing?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#9>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIZNSWVSVXUBTLQT6LWIZLRFQUMVANCNFSM4K75R2IA>
.
|
LCI (currently) uses the PMI implementation in simple_pmi.c, not the PMIx library, right? |
I'll note that the workaround works though, for obvious reasons. That was the last issue I've seen with PaRSEC/LCI, so it's fully working (on psm2)—so I can now move on to performance testing. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
lc_finalize
in the v2 branch is currently a no-op. I'm running into issues where one process exits before others and so I get a SIGPIPE when others try to write to stdout or stderr (I think PMI is forwarding this over a pipe to rank 0 or something).Also, we should probably just generally clean up when finalizing?
The text was updated successfully, but these errors were encountered: