-
Notifications
You must be signed in to change notification settings - Fork 878
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
Wrong parsing of command line arguments with mpirun #6372
Comments
Hah, that's a fun bug! Yes, it looks like something in the command line parsing is reading
A similar error happens on master HEAD, so this is likely behavior that has escaped attention for quite a while. |
Dear everyone, |
I can take a look and see what's going on. |
FWIW: this is fixed in the RTE for OMPI v5. Problem is in not processing the cmd line before searching for mca params (including --am), thereby not knowing where the executable starts. |
I confirm: this is fixed on Open MPI master / what will eventually become v5.0. I'm kinda guessing that this is not going to be fixed in prior releases, mainly due to lack of resources (i.e., no one has submitted a fix in 20+ months). |
The bug sounds similar to this one open-mpi/ompi#6372, though that is supposedly fixed in Open MPI 5.. This creates a tmp script to call the launched application instead of calling it directly on the command line. This way, mpirun does not see the command line arguments and cannot try to wrongly parse them.
* Use mpirun wrapper to circumvent parser bug in mpirun The bug sounds similar to this one open-mpi/ompi#6372, though that is supposedly fixed in Open MPI 5.. This creates a tmp script to call the launched application instead of calling it directly on the command line. This way, mpirun does not see the command line arguments and cannot try to wrongly parse them. * Add mechanism to get rid of the workaround again in future
For those that have also scrambled across this like myself, it appears this is still not fully resolved:
In the second case, |
It is (and has been for some time) fixed on the main branch, but there is a missing cherry-pick into the release branch. Have that in-process now. |
Great. Thanks for addressing this quickly! |
I also just encountered this. |
Ok....but as stated, it has been fixed and just awaits another release. |
To be clear - the fix that @rhc54 was referencing did work for me. Just hasn't made it into a release. |
In 3.1.1 and 3.1.3 I get the following warning starting MPI applications that take the
--am
argument:Of course
ls
doesn't take this argument, but that is only to illustrate the problem. It seems thatmpirun
is trying to analyze the application's arguments, and beyond the application name. Note that the above message is only displayed when the-H
switch is present, and when there are at least 2 arguments after--am
. Pretty funny, and only annoying: I do not see any other side effects except for the noise.The text was updated successfully, but these errors were encountered: