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

Remove hint nc_header_collective but still allow collective I/O for header #138

Merged
merged 6 commits into from
Mar 22, 2024

Conversation

wkliao
Copy link
Member

@wkliao wkliao commented Mar 22, 2024

#104 add a new PnetCDF hint nc_header_collective which is not necessary,
because we can simply check ROMIO hint romio_no_indep_rw to serve the
same purpose. Thus, this PR removes PnetCDF hint nc_header_collective
added in #104.

In all previous PnetCDF's releases, file header is written/read by rank 0 only
using MPI independent APIs. This can nullify ROMIO hint romio_no_indep_rw
if set by the user. To warrant no independent read/write, this PR first checks hint
romio_no_indep_rw and if set to true, then all file header I/Os are done
using MPI collective I/O calls, where only rank 0 makes non-zero length requests
while all others zero length (in order to participate the collective calls).

@wkliao wkliao merged commit 93b3de2 into master Mar 22, 2024
3 of 4 checks passed
@wkliao wkliao deleted the remove_hint_nc_header_collective branch May 29, 2024 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant