Skip to content

fix(ci-mesh): K3 AgentX cells — FP8 prefill attention and capture from bs=1 - #2417

Merged
gbyu-amd merged 2 commits into
zwan/feat-k3-pd-disagg-cifrom
ganyi/k3-pd-ci-recipe-sync
Sep 28, 2026
Merged

gbyu-amd merged 2 commits into
zwan/feat-k3-pd-disagg-cifrom
ganyi/k3-pd-ci-recipe-sync

Conversation

@gbyu-amd

@gbyu-amd gbyu-amd commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Brings two items from
recipes/Agentic-Kimi-K3.md
on main (#2382, d8cdd4e2) into the Kimi-K3 P/D nightly cells. Targets zwan/feat-k3-pd-disagg-ci.

Recipe item Before After
ATOM_USE_FLYDSL_FP8_PREFILL_ATTN=1 on every band unset model-level env.common → all 16 K3 cells
CUDA-graph capture from bs=1 cudagraph: auto → [2..graph_max] [1..graph_max] (pd_server_atom.sh; only K3 cells use auto)

Deliberately not taken: the recipe's PrefillDelayer from CONC 16 up
(ATOM_PREFILL_DECODE_INTERVAL=4, ATOM_PREFILL_DELAYER_MAX_QUEUE_MS=5000).
Its gain was measured on the aggregated recipe server. It is unclear whether it helps on the P/D split: main
(#2238) never installs it on the decode instance (disagg_is_decode), and the
prefill instance has no decode batch to protect. This branch also predates
#2238, so it would currently only take effect in DP mode. The first commit added it and the second commit removes it,
so the net diff doesn't include it.

The rest of the recipe's per-concurrency table (DCP, spec/AL, ReplaySSM, LMCache 128/192 GiB, max-num-seqs, AITER_REUSE_IDENTICAL_COMM_GROUPS from C56, state_checkpoint_interval_tokens=-1, ATOM_STATE_CHECKPOINT_DEMAND=0) already matched.

Verification

  • Expanded the K3 nightly matrix with pd_matrix.py before/after: across all 16 cells the only change is ATOM_USE_FLYDSL_FP8_PREFILL_ATTN=1 (64 = 16 cells × 4 role env maps).
  • auto_cudagraph_capture_sizes: C1 spec7 → [1..16], C72 seqs144 → [1..144], matching the recipe's graph_max.
  • tests/test_atomesh_pd_matrix.py, test_atomesh_spur_dispatch.py, test_atomesh_lmcache_disk.py: 26 passed (same as base).

Notes

  • The recipe also lists C12, C14 and C80, which have no cells here. This PR does not add them.

🤖 Generated with Claude Code

ganyi1996ppo and others added 2 commits September 27, 2026 23:01
Sync the K3 P/D nightly cells with recipes/Agentic-Kimi-K3.md as of
d8cdd4e:

- ATOM_USE_FLYDSL_FP8_PREFILL_ATTN=1 on every band
- PrefillDelayer (ATOM_PREFILL_DECODE_INTERVAL=4,
  ATOM_PREFILL_DELAYER_MAX_QUEUE_MS=5000) from CONC 16 up
- `cudagraph: auto` captures the dense range from bs=1 instead of 2

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Its benefit was measured on the aggregated recipe server; on the P/D
split the decode instance never installs it and the prefill instance
has no decode batch to protect, so keep the cells unchanged there.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@gbyu-amd gbyu-amd changed the title fix(ci-mesh): align Kimi-K3 AgentX cells with the recipe fix(ci-mesh): K3 AgentX cells — FP8 prefill attention and capture from bs=1 Sep 28, 2026
@gbyu-amd
gbyu-amd merged commit d64a180 into zwan/feat-k3-pd-disagg-ci Sep 28, 2026
1 check passed
@gbyu-amd
gbyu-amd deleted the ganyi/k3-pd-ci-recipe-sync branch September 28, 2026 04:07
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