已合并
【PR】: 删除short soc version的枚举值 #3175
【PR】: 删除short soc version的枚举值 #3175
已合并
jiang-mingming01创建于 6月26日
6月26日

Pull Request

描述

经过讨论,short soc verion是内部使用的,不对外暴露。对外没有这个概念,对外可以查NPU ARCH。 现在产品以及算子以前使用仓间接口获取short soc verion的都正在计划整改了。后续就不依赖了。
此 ABI 不兼容变更;因为功能还没发布,没有调用方,不影响兼容性

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:
1.无
2.

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 jiang-mingming01 的贡献)
Jjiang-mingming01
6月26日 添加了label:enhancement
Jjiang-mingming01
6月26日 创建了 pull request,commit 27cc5eed
CANN-robotCANN-robot成员
6月26日 添加了label:cann-cla/yes
jiang-mingming01
6月26日 评论:

compile

likedislike
CANN-robot
CANN-robot成员
6月26日 评论:

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
docs 王涛, 张子菁 (2/2) 王涛, 张子菁 (2/1)
include 王涛, 张子菁 (2/2) 王涛, 张子菁 (2/1)
src/platform shikai-1, 王涛 (2/2) shikai-1 (1/1)
tests/ut/platform shikai-1, 王涛 (2/2) shikai-1 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
Jjiang-mingming01
6月26日 update merge request[project id: 8810053, iid: 3175, commit_id: f508946f62b303bbd1fa3f150c057e2f5700e144] virtual merging success
CANN-robotCANN-robot成员
6月26日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
6月26日 评论:

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_acl ✅ SUCCESS >>>>>
UT_Test_rts_v201 ✅ SUCCESS >>>>>
UT_Test_rts_c ✅ SUCCESS >>>>>
UT_Test_rts_david ✅ SUCCESS >>>>>
UT_Test_rts_910b ✅ SUCCESS >>>>>
UT_Test_rts_common ✅ SUCCESS >>>>>
UT_Test_platform ✅ SUCCESS >>>>>
UT_Test_qs ✅ SUCCESS >>>>>
UT_Test_aicpusd ✅ SUCCESS >>>>>
UT_Test_tsd ✅ SUCCESS >>>>>
UT_Test_error_manager ✅ SUCCESS >>>>>
UT_Test_slog ✅ SUCCESS >>>>>
UT_Test_atrace ✅ SUCCESS >>>>>
UT_Test_msprof_part1 ✅ SUCCESS
UT_Test_msprof_part2 ✅ SUCCESS
UT_Test_adump ✅ SUCCESS >>>>>
UT_Test_mmpa ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>
UT_Test_rts_report ✅ SUCCESS >>>>>
UT_Test_msprof_report ✅ SUCCESS

[2026-06-26 17:18:18]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6月26日 评论:

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Check_Pr ✅ SUCCESS
codecheck_style ✅ SUCCESS >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-06-26 17:16:53]    CI执行结束

likedislike
CANN-robotCANN-robot成员
6月26日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
6月26日 添加了label:ci-pipeline-passed
atomgit-bot
atomgit-bot
6月26日 评论:

变更摘要

此 PR 从对外公开接口中删除了 short SOC version(短 SoC 版本)的枚举值及相关映射。short SOC version 属于内部概念,不应对用户暴露;对外查询 NPU 架构应使用 ACL_PLATFORM_NPU_ARCH。同时新增了 aclplatformNpuArch 枚举(定义已知 NPU 架构常量,如 DAV_1001 等)和 aclplatformLocalMemType 枚举(本地内存类型映射),并在 kDevInfoTable 中补充了 ACL_PLATFORM_NPU_ARCHACL_PLATFORM_MEMORY_TYPE 的映射条目。对应的单元测试用例也同步删除。

主要改动

  • 移除 short SOC version 枚举值:在 include/external/acl/acl_platform.haclplatformDevInfo 枚举中删除了 short SOC version 对应的枚举项,该值仅内部使用,不对外暴露。
  • 新增 aclplatformNpuArch 枚举:在 acl_platform.h 中新增了 aclplatformNpuArch 枚举,定义 DAV_1001DAV_RESV 等 NPU 架构常量,作为对外查询 NPU 架构的标准化方式。
  • 更新 kDevInfoTable 映射表:在 src/platform/acl_platform.cppkDevInfoTable 中删除了 short SOC version 的映射行,新增了 ACL_PLATFORM_NPU_ARCH(映射到 [version] NpuArch)和 ACL_PLATFORM_MEMORY_TYPE(映射到 [SoCInfo] memory_type)条目。
  • 新增 aclplatformLocalMemType 枚举:在 acl_platform.h 中定义了 L0_AHBM 的本地内存类型枚举,并附带与 aclplatformDevInfo 的对应关系注释。
  • 删除相关单元测试:在 tests/ut/platform/ut/testcase/platform_manager_acl_utest.cpp 中移除了 short SOC version 对应的测试用例。
likedislike
atomgit-bot
atomgit-bot
6月26日 评论:

代码审查

✅ 未发现问题

likedislike
shikai成员
6月27日 评论:

/lgtm
/approve

likedislike
wangtao成员
6月27日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
6月27日 添加了label:approved
yanmingxiang成员6月27日进行代码检视2
include/external/acl/acl_platform.h
@@ -74,3 +89,3 @@
7489// HBM (6) → ACL_PLATFORM_MEMORY_SIZE (5)
7590// --------------------------------------------------------------------------
76-enum class aclplatformLocalMemType {
91+typedef enum aclplatformLocalMemType {
yanmingxiang6月27日评论:

这个已经开出去了,直接修改会不会有兼容性问题?

likedislike
jiang-mingming01
6月27日 评论:

初版版本还没有发布,暂时不影响

李俊谕成员
6月27日 解决了最后一个问题
tingwood成员
6月27日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
6月27日 添加了label:lgtm
CANN-robotCANN-robot成员
6月27日 合入了pull request