Skip to content

[CI] Run test_unified_radix_cache_kl_dcp on cutedsl_mla with bf16 KV cache - #39553

Merged
kpham-sgl merged 1 commit into
sgl-project:mainfrom
kpham-sgl:kpham/kl-dcp-cutedsl
Sep 15, 2026
Merged

kpham-sgl merged 1 commit into
sgl-project:mainfrom
kpham-sgl:kpham/kl-dcp-cutedsl

Conversation

@kpham-sgl

@kpham-sgl kpham-sgl commented Sep 15, 2026 •

Copy link
Copy Markdown
Collaborator

Motivation

test_unified_radix_cache_kl_dcp.py has failed ~50% of scheduled extra-b-test-4-gpu-b200 runs since 09/11 (avg_kl_div 0.010-0.013 vs threshold 0.01, both retry attempts).

Bisected to #35770: resolve_attn_backend() now unwraps HybridLinearAttnBackend.full_attn_backend, so Kimi Linear prefill hits TokenspeedMLABackend.prepare_prefill_qkv and quantizes Q/K/V to FP8 before the prefill kernel (#38152 made the hook NoPE-safe after it first crashed on 09/05). Scheduled runs measured 0.003-0.006 through 09/04 and 0.007-0.013 since. Not runner-specific; no package versions changed.

Modifications

Switch the test to cutedsl_mla on the default bf16 KV cache (drop --kv-cache-dtype fp8_e4m3), like every other registered KL test. cutedsl_mla is the DCP-native MLA decode kernel on Blackwell and already serves Kimi Linear DCP in test_kimi_linear_unified_memory_dcp_blackwell.py. Threshold unchanged at 0.01.

Test

10 consecutive runs on 4x B200, main @ 2fca6d6: 10/10 OK, ~4.5 min each.

case avg_kl_div min max threshold
logprobs_match 0.0052 0.0057 0.01
prefill_cache_hit 0.0029 0.0037 0.01
decode_cache_hit 0.0026 0.0042 0.01

GSM8K 0.930-0.945 (threshold 0.85). Same-config tokenspeed_mla runs on scheduled CI since 09/06: 0.007-0.013.

@kpham-sgl kpham-sgl added run-ci CI: run the baseline test suite on this PR run-ci-extra CI: also run the extra suite (requires run-ci) and removed run-ci CI: run the baseline test suite on this PR run-ci-extra CI: also run the extra suite (requires run-ci) labels Sep 15, 2026
@kpham-sgl

Copy link
Copy Markdown
Collaborator Author

/rerun-test test/registered/radix_cache/unified_radix_tree/test_unified_radix_cache_kl_dcp.py

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Rebase Required Before Re-run

A major update has landed on main. Your PR is diverged relative to required base commit ddd4600197a4.

Re-run was not dispatched. What to do:

  • Rebase your branch onto the latest main and push again
  • Follow issue #21065 for context
  • CI-fix PRs may request the bypass-maintenance label to skip this check

…cache

The test has been failing ~50% of scheduled extra-b B200 runs since 09/11
with avg_kl_div 0.010-0.013 vs the 0.01 threshold. sgl-project#35770 routed Kimi Linear
prefill through the tokenspeed_mla FP8 prepare_prefill_qkv hook, which
doubled the KL from 0.003-0.006. Switch to cutedsl_mla on the default bf16
KV cache like every other KL test; threshold unchanged.
@kpham-sgl

Copy link
Copy Markdown
Collaborator Author

/rerun-test test/registered/radix_cache/unified_radix_tree/test_unified_radix_cache_kl_dcp.py

@github-actions

github-actions Bot commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

Results for /rerun-test test/registered/radix_cache/unified_radix_tree/test_unified_radix_cache_kl_dcp.py:

🚀 4-gpu-b200 (1 test): ✅ View workflow run

cd test/ && python3 registered/radix_cache/unified_radix_tree/test_unified_radix_cache_kl_dcp.py

@kpham-sgl
kpham-sgl merged commit e4a6b09 into sgl-project:main Sep 15, 2026
107 of 119 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant