-
Notifications
You must be signed in to change notification settings - Fork 280
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
stream: implement multiplex stream comm #5946
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hzhou
changed the title
stream: implement pt2pt stream APIs
stream: implement multiplex stream comm
Apr 16, 2022
hzhou
force-pushed
the
2204_stream_multiplex
branch
10 times, most recently
from
April 20, 2022 14:57
8403271
to
80cbd1f
Compare
test:mpich/ch4/most |
hzhou
force-pushed
the
2204_stream_multiplex
branch
3 times, most recently
from
April 20, 2022 19:13
6a9af63
to
cefa69a
Compare
7 tasks
hzhou
force-pushed
the
2204_stream_multiplex
branch
from
April 20, 2022 22:24
cefa69a
to
eddfab3
Compare
test:mpich/ch4/most |
test:mpich/ch3/tcp |
hzhou
force-pushed
the
2204_stream_multiplex
branch
from
April 26, 2022 22:10
eddfab3
to
101e2c1
Compare
test:mpich/ch4/most Peculiar failures:
Why only |
test:mpich/ch4/most |
hzhou
force-pushed
the
2204_stream_multiplex
branch
2 times, most recently
from
April 27, 2022 23:13
d531d5a
to
78af0d8
Compare
test:mpich/ch4/most
|
hzhou
force-pushed
the
2204_stream_multiplex
branch
2 times, most recently
from
April 28, 2022 04:44
2f76dd2
to
954fbb6
Compare
test:mpich/ch4/most |
Define specific macros so we can have tests specific to certain netmod.
When MPICH_CH4_OFI is not define, the test reduce to a dummy test.
Some tests are skipped by checking configure macros. Let runtests to recognize special output "Test Skipped" so we can tell from test report whether a test is skipped or actually ran.
We special treated the REQUEST array and STATUS array in the binding scripts, now we are paying the price.
Each process has multiple streams associated with a multiplex stream communicator.
This is a utility routine to support pt2pt apis on stream multiplex communicator.
Add -- * MPIX_Stream_send * MPIX_Stream_isend * MPIX_Stream_recv * MPIX_Stream_irecv
It tests MPIX_Stream_comm_create_multiplex, MPIX_Stream_send, MPIX_Stream_recv, MPIX_Stream_isend, MPIX_Stream_irecv.
hzhou
force-pushed
the
2204_stream_multiplex
branch
from
April 30, 2022 04:00
954fbb6
to
221285f
Compare
test:mpich/ch4/most |
raffenet
approved these changes
May 2, 2022
5 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Description
Implement --
MPIX_Stream_comm_create_multiplex
MPIX_Stream_send
MPIX_Stream_recv
Depend on:
[skip warnings]
Author Checklist
Particularly focus on why, not what. Reference background, issues, test failures, xfail entries, etc.
Commits are self-contained and do not do two things at once.
Commit message is of the form:
module: short description
Commit message explains what's in the commit.
Whitespace checker. Warnings test. Additional tests via comments.
For non-Argonne authors, check contribution agreement.
If necessary, request an explicit comment from your companies PR approval manager.