执行摘要
移除 AMD CI 中过时的 ROCm 7.2 hotpatch
PR body 说明 'The hot patch is no longer needed.',即该临时补丁已过时,需要清理以保持 CI 脚本简洁。
该 PR 是简单的清理工作,无需重点阅读。但可以作为 AMD CI 维护者了解历史 hotpatch 的参考。
没有 review 评论或讨论。
PR body 说明 'The hot patch is no longer needed.',即该临时补丁已过时,需要清理以保持 CI 脚本简洁。
该 PR 是简单的清理工作,无需重点阅读。但可以作为 AMD CI 维护者了解历史 hotpatch 的参考。
没有 review 评论或讨论。
scripts/ci/amd/amd_ci_install_dependency.sh 中删除从第 324 行开始的 # Re-apply Dockerfile hotpatches for ROCm 7.2 到对应 fi 的整个代码块,共 18 行。sed 修改 aiter/ops/triton/attention/pa_mqa_logits.py 中的一行,否则跳过。由于 triton 问题已修复,此补丁不再需要。| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
scripts/ci/amd/amd_ci_install_dependency.sh |
CI 脚本 | modified | 3.36 |
分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。
当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。
风险极低。仅删除一段条件性执行的 hotpatch 代码,且该 hotpatch 只在 ROCm 7.2 下生效。如果未来仍需要此补丁,则需要重新添加。但鉴于 PR 已合并,表明维护者认为此补丁确实不再需要。
影响范围仅限于 AMD CI 构建流程中 ROCm 7.2 环境。移除后,在 ROCm 7.2 下构建时将不再应用 hotpatch,构建流程与其他版本一致。
当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。
参与讨论