Skip to content

[https://nvbugs/6104831][test] PR #14768 rebased on latest main — perf-sanity verification (DO NOT REVIEW) - #14916

Draft
chienchunhung wants to merge 1 commit into
NVIDIA:mainfrom
chienchunhung:pr14768-rebased-on-main
Draft

[https://nvbugs/6104831][test] PR #14768 rebased on latest main — perf-sanity verification (DO NOT REVIEW)#14916
chienchunhung wants to merge 1 commit into
NVIDIA:mainfrom
chienchunhung:pr14768-rebased-on-main

Conversation

@chienchunhung

Copy link
Copy Markdown
Collaborator

Purpose — verification only, NOT for merge

This PR is an experiment to isolate whether the perf-sanity regressions seen on PR #14768 (builds #40858, #41014, #41125 — DeepSeek R1 FP4 V2 on GB200) are attributable to PR #14768's code or to branch staleness.

What this PR contains

The net diff of PR #14768 (A1+A2+A4+A7+A8 disaggregated KV cache transceiver lifetime / RAII / dedup / NIXL keep-alive fixes), applied as a single squashed commit on top of the latest upstream/main HEAD.

Hypothesis being tested

PR #14768 fails 5 perf-sanity tests on GB200 against an auto-uploaded baseline:

  • aggr_upload-deepseek_r1_fp4_v2_grace_blackwell-r1_fp4_v2_tep4_mtp3_{1k1k,8k1k}
  • aggr_upload-deepseek_r1_fp4_v2_grace_blackwell-r1_fp4_v2_tp4_mtp3_{1k1k,8k1k}
  • aggr_upload-k25_thinking_fp4_grace_blackwell-k25_thinking_fp4_dep4_8k1k

The PR's diff only modifies disaggregated transceiver code paths (gated by mCacheTransceiver != nullptr / if (self.kv_cache_transceiver):). The failing tests are aggregated-mode (aggr_upload-*) — config files in tests/scripts/perf-sanity/aggregated/ with no cache_transceiver_config field — so PR #14768's modifications are not on the failing tests' execution path. Same tests pass on main postmerge (#2750, #2751, #2754) consistently.

Hypothesis: the perf "regression" reflects baseline drift — main has perf improvements (~632 LOC in fused-MoE / NVFP4 / speculative since PR #14768's merge-base) that the auto-uploaded baseline reflects, but PR #14768's stale branch lacks. Rebasing should resolve the apparent regression.

How this will be verified

/bot run --stage-list "GB200-4_GPUs-PyTorch-PerfSanity-1, GB200-4_GPUs-PyTorch-PerfSanity-2" — runs only the 5 failing tests.

Will close this PR once the signal is collected.

…test main

Verification PR — equivalent net diff of NVIDIA#14768 (A1+A2+A4+A7+A8 disagg
tier-1 baseline) applied as a single commit on top of the latest
upstream/main HEAD.  Purpose: verify whether the perf-sanity regressions
seen on NVIDIA#14768 (build #41014, #41125 — DeepSeek R1 FP4 V2 perf) are
attributable to PR code or to branch staleness (32 commits behind main).

Do NOT merge.  Will be closed once CI signal is collected.

Signed-off-by: Chien-Chun Hung <2679986+chienchunhung@users.noreply.github.com>
@chienchunhung chienchunhung changed the title [https://nvbugs/6104831][test] PR #14768 rebased on latest main — perf-sanity verification [DO NOT REVIEW][https://nvbugs/6104831][test] PR #14768 rebased on latest main — perf-sanity verification Jun 3, 2026
@chienchunhung chienchunhung changed the title [DO NOT REVIEW][https://nvbugs/6104831][test] PR #14768 rebased on latest main — perf-sanity verification [https://nvbugs/6104831][test] PR #14768 rebased on latest main — perf-sanity verification (DO NOT REVIEW) Jun 3, 2026
@chienchunhung

Copy link
Copy Markdown
Collaborator Author

/bot run --stage-list "GB200-4_GPUs-PyTorch-PerfSanity-1, GB200-4_GPUs-PyTorch-PerfSanity-2"

@chienchunhung

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast --stage-list "GB200-4_GPUs-PyTorch-PerfSanity-1, GB200-4_GPUs-PyTorch-PerfSanity-2"

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #51879 [ run ] triggered by Bot. Commit: 152f6b3 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #51879 [ run ] completed with state SUCCESS. Commit: 152f6b3
/LLM/main/L0_MergeRequest_PR pipeline #41238 (Partly Tested) completed with status: 'SUCCESS'

CI Report

Link to invocation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants