

Pull Request已成功合入, 合并人@ascend-robot
(感谢 SCh_zx 的贡献)Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-Ascend/pytorch | ✅ liangsongwei, huangjingwei (2/2) | ✅ liangsongwei (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
SCh_zx, thanks for your pull request. All authors of the commits have signed the CLA. 👍


Linking Issue Notice
@SCh_zx , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .


当前仓库存在以下 保护分支 :
| Protected Branch | Version | Release |
|---|---|---|
| master | ||
| v2.7.1 | ||
| v2.7.1-26.1.0 | ||
| v2.12.0-26.1.0 | ||
| v2.11.0-26.1.0 | ||
| v2.10.0-26.1.0 | ||
| v2.9.0-26.1.0 | ||
| v2.12.0 | ||
| v2.11.0 | ||
| v2.10.0 | ||
| v2.9.0 | ||
| ci-test | ||
| sync-pr28113--to-v2.9.0 |
评论 /sync <branch1> <branch2> ... 可将当前 PR 修改同步到其它分支(创建同步 PR):
a) 如果当前 PR 是 Open 状态,同步操作将延迟到 PR 被合并时执行
b) 如果当前 PR 已经 Merged,将立即执行同步操作
注意:
- /sync 命令可以指定同步到多个分支,仅最后一个 /sync 命令生效
- 如果创建的同步 PR 不正确,可通过向同步 PR 的源分支提交轻量级 PR 完善,或使用 /close 命令关闭


Ascend docs pipeline is running...


Ascend docs pipeline is running...


根据 PR 标题"update_gitmodules"以及变更统计(2 个文件,+9/-9 行),此 PR 主要涉及更新 Git 子模块配置文件。
变更摘要
此 PR 旨在更新仓库的 .gitmodules 子模块配置,同步调整相关文档文件中的子模块引用信息。变更涉及 v2.10.0 分支,以等量的增删行数(+9/-9)对子模块配置项进行了对等替换或修正,确保子模块指向正确且文档与配置保持一致。
主要改动
.gitmodules子模块配置更新: 修改了.gitmodules文件中若干子模块的配置项(如 URL 地址、分支名称或update策略等),以适配v2.10.0分支的依赖版本要求SECURITYNOTE.md子模块登记同步: 随.gitmodules的变更,更新了安全声明文档中记录的子模块 URL 列表,保持第三方依赖声明与实际配置一致


代码审查
现在进行最终扫描,检查边界值、热路径成本和已报告位置的共置次生缺陷。
边界值扫描:本次 diff 不涉及数值配置、阈值或大小限制,均为 URL 字符串变更,不存在边界值问题。
热路径扫描:变更仅涉及静态配置文件(.gitmodules 和 SECURITYNOTE.md),不引入任何运行时调用或分配。
已报告位置的共置缺陷扫描:
- 第 7 行(
.gitmodulesgoogletest URL):URL 格式正确,无次生缺陷。 - 第 16 行(
.gitmodulesfmt URL):URL 格式正确,无次生缺陷。 - 第 19 行(
.gitmodulesnlohmann URL):仅增加.git后缀,语义不变,无次生缺陷。 - 第 22 行(
.gitmodulestorch-mlir URL):URL 格式正确,无次生缺陷。 SECURITYNOTE.md各 URL 均与.gitmodules保持一致,无次生缺陷。
审查总结
| 文件 | 审查结论 |
|---|---|
.gitmodules |
发现 1 个 P2 问题(供应链风险) |
SECURITYNOTE.md |
无问题(文档更新与 .gitmodules 保持一致,修复了之前 torchair 条目文档与实际 URL 不符的问题) |
- P0: 0
- P1: 0
- P2: 1(子模块源从 gitee.com 迁移至 gitcode.com 非官方命名空间的供应链风险)
- P3: 0
整体风险判断:本次变更的核心操作是将残留的 gitee.com 子模块源统一迁移至 gitcode.com,与项目其他子模块保持一致,方向合理。但 googletest、fmt、torch-mlir 三个子模块指向 GitHub_Trending 和 gh_mirrors 命名空间,其镜像来源的权威性和完整性未经证实,存在中等程度的供应链风险,建议 PR 作者核实这些镜像的官方维护状态后再合入。
| 类型 | 数量 |
|---|---|
| 🔴 阻塞 | 0 |
| 🟡 建议 | 1 |
💬 仅评论


🟡 Medium Priority
本次变更将三个子模块的源地址从 gitee.com/mirrors(Gitee 官方镜像服务)迁移至 gitcode.com 下的非官方命名空间:
- googletest(第 7 行):
gitee.com/mirrors/googletest.git→gitcode.com/GitHub_Trending/go/googletest.git - fmt(第 16 行):
gitee.com/mirrors/fmt.git→gitcode.com/GitHub_Trending/fm/fmt.git - torch-mlir(第 22 行):
gitee.com/mirrors_llvm/torch-mlir.git→gitcode.com/gh_mirrors/to/torch-mlir.git
GitHub_Trending 命名空间暗示这些是自动化抓取的镜像(如 gitcode 平台的 Trending 仓库自动同步),而非经过审核的官方维护镜像。gh_mirrors 同样缺乏明确的官方维护标识。与项目中其他使用 ascend/、cann/、mindspore/ 等明确归属命名空间的子模块不同,这三个新 URL 指向的仓库来源缺乏可验证的权威性。
触发条件:当开发者执行 git submodule update --init 时,将从这些新 URL 拉取代码。如果镜像仓库被篡改或未及时同步上游安全补丁,将引入供应链攻击面。
影响范围:googletest 影响测试框架的完整性;fmt 是 C++ 格式化库,编译进最终产物;torch-mlir 是编译器基础设施,直接影响代码生成。三者均处于关键供应链路径上。
建议:建议确认 gitcode.com/GitHub_Trending 和 gitcode.com/gh_mirrors 是否为 gitcode 平台官方维护的镜像命名空间,并核实这些镜像仓库是否与上游保持同步、未被篡改。如非官方镜像,应迁移至可信的官方源或项目可控的 fork 仓库。同时建议在 SECURITYNOTE.md 中对这些镜像来源做出说明。


| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_LibTorch_ARM | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| patch_test | 🛑 | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| CodeCheck_lintrunner | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | 🛑 | >>> |
| UT_X86_Part_02 | 🛑 | >>> | |
| UT_ARM_A3_Part_01 | 🛑 | >>> | |
| UT_ARM_A3_Part_02 | 🛑 | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| UT_ARM_A2_Part_03 | ✅ | >>> | |
| UT_inductor_Part_01 | 🛑 | >>> | |
| UT_inductor_Part_02 | 🛑 | >>> | |
| UT_inductor_Part_03 | 🛑 | >>> | |
| UT_inductor_Part_04 | 🛑 | >>> | |
| UT_DIST_ARM_Part_01 | 🛑 | >>> | |
| UT_DIST_ARM_Part_02 | 🛑 | >>> | |
| UT_DIST_ARM_Part_03 | 🛑 | >>> | |
| UT_DIST_ARM_Part_04 | 🛑 | >>> | |
| UT_ARM_A2_Select_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Select_Part_02 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ✅ | >>> |


/lgtm


/lgtm


The following users do not have permission to comment /lgtm or /approve on any module in this PR:
lyx324521


/approve


The following users do not have permission to comment /lgtm or /approve on any module in this PR:
lyx324521


Pull Request 已合并或已关闭。
If you want to solve this problem, you can click here to do it in the FAQs.



