已合并
[fix] stop packaging HCCL internal headers #5402
[fix] stop packaging HCCL internal headers #5402
已合并
izanaami创建于 8月27日
izanaami
izanaami成员
8月27日

修改内容

  • 移除 asc-devkit 对 internal/hccl.h、internal/hccl_mc2.h 的安装规则和打包声明
  • 保留源码头文件供 asc-devkit 自身编译使用
  • 保留 hccl_msg.h、hccl_tiling_msg.h 及 include/version 的现有打包行为

问题说明

这两个内部头文件与 HCCL 包安装到相同目标路径,可能在安装 asc-devkit 后覆盖 HCCL 提供的头文件。两仓版本不一致时会引入编译或兼容性风险。

验证

  • git diff --check 通过
  • AscHcclLib.xml XML 解析通过
  • A5_4p 使用 CANN 9.2.0 执行 bash build.sh --pkg -j32,编译打包通过
  • 解包 cann-asc-devkit_9.2.0_linux-x86_64.run,确认冲突头文件和对应安装清单不存在
  • 确认包内 adv_api/hccl/internal 仍包含 hccl_msg.h、hccl_tiling_msg.h

未实际安装 .run,未执行 NPU 运行时用例。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 izanaami 的贡献)
izanaamiizanaami成员
8月27日 创建了 pull request,commit 321526e7
atomgit-bot
atomgit-bot
8月27日 评论:

变更摘要

本 PR 旨在修复 asc-devkit 打包时误将 HCCL 内部头文件一起安装的问题。由于 internal/hccl.h、internal/hccl_mc2.h 与 HCCL 包安装到相同目标路径,安装 asc-devkit 后可能覆盖 HCCL 提供的同名头文件,两仓版本不一致时会产生编译或兼容性风险。该改动主要聚焦于移除 asc-devkit 对这两个内部头文件的安装规则与打包声明,同时保留源码头文件供 asc-devkit 自身编译使用,并保持 hccl_msg.h、hccl_tiling_msg.h 及 include/version 的现有打包行为不变。

主要改动

  • 移除 include/hccl 目录安装声明: 在 AscHcclLib.xml 的 dir_info 中删除 <path value="$(TARGET_ENV)/include/hccl" install_type="devel"/>,不再将该目录作为 devel 类型安装项打包。
  • 移除内部头文件打包规则: 删除对应的 file_info 配置项,该配置原会将 hccl.h、hccl_mc2.h 从 $(TARGET_ENV)/include/hccl 拷贝并安装到目标 include/hccl 路径,从而避免覆盖 HCCL 包提供的头文件。
  • 保留既有打包行为: hccl_msg.h、hccl_tiling_msg.h(adv_api/hccl/internal)与 include/version 的安装/打包逻辑不受影响,继续随包发布。
likedislike
不准确?
atomgit-bot
atomgit-bot
8月27日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
8月27日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
8月27日 评论:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/asc-devkit ✅ YeZZzzz1, 石楠翔, 诸葛洵 (3/2) ✅ 诸葛洵, 石楠翔 (2/1)
scripts ✅ 诸葛洵, YeZZzzz1 (2/2) ✅ 诸葛洵 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

jiangxinyu3, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robotCANN-robot成员
8月27日 将zengjuan,zhanghao_0689,suqwe,kevin365,wulinyu,kong0808,dengjing_aoe,YeZZzzz1,bluesky901,yangbinhua,zhudi22,chenyiyuan,xun_zhuge,XuebinYang,sjtulxh,bianque,houyuyang,Lycheeeee,YANXI_ZHAO,chentianyu19,shi_nanxiang,wuyang_hw,ascendhjh,hujiawen_kaven,wangxiaozhi,DragonBornHD84,louyuxuan,hyolee,yukangchen设为评审人
CANN-robotCANN-robot成员
8月27日 将zengjuan,zhanghao_0689,suqwe,kevin365,wulinyu,kong0808,dengjing_aoe,YeZZzzz1,bluesky901,yangbinhua,zhudi22,chenyiyuan,xun_zhuge,XuebinYang,sjtulxh,bianque,houyuyang,Lycheeeee,YANXI_ZHAO,chentianyu19,shi_nanxiang,wuyang_hw,ascendhjh,hujiawen_kaven,wangxiaozhi,DragonBornHD84,louyuxuan,hyolee,yukangchen设为审查人
gao_dafa成员
8月27日 评论:

jiangxinyu3,感谢您的提交,建议在完成PR充分自验证后,再触发线上构建任务。

若您的PR已具备检视条件,可根据下表的责任田,@对应的committer进行代码检视,我们会在第一时间对您的提交进行处理。

代码责任田:

module committer
adv_api chenyiyuan, wuyang_hw
basic_api chenyiyuan, wuyang_hw
c_api pangfd, haipingmao
cmake xun_zhuge, DragonBornHD84
docs munanhw, ai_xin
examples changxianyu, bianque
simt_api wulinyu, zhanghao_0689
scripts xun_zhuge, kevin365
tools YANXI_ZHAO, XuebinYang, sjtulxh

💡 如您不确定可以@谁,可以直接@我或者ascend-c sig的maintainer,xun_zhuge和ascendhjh,我们会为您分配合适的committer进行代码检视。

再次感谢您参与CANN的社区贡献。

likedislike
izanaami
izanaami成员
8月27日 评论:

compile

likedislike
izanaamiizanaami成员
8月27日 预合并成功(commit_id: dea2ce3e35c419c19877338b503f0f65d1466338)
CANN-robotCANN-robot成员
8月27日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
8月27日 评论:

流水线任务触发成功
任务链接 [54f22f0277be4df1a1f08dae21b298c3][流水线指导]

任务名称状态日志下载链接
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck ✅ SUCCESS >>>>>
codecheck_Pr ✅ SUCCESS
pre_comment ✅ SUCCESS >>>>>
codecheck_codestyle ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_md ✅ SUCCESS

[2026-08-27 14:54:03]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
8月27日 评论:

流水线任务触发成功
任务链接 [fde92d38138a46618393f7863b3c4a99][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_adv_test_one ✅ SUCCESS
UT_Test_adv_test_two ✅ SUCCESS
UT_Test_basic_test_one ✅ SUCCESS
UT_Test_basic_test_two ✅ SUCCESS
UT_Test_basic_test_three ✅ SUCCESS
UT_Test_basic_test_four ✅ SUCCESS
UT_Test_basic_test_five ✅ SUCCESS
UT_Test_arm ✅ SUCCESS
API_Check ✅ SUCCESS >>>>>
UT_Test_report ✅ SUCCESS >>>>>

[2026-08-27 15:06:33]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
8月27日 评论:

流水线任务触发成功
任务链接 [31e096bfe9584a688b81537238be3981][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-08-27 15:03:44]    CI执行结束

likedislike
CANN-robotCANN-robot成员
8月27日 添加了label:api-check-pass
CANN-robotCANN-robot成员
8月27日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
8月27日 添加了label:ci-pipeline-passed
YeZZzzz1成员
8月28日 评论:

/lgtm

likedislike
shi_nanxiang成员
8月28日 评论:

/approve

likedislike
zhugexun_sjtu
zhugexun_sjtu成员
8月29日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
8月29日 添加了label:lgtmapproved
CANN-robotCANN-robot成员
8月29日 合入了pull request