Conversation
|
[P1] super().update_block_size_for_backend() 会覆盖 HCU 已经选好的 block size
|
There was a problem hiding this comment.
AI Review
🟢 未发现有明确证据的问题
在已提供并完成审查的 diff 范围内未形成可确认问题。
变更概览
本次变更主要包含:
- 将平台的 update_block_size_for_backend 从空实现改为调用父类实现,使 vLLM 共享的混合注意力/Mamba 缓存块大小对齐逻辑在该后端重新生效。
文件审查摘要
| 文件 | 变更 | 审查结果 |
|---|---|---|
vllm_hcu/platforms/hcu.py |
修改 · +5/-3 | — |
审查信息
- 变更统计:1 个文件,+5/-3。
- 覆盖情况:共 1 个文件,已完整审查 1 个。
- 候选问题:0 项;证据复核过滤:0 项;发布前敏感信息保护:0 项。
- 本服务只审查 GitHub 提供的 PR diff,未执行代码或重跑测试;结论仍需维护者核验。
AI CI 失败分析工作流:HCU PR CI 总结
与本次改动的关系无法确定:已定位的直接异常对本次改动的关联判断一致。 建议处理
|
Summary
Problem
HCUPlatform.update_block_size_for_backend()was a no-op, which skipped vLLM's hybrid cache-page alignment.For Qwen3.5-27B with TP=2 and
max_model_len=262144, this caused the KV cache requirement to be overestimated at approximately 391 GiB, preventing large-context startup.After restoring the shared alignment logic, the attention block size is adjusted to 784 tokens with 0.13% Mamba-page padding. The resulting KV cache capacity is 807,957 tokens, supporting 3.08 concurrent 262,144-token requests.
Validation
python -m py_compile vllm_hcu/platforms/hcu.pygit diff --checkmax_model_len=262144/healthreturns HTTP 200