perf(qwen35): improve concurrent AR prefill latency - #622
Conversation
|
Emitter latency follow-up (commit abae692) Upstream behavior checked before implementation:
Luce had imposed its 15-byte tool-syntax holdback even for ordinary requests with no declared tools. This follow-up makes ordinary content immediate while retaining split reasoning-marker detection; requests with tools or custom stops retain the existing conservative buffering. Validation:
This is a one-repeat screen. Prefill order varied between runs, so the direct unit regression and visible-minus-native gap are the cleaner causal evidence. Six C16 requests became visible about 0.33 s after native prefill; ten still waited about one 22.37 s traversal because their first sampled token produced no visible text. That remaining delay is a separate scheduler/detokenizer follow-up. |
abae692 to
4742333
Compare
4742333 to
812eabc
Compare
This PR is based directly on
main. Its complete GitHub diff contains only server runtime code, server documentation, and server-side tests. The benchmark tooling and detailed measurement methodology are isolated in #596.What this changes
Runtime behavior
The screened default remains
DFLASH_PREFILL_FIRST_BURST_STEPS=0. Prefill-first bursts can improve a native prefill tail but delay decode, so they remain opt-in.DFLASH_PREFILL_COHORT_ROUND_ROBINis also opt-in; the default remains FIFO plus the existing intra-cohort partial-budget cursor.Dynamic graph sizing retains the old 16,384-node capacity through eight recurrent segments, then adds 2,048 nodes per additional segment. The observed C16 mixed schedule requires 16,741 nodes, so it receives an 18,432-node graph.
Strix Halo validation
The benchmark tooling lives in #596. One-repeat screens ran exclusively on
gfx1151/ Radeon 8060S.The SSE follow-up used the exact same long-C16 selected prompt set (54,148 prompt tokens, 16/16 valid, exactly 64 output tokens/request). Median TTFT moved from 140.464 s to 123.210 s (-12.3%); max TTFT was essentially unchanged at 157.854 s. This is a one-repeat screen, so the direct emitter regression test and the reduced visible-minus-native delay are the stronger causal evidence.
Validation
dflash_serverbuilt and linked for gfx1151;maincontains only 13server/files.Review notes
<think>/</think>fragments remain boundary-safe.