Prhub

#25044 Revert "Migrate Intel CPU cases to the test/registered."

原始 PR 作者 mingfeima 合并时间 2026-05-12 13:32 文件变更 28 提交数 1 评论 3 代码增减 +35 / -4954

执行摘要

回滚 #22670 并修复 docs_new/index.mdx 意外修改

PR author mingfeima 在评论中指出 'revert #22670 which unintentionally changed docs_new/index.mdx',因此此 PR 是为了修复该意外修改。

建议快速批准此 revert,因为它修复了文档意外变更。但团队应安排后续 PR 以不修改 index.mdx 的方式重新引入 CPU 测试,确保 CI 覆盖不长期缺失。

讨论亮点

由于本次 PR 是直接 revert,没有引发 review 讨论。有一条评论来自作者说明 revert 原因,另一条来自 mintlify bot 报告文档预览失败(这可能是由于文档结构变更导致的副作用)。

实现拆解

该 PR 仅包含一个 commit,直接 git revert 了 #22670 的提交。变更如下:

  1. 撤销测试迁移:删除 test/registered/cpu/ 下所有新增的测试文件(共 24 个文件),这些文件本是 #22670 为 Intel CPU 添加的单元测试。
  2. 恢复文档:docs_new/index.mdx 被恢复到合并前的状态(+35 行)。
  3. 无其他逻辑变更:没有引入新的功能或修复其他 bug。

测试删除涉及的范围包括:QKV projection with RoPE、norm、mamba attention、MoE、GEMM、causal conv1d、RoPE、flash attention、shared expert、extend attention、topk 等算子的 CPU 测试。配套工具文件 utils.py 和 CI 注册装饰器也被移除。

文档的恢复确保了门户页面的正确渲染。

文件 模块 状态 重要度
docs_new/index.mdx 文档 modified 7.44
test/registered/cpu/test_qkv_proj_with_rope.py 投影测试 deleted 7.48
test/registered/cpu/utils.py 测试工具 deleted 7.76
test/registered/cpu/test_norm.py 归一化测试 deleted 7.49
test/registered/cpu/test_gemm.py GEMM 测试 deleted 7.49

关键符号

TestQKVProjWithROPE TestGemm TestNorm TestCausalConv1d TestMambaAttention

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

评论区精华

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

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

风险与影响

主要风险是 Intel CPU 测试覆盖率的降低——#22670 引入了 22 个新测试文件,回滚后这些测试将不再运行,可能遗漏 CPU 上的算子回归。但原 PR 在合并时本身可能未经过充分 review,且意外修改了文档,因此回滚是稳妥之举。后续需要以正确的方式重新引入这些测试。此外,文档预览失败提示可能存在文档构建问题,但回滚后已恢复。

影响范围:

  • 用户:无直接影响(测试不涉及用户可见功能)。
  • 系统:CPU CI 阶段不再运行这些特定测试,降低了对 Intel CPU 算子的回归检测能力。
  • 团队:需要重新审视 CPU 测试策略,在后续 PR 中更小心地迁移测试。
    影响程度:中等(对 CI 覆盖有负面影响,但避免了文档错误)。
测试覆盖率降低 文档修复 CI 影响 (CPU 测试缺失 )

关联 Issue

#22670 Migrate Intel CPU cases to the test/registered.

完整报告

参与讨论