Skip to content

Conversation

@yukirora
Copy link
Contributor

@yukirora yukirora commented Sep 8, 2025

Description
Add simultanneously all-to-host / host-to-all bandwidth testcases to nvbandwidth .

Major Revision

  • nvbandwidth.patch: Add simultanneously all-to-host / host-to-all bandwidth testcases to nvbandwidth
  • upgrade nvbandwidth submodule into v0.8
  • add patch into makefile build

@yukirora yukirora requested a review from a team as a code owner September 8, 2025 13:00
@yukirora yukirora added the benchmarks SuperBench Benchmarks label Sep 8, 2025
@codecov
Copy link

codecov bot commented Sep 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.97%. Comparing base (76066b6) to head (bfc2d7b).
⚠️ Report is 1 commits behind head on main.

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           
Flag Coverage Δ
cpu-python3.10-unit-test 71.05% <ø> (ø)
cpu-python3.12-unit-test 71.05% <ø> (ø)
cpu-python3.7-unit-test 70.49% <ø> (ø)
cuda-unit-test 83.86% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yzygitzh
Copy link
Contributor

A general question: why do we need to test fully parallel all-to-host and host-to-all?

@yukirora
Copy link
Contributor Author

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

@cp5555 cp5555 requested review from cp5555 and yzygitzh September 18, 2025 09:27
@guoshzhao guoshzhao added the micro-benchmarks Micro Benchmark Test for SuperBench Benchmarks label Sep 19, 2025
@guoshzhao
Copy link
Contributor

guoshzhao commented Sep 21, 2025

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.

@yukirora yukirora enabled auto-merge (squash) September 30, 2025 02:28
@yukirora yukirora merged commit 93e9d26 into main Sep 30, 2025
24 of 25 checks passed
@yukirora yukirora deleted the yutji/nvbandwidth branch September 30, 2025 04:31
@guoshzhao guoshzhao mentioned this pull request Oct 2, 2025
30 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmarks SuperBench Benchmarks micro-benchmarks Micro Benchmark Test for SuperBench Benchmarks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants