Skip to content
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

Need to fix Fortran "use mpi" module array declarations in APIs such as MPI_WAITALL #9484

Closed
jsquyres opened this issue Oct 7, 2021 · 3 comments

Comments

@jsquyres
Copy link
Member

jsquyres commented Oct 7, 2021

It was recently noticed that some of the array declarations in the ignore TKR mpi module have specifically-sized arrays, but the MPI standard says that the arrays should be (*). The easiest API to note is MPI_WAITALL.

All the other Fortran APIs with array dummy parameters should also be checked; I don't know how pervasive the problem is. This is probably not nearly as daunting a job as it sounds -- just search ompi/mpi/fortran/use-mpi-ignore-tkr/*.h for dummy parameters that are arrays that are not (*) and double-check them against the MPI standard.

The TKR mpi module should also be checked.


History:

@markalle
Copy link
Contributor

In this PR I just looked at the various "dimension ... array_of_..." lines:
#9566

@markalle
Copy link
Contributor

markalle commented Nov 8, 2021

(is in master, and I just rebased the v5.0.x cherry-pick):
#9608

@jsquyres
Copy link
Member Author

jsquyres commented Nov 8, 2021

We fixed this on master and v5.0.x.

Although there are no ABI issues associated with this change, and it's probably low risk to take these changes back to v4.0.x and v4.1.x, we made the executive decision to not take them back to these old branches just because there doesn't seem to be much benefit (no one has complained about this for years).

So we're leaving this note for our future selves: if we need to, we can back-port this to the v4.x branches. But unless we have to, let's not.

@jsquyres jsquyres closed this as completed Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants