Prhub

#43664 [Misc][Rocm] Remove redundant `AiterUnifiedAttentionBackend` block size log

原始 PR 作者 NickLucche 合并时间 2026-05-28 11:19 文件变更 1 提交数 2 评论 1 代码增减 +0 / -3

执行摘要

移除 ROCm Aiter 后端冗余日志

PR body 指出日志是冗余的,且不应作为 warning 级别(用户无法对此采取任何行动)。块大小管理系统已处理并记录了 INFO 日志,因此此日志重复且无用。

可直接合入,无需关注。

讨论亮点

无实质性讨论,AndreasKaratzas 直接批准。

实现拆解

vllm/v1/attention/backends/rocm_aiter_unified_attn.py 文件的 get_preferred_block_size 方法中,删除了调用 logger.warning_once 输出冗余警告的代码行。方法现在直接返回 64,不再产生日志。

文件 模块 状态 重要度
vllm/v1/attention/backends/rocm_aiter_unified_attn.py 注意力层 modified 3.66

关键源码片段

vllm/v1/attention/backends/rocm_aiter_unified_attn.py core-logic

唯一变更文件,删除了冗余日志调用。

# vllm/v1/attention/backends/rocm_aiter_unified_attn.pyclass RocmAiterUnifiedAttentionBackend(RocmAttentionBackend):
    # 删除前:classmethod get_preferred_block_size 中有一行冗余的 logger.warning_once
    # 删除后:直接返回 64,不再输出警告
    @classmethod
    def get_preferred_block_size(cls, default_block_size: int) -> int:
        # logger.warning_once("[ROCM_AITER_UNIFIED_ATTN]: Setting kv cache block size to 64.") # <-- 被删除
        return 64

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

无风险。仅删除一行无关紧要的日志,不影响任何功能逻辑。

对用户无影响;减少 ROCm Aiter Unified Attention 后端启动时的一条无用警告日志。

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论