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

io/romio314: mark datatypes of size 0 as contiguous #4895

Merged

Conversation

edgargabriel
Copy link
Member

this commit fixes an issue observed with romio314 and the hdf5 1.10.x testsuite.
The ADIOI_Datatype_iscontig() routine in romio314/src/io_romio314_module.c
will now return for a datatype of size 0 that it is contiguous, even if the extent
of the datatype is non-zero. This avoids a segmentation fault observed in the
ADIOI_Flatten routine, and fixes this particular with the hdf5 1.10.x testsuite in
OpenMPI with romio314.

Signed-off-by: Edgar Gabriel egabriel@central.uh.edu
(cherry picked from commit c83b47c)

this commit fixes an issue observed with romio314 and the hdf5 1.10.x testsuite.
The ADIOI_Datatype_iscontig() routine in romio314/src/io_romio314_module.c
will now return for a datatype of size 0 that it is contiguous, even if the extent
of the datatype is non-zero. This avoids a segmentation fault observed in the
ADIOI_Flatten routine, and fixes this particular with the hdf5 1.10.x testsuite in
OpenMPI with romio314.

Signed-off-by: Edgar Gabriel <egabriel@central.uh.edu>
(cherry picked from commit c83b47c)
@edgargabriel
Copy link
Member Author

the comit email checker seems to be stuck

@edgargabriel
Copy link
Member Author

bot:retest

@jsquyres
Copy link
Member

jsquyres commented Mar 9, 2018

@edgargabriel This periodically happens -- sometimes our email checker bot gets a malformed request from Github and gets confused. I just re-triggered it on the back end, and now everything is good.

@hppritcha hppritcha merged commit d8b0b2d into open-mpi:v2.x Mar 13, 2018
@edgargabriel edgargabriel deleted the pr/v2.1.x_romio_size0_contiguous_flag branch March 21, 2018 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants