Prhub

#7452 [doc] chore: Update ckpt engine readme

原始 PR 作者 LeoYao123 合并时间 2026-08-18 10:55 文件变更 1 提交数 1 评论 0 代码增减 +14 / -1

执行摘要

Ascend 950 文档更新:kimi checkpoint engine 改白名单支持

PR body 明确目标是 add kimi-ckpt-engine support in ascend 950,说明此前 Ascend 950 被认为不支持 kimi-checkpoint-engine,实际上可通过 HCCL 白名单机制开启。README 修改正是为了纠正这一认知偏差,并给出可操作的环境变量配置方法。

可以花少量时间精读这段文档,重点是理解 HCCL 白名单机制在 Ascend 950 上的启用方式;建议在合并后跟进修正 JSON 示例格式,并补充 IP 格式与适用范围说明。若后续有相关源码支持 PR,可与此文档同步更新。

讨论亮点

该 PR 没有实质 review 评论;唯一的审核人 wucong25 直接以 APPROVED 状态通过,未留下讨论或反对意见。

实现拆解

  1. 变更入口:修改 verl/checkpoint_engine/README.md 的“Important Configuration for Ascend Devices”小节。
  2. 核心内容:将“kimi-checkpoint-engine hasn't been supported in Ascend 950”改为需设置环境变量 HCCL_WHITELIST_DISABLE=0HCCL_WHITELIST_FILE=/path/to/whitelist.json,并创建 JSON 白名单文件登记设备 IP。
  3. 示例补充:给出 whitelist JSON 文件的格式模板;原 PR 中的写法 { "host_ip": ["ip1"], ["ip2"] } 不是合法键值对,本报告按合理语义整理为 { "host_ip": ["ip1", "ip2"] }
  4. 兼容衔接:保留原有“CANN >= 8.5.0 时需设置环境变量启用 node 内 ROCE”的提示,确保新旧配置说明连续。
  5. 配套说明:无源码、测试、配置模板改动,纯文档变更。
文件 模块 状态 重要度
verl/checkpoint_engine/README.md 检查点引擎 modified 2.92

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

评论区精华

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

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

风险与影响

主要风险在文档正确性:README 中的 JSON 示例 { "host_ip": ["ip1"], ["ip2"] } 不是合法键值对,["ip2"] 缺少键,用户照抄可能导致配置失败;按语义应为 { "host_ip": ["ip1", "ip2"] }。此外,文档未说明 HCCL_WHITELIST_FILE 的路径要求、IP 格式(IPv4/IPv6)以及是否适用于其他 Ascend 型号,信息完整性不足。由于是纯文档变更,无运行时影响,技术风险极低。

影响范围为使用 Ascend 950 并尝试启用 kimi-checkpoint-engine 的用户:本次文档使其从“认为不支持”变为可配置,降低上手门槛。对仓库代码功能和现有测试无影响。对团队而言,文档维护成本低,但 JSON 示例若被照搬可能导致用户配置报错,需关注后续反馈或补丁。

JSON 示例格式存疑 信息完整性不足

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论