Fortram mpi wrappers have wrong parameter types

[wangvsa]

Description

There's a strange error when running a FLASH5 simulation on Summit:

*** The MPI_Init_thread() function was called before MPI_INIT was invoked.
*** This is disallowed by the MPI standard.
*** Your MPI job will now abort.

The actual issue is not calling MPI_Init_thread before MPI_Init, but caused by the invalid required parameter value.
(strange how this irrelevant error message showed up).

Fix

The previous MPI_Init_thread wrapper has a wrong parameter type for required. Instead of int required, it should be MPI_Fint* required.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Performance enhancement (non-breaking change which improves efficiency)
• Code cleanup (non-breaking change which makes code smaller or more readable)
• Breaking change (fix or feature that would cause existing functionality to change)
• Testing (addition of new tests or update to current tests)
• Documentation (a change to man pages or other documentation)

[MichaelBrim]

Thanks Chen, you beat me to the PR. I had the same fix in one of my branches for Seung-Hwan.

[adammoody]

Thanks @wangvsa and @MichaelBrim !