-
Notifications
You must be signed in to change notification settings - Fork 885
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
NetBSD: Runtime error using Open MPI 2.1.0rc2 #3117
Comments
Here is the error report:
@artpol84 Can you please take a look? |
The only place that can cause this error is @PHHargrove can you check |
Thanks, Artem. I would suggest that in the future a better error message would be helpful, but is not critical. |
I spoke too soon. |
I see a part of the problem: posix_fallocate does not set errno on failure. Here is a relevant portion of the posix_fallocate manpage on NetBSD:
And Linux says much the same:
|
But still it fails for some reason. |
I am working on it, but I am traveling this week (sitting in the middle of a large meeting). |
The return value is Perhaps the configure test for |
To completely discharge my role as the "portability police", I want to note when
The Mac OS X, FreeBSD and OpenBSD manpages say something similar in their "BUGS" section:
The Linux manpage provides the most useful information on the subject:
So, note that "returning an error" is a POSX-compliant behavior when However, in reality this is likely not an issue for 2 reasons.
|
So it sounds like we need to do one or two things, and perhaps a third:
|
@rhc54 I agree on all three points above, including placing low priority on allowing for failure of ftruncate(). I agree that with the proper checks item 2 alone should be sufficient. There is a 50%+ chance that a tested-and-signed-off patch will appear here in the next two hour that implements item 2. |
Proposed patch which has been tested (via ring_c) on the NetBSD/amd64 system where the problem was first observed. |
thanks Paul, can you create a PR against PMIx master or you want me to do that? |
@artpol84 I would really appreciate it if you could handle the PR. |
Thanks, Paul. We will take care. |
@karasevb please create PR |
@jsquyres I vote for moving this to 2.1.1 |
I only test on the effect system(s) - I don't use them. |
@hppritcha I just moved the milestone, but then I looked closer / remembered what this one was -- wasn't it fixed in #3130 ? |
yes, it was, so closing this one |
Thanks @artpol84 this saved me a lot of time and frustration. Redefined TMPDIR and it works fine. Cheers |
@PHHargrove reports hitting a runtime error (using ring_c).c) on NetBSD when using the Open MPI 2.1.0rc2 tarball. Error is in PMIX.
See https://www.mail-archive.com/[email protected]//msg19987.html
The text was updated successfully, but these errors were encountered: