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

[Question] Buffered mode #61

Closed
dutkalex opened this issue May 14, 2024 · 4 comments
Closed

[Question] Buffered mode #61

dutkalex opened this issue May 14, 2024 · 4 comments

Comments

@dutkalex
Copy link
Collaborator

Hi! I notice that the CommMode enum does not include the buffered mode. Is this intentional or just an oversight?
Best regards,
Alex

@dssgabriel
Copy link
Collaborator

dssgabriel commented May 14, 2024

This is intentional (see related PR), as the Buffered communication mode requires manual buffer attachment from the user through MPI_Buffer_attach().
As our currently targeted applications do not need support for buffered communications, we just dismissed it.

We're currently thinking about adding functions for initializing, finalizing and, more generally, managing KokkosComm; this may change soon.

@devreal
Copy link
Collaborator

devreal commented May 14, 2024

For the record: MPI 4.1 added MPI_COMM_ATTACH_BUFFER and MPI_SESSION_ATTACH_BUFFER to limit the scope of buffers to a communicator or session. It also added MPI_BUFFER_AUTOMATIC to enable automatic buffer allocation. Not sure what the implementation status is but this would be useful here once its provided.

@dssgabriel
Copy link
Collaborator

Indeed, especially if we want to go the MPI_Session route. I am going to look into it, thanks @devreal.

@dutkalex
Copy link
Collaborator Author

dutkalex commented May 14, 2024

The argument makes sense. Thanks for the answer 🙃
(You can close the issue if you wish)

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