Skip to content

Commit dce203f

Browse files
authored
Merge pull request #7057 from edgargabriel/topic/romio321-status-set-elements-fix
MPIR_Status_set_bytes: fix for large counts
2 parents b1ef5a4 + 8a3abbf commit dce203f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

ompi/mca/io/romio321/src/io_romio321_module.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
* because ROMIO just expects these functions to exist.
3434
*/
3535
int MPIR_Status_set_bytes(ompi_status_public_t *status,
36-
struct ompi_datatype_t *datatype, int size);
36+
struct ompi_datatype_t *datatype, MPI_Count size);
3737
void ADIOI_Datatype_iscontig(MPI_Datatype datatype, int *flag);
3838

3939

@@ -117,7 +117,7 @@ mca_io_base_module_2_0_0_t mca_io_romio321_module = {
117117
* MPI_Status_set_elements (almost like they planned that... hmmm...).
118118
*/
119119
int MPIR_Status_set_bytes(ompi_status_public_t *status,
120-
struct ompi_datatype_t *datatype, int nbytes)
120+
struct ompi_datatype_t *datatype, MPI_Count nbytes)
121121
{
122122
/* Note that ROMIO is going to give a number of *bytes* here, but
123123
MPI_STATUS_SET_ELEMENTS requires a number of *elements*. So
@@ -127,7 +127,7 @@ int MPIR_Status_set_bytes(ompi_status_public_t *status,
127127
we do the right calculations there. This prevents roundoff
128128
errors here, potentially "losing" bytes in the process. */
129129

130-
MPI_Status_set_elements(status, MPI_CHAR, nbytes);
130+
MPI_Status_set_elements_x(status, MPI_CHAR, nbytes);
131131
return MPI_SUCCESS;
132132
}
133133

0 commit comments

Comments
 (0)