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

Advice regarding PIO async in multi-component model #1853

Closed
nichannah opened this issue Feb 8, 2021 · 4 comments
Closed

Advice regarding PIO async in multi-component model #1853

nichannah opened this issue Feb 8, 2021 · 4 comments

Comments

@nichannah
Copy link

nichannah commented Feb 8, 2021

Hi,

I'd like to get some advice re implementing asynchronous IO for a single model in a multi-component model. I think my main query is where I should call pio_init() wrt. the existing call to MPI_Split that creates the individual communicators for the component models. The options I can think of:

  1. Call pio_init() on all PEs before my existing MPI_Split. This will leave me with a new computation communicator which will become the new COMM_WORLD and an IO communicator. The new COMM_WORLD will then be split into component models.

  2. Call pio_init() after the existing MPI_Split with only the PEs that are a part of the component model which uses the async IO. For various reasons I'm not sure how this would work, e.g. any subsequent use of MPI_COMM_WORLD for collectives in the code would break because the IO PEs are not going to join in.

  3. Somehow replace the existing MPI_Split with the one that (presumably) is being called within pio_init()

Any thoughts or examples of how this is done elsewhere would be much appreciated.

@edwardhartnett
Copy link
Collaborator

You do not do the MPI_split.

You call PIO_Init() on all processors. Those that are I/O will not return from PIO_Init() until all computational units call PIO_Finalize().

@edwardhartnett
Copy link
Collaborator

Also, make sure you use the current master. It has some very important async fixes. I am hoping to do a release this week with those fixes and improvements.

@jedwards4b
Copy link
Contributor

jedwards4b commented Feb 8, 2021 via email

@nichannah
Copy link
Author

Thank you very much @edwardhartnett and @jedwards4b - this is what I was after.

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

No branches or pull requests

3 participants