Prhub

#27669 [AMD] Update MoRI to v1.2.0

原始 PR 作者 bingxche 合并时间 2026-06-10 13:41 文件变更 1 提交数 1 评论 0 代码增减 +1 / -1

执行摘要

AMD Docker 镜像中 MoRI 库更新至 v1.2.0

PR body 明确指出更新是为了解决 EP16 with CX7+RoCEv2 的问题。MoRI v1.2.0 版本包含必要的修复以支持该场景。

该 PR 为简单的依赖版本更新,无需详尽审查。建议快速合入以确保 AMD 用户能正常使用最新网络功能。

讨论亮点

无人类评审讨论。Gemini 代码辅助机器人自动评论表示无反馈意见。仓库维护者 HaiShaw 直接批准。

实现拆解

  1. 修改 Dockerfile 构建参数:在 docker/rocm.Dockerfile 中,将 MORI_COMMIT 参数的值从 d87651c998296c5bddbeefc4cc525b58663b1636 改为 bf99bdf18fc69887a346913ca01c315c2aa9bd4c,该提交对应 MoRI 仓库的 v1.2.0 标签。
  2. 构建与测试:通过 CI 验证了 EP(执行提供器)和 disagg(分离式)测试均通过,确保兼容性和稳定性。
文件 模块 状态 重要度
docker/rocm.Dockerfile 部署脚本 modified 3.18

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

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

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

风险与影响

风险极低。变更仅限于 Docker 构建过程中 MoRI 库的版本固定提交,属于常规依赖升级。如果新版本存在未预期的回归,仅影响 AMD 平台上的高性能网络功能(EP16 + RoCEv2),但 CI 测试已验证关键场景通过。

直接影响使用 AMD ROCm Docker 镜像的用户,特别是需要 CX7+RoCEv2 网络进行 EP16 的用户。修复了此前可能存在的功能问题。对其他平台和无相关网络需求的用户无影响。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论