-
Notifications
You must be signed in to change notification settings - Fork 80
Benchmarks: Micro benchmark - Add simultanneously all-to-host / host-to-all bandwidth testcases to nvbandwidth #736
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #736 +/- ##
=======================================
Coverage 85.97% 85.97%
=======================================
Files 102 102
Lines 7578 7578
=======================================
Hits 6515 6515
Misses 1063 1063
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
A general question: why do we need to test fully parallel all-to-host and host-to-all? |
we want to check the nvidia c2c performance between grace gpu and grace cpu, which need fully parallel alltohost and sum, if we only use one gpu to the cpu, it's not accurate |
|
We have another PR [#739] will do the nvbandwidth submodule upgrade, let's see which one can be the first to merge, then we can remove the upgrade from the other PR. |
…h according to comment
Description
Add simultanneously all-to-host / host-to-all bandwidth testcases to nvbandwidth .
Major Revision