Use LargeCount MPI_write_at and MPI_read_at functions #13851
Conversation
| sizes_fixed_cumulative.size(), | ||
| MPI_UNSIGNED, | ||
| MPI_STATUS_IGNORE); | ||
| ierr = Utilities::MPI::LargeCount::MPI_File_write_at_c( |
There was a problem hiding this comment.
This is not strictly necessary, as the header will not exceed 2GB, but this is what @bangerth suggested in his review of the other PR. So let's do the change here as well.
tjhei
changed the title
forgot to include the big mpi file removing empty line
pengfej
force-pushed
the
updating_read_write_at_functions
branch
from
4a7d7b7
to
7ade5e9
Compare
|
Let me run the large tests to be sure... |
Sounds good! Thanks a lot! |
| sizes_fixed_cumulative.size(), | ||
| MPI_UNSIGNED, | ||
| MPI_STATUS_IGNORE); | ||
| ierr = Utilities::MPI::LargeCount::MPI_File_write_at_c( |
|
Thanks, @tjhei ! This is the kind of stuff that is often difficult to test because we don't check these things easily within a single program. Out of curiosity, do you know whether these calls are checked in any of our existing tests? |
The functions Utilities::MPI::LargeCount::* are tested to work with various MPI libraries here: https://github.com/tjhei/BigMPICompat#test-results |
…unctions Use LargeCount MPI_write_at and MPI_read_at functions
No description provided.