-
Notifications
You must be signed in to change notification settings - Fork 10
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
performance/multi-node: use fi_inject_write. #59
base: master
Are you sure you want to change the base?
Conversation
motivation for this PR? Why is this being done? |
Mainly because @hppritcha asked me to. Also, to compare the performance of fi_write to fi_write_inject. |
You need to amend your commit and PR descriptions to reflect that. |
- Use fi_write_inject until the inject threshold is exceeded. - This is to compare the performance of fi_inject_write to fi_write. Signed-off-by: Evan Harvey <[email protected]>
306256e
to
bdc4d8a
Compare
This PR should change to add the option to using fi_write_inject. The default behavior should be as it is now. |
- Enable fi_inject_write by using the -j flag. Signed-off-by: Evan Harvey <[email protected]>
Why |
|
Oh yeah. I'd use |
Did you mean |
Signed-off-by: Evan Harvey <[email protected]>
With the changes introduced in this PR we get:
at https://github.com/ofi-cray/libfabric-cray/tree/6cdc8c283feecc2b3b881666598edcf96dd56d5e:
and at https://github.com/ofi-cray/libfabric-cray/tree/2b63528292d4ead86c120f134edfc6ccb25f8369:
Signed-off-by: Evan Harvey [email protected]