Multi Memory Region, Multi Domain, Multi Endpoint, Recv Cancel, & Unexpected_msg FT #689
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
-simple/multi_mr functional test (fi_multi_mr) - Performs multikey/multi memory region ping pong testing - Uses fi_mr_bind on each memory region to track individual writes - Uses separate key for each memory region to target each write call - Performs data verification on the result from each write - Allows for testing FI_EP_MSG/FI_EP_RDM/FI_EP_DGRAM - test added to "simple" test list -common/shared - Added ft_mr_alloc_func function pointer to shared.h - When set, allocation of memory regions done by ft_mr_alloc_func Signed-off-by: nrspruit <neil.r.spruit@intel.com>
-simple/rdm_multi_domain functional test (fi_rdm_multi_domain)
- Uses multi-domains each with memory region for ping pong RMA testing
- Uses fi_mr_bind on each memory region to track individual writes
- Uses separate domain ep address to target each write call
- test added to "simple" test list
-common/shared
- Added init functions for Address Vectors
- ft_init_av_addr for passing ep addresses between client and server
- ft_init_av_dst_addr for init dst addr on server and server's ep
addr on client
Signed-off-by: nrspruit <neil.r.spruit@intel.com>
-simple/multi_ep functional test (fi_multi_ep)
- Allows users to create n number of active endpoints
- Supports dgram,msg,rdm endpoint types
- Sends messages from each EP to the corresponding EP on the remote host
- Supports data validation
-simple/multi_mr
- Updated to use common init function for cm testing
-common/shared
- Added support for setup of eps with user specified resources
- Added functions to split up CM connect/accept functionality
- Added init function for cm testing
- Added option for setting the default address vector's size
Signed-off-by: nrspruit <neil.r.spruit@intel.com>
-simple/recv_cancel functional test (fi_recv_cancel)
- Tests cancelling a single pending recv
- Verifies that the error entry is valid
- Supports dgram,msg,rdm endpoint types
- Added to simple test list
-common/shared
- Added ft_tag for setting tags in send/recv ops
- Added verbose option for tracking enabling of prints
in a test
Signed-off-by: nrspruit <neil.r.spruit@intel.com>
-simple/unexpected_msg functional test (fi_unexpected_msg)
- Tests sending N messages to N recvs as unexpected messages
- Allows for testing with cq_sread or cq_read
- data verification on messages is available
- Configurable number of concurrent messages and iterations
- Configurable size for the unexpected messages
- Supports dgram,msg,rdm endpoint types
- Added to simple test list
Signed-off-by: nrspruit <neil.r.spruit@intel.com>
|
@jsquyres - The DCO sign-off-by check is failing for this PR, but our check works fine. Manually inspecting the patches, everything looks okay to me. Any idea why DCO fails? |
nrspruit
force-pushed
the
unexpected_msg_test
branch
2 times, most recently
from
6c1768e
to
c2a702b
Compare
nrspruit
changed the title
nrspruit
force-pushed
the
unexpected_msg_test
branch
from
c2a702b
to
d59989a
Compare
aingerson
reviewed
Jul 21, 2017
aingerson
reviewed
Jul 21, 2017
|
I filed a question on the DCO Probot project asking why it returned a failure on this PR (dcoapp/app#17). I suspect it's because the name does not match between the commit author and the DCO signed off line, but let's wait to see what the project authors say. |
|
Thank you jsquyres for posting the question, I had changed the commits to nrspruit instead of Neil Spruit as they failed with that too, I have a feeling that my original may have had a character off and that is why the issue, thank you for following up and my next pull request will follow the correct guidelines. Thank you very much for your time! |
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.
Implemented new Functional tests for verifying:
Implemented support in the common infrastructure for:
Signed-off-by: nrspruit neil.r.spruit@intel.com