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-cann/runtime	✅ ykl999, newstarzj (2/2)	✅ ykl999 (1/1)

💡 Tip:

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

🤖 CANN 自动审查报告

PR: #650 - add ascend_kms
作者: duanpengliang
时间: 2026-03-05 09:50
审查者: CANN 自动审查系统

---

📊 变更概览

• 修改文件: 1个
• 新增代码: 1行
• 删除代码: 0行
• 变更类型: 打包配置更新

修改内容: 在 scripts/package/module/ascend/RuntimeSo.xml 中添加库文件 libascend_kms.o

---

✅ 优点

1. 变更最小化 - 仅添加1行配置，风险极低
2. 可选配置 - 使用 optional=\"true\"\，不影响现有功能
3. 格式规范 - XML格式正确，缩进一致
4. CLA已签署 - 法律合规性通过

---

⚠️ 需要关注 (Low)

1. PR描述为空 - 建议补充 ascend_kms 的功能说明
2. 缺少测试信息 - 建议添加测试验证结果
3. 文件扩展名 - .o 通常表示目标文件，需确认类型是否正确

---

🔍 技术分析

配置解析:

<file value=\"libascend_kms.o\" file_type=\"shared\" optional=\"true\"/>

• 文件类型: shared library
• 可选性: 是（文件缺失不会导致打包失败）
• 安装路径: `$`(TARGET_ENV)/devlib/device

风险评估: 低风险（可选配置，容错性好）

---

💡 最终建议

建议合入 ✅

理由:

• 变更极小且安全
• 使用 optional 标志，容错性好
• 遵循现有配置模式
• CLA已签署

合入后建议:

1. 补充 PR 描述，说明 ascend_kms 功能
2. 确认 .o 文件扩展名的合理性
3. 在合入说明中标注为打包配置更新

---

此审查报告由 CANN 自动审查系统生成
审查时间: 2026-03-05 09:50

CLA Signature Pass

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

compile

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

任务名称	状态	日志	下载链接
codecheck	✅ SUCCESS	>>>>>
SCA	✅ SUCCESS	>>>>>
anti_virus	✅ SUCCESS	>>>>>
Check_Pr	✅ SUCCESS	>>>>>
Compile_Ascend_X86	✅ SUCCESS	>>>>>	>>>>>
Compile_Ascend_ARM	✅ SUCCESS	>>>>>	>>>>>
pre_comment	✅ SUCCESS	>>>>>
Compile_Ascend_X86_ubuntu24	✅ SUCCESS	>>>>>
UT_Test_acl	✅ SUCCESS		>>>>>
UT_Test_rts	✅ SUCCESS		>>>>>
UT_Test_rts_c	✅ SUCCESS		>>>>>
UT_Test_platform	✅ SUCCESS		>>>>>
UT_Test_qs	✅ SUCCESS		>>>>>
UT_Test_aicpusd	✅ SUCCESS		>>>>>
UT_Test_tsd	✅ SUCCESS		>>>>>
UT_Test_dfx	✅ SUCCESS		>>>>>
API_Check	✅ SUCCESS	>>>>>
UT_Test_mmpa	✅ SUCCESS		>>>>>

[2026-03-05 11:24:09]    CI执行结束

/approve

🤖 自动审查报告

📋 基本信息

• PR: #650 - add ascend_kms
• 作者: @duanpengliang
• 审查时间: 2026-03-05 14:14
• 严重性: ✅ Low

---

✅ 审查结论: 建议合入

---

📊 变更概览

修改范围: 极小

• 修改文件: 1 个
• 代码变更: +1 -0 (净增 1 行)

变更内容:

• 文件: scripts/package/module/ascend/RuntimeSo.xml
• 添加: libascend_kms.so 到运行时共享库列表

---

✅ 优点

1. 变更极小且明确 ✅

   • 仅 1 个文件，1 行新增
   • 风险极低，易于验证

2. 质量检查全部通过 ✅

   • CI Pipeline: ✅ passed
   • API Check: ✅ passed
   • CLA: ✅ signed
   • Review: ✅ approved

3. 配置规范 ✅

   • 使用 optional="true" 标记
   • 格式与现有条目一致
   • 不影响现有功能

---

⚠️ 建议改进 (非阻塞)

1. PR 描述缺失

当前状态: 只有模板内容

建议补充:

• libascend_kms.so 的用途和功能
• 添加这个库的背景和原因
• 相关的 Issue 或文档链接

示例:

## 描述
添加 ascend_kms 库到运行时打包配置中。

## 背景
libascend_kms.so 是用于 [功能描述] 的库，用于支持 [具体用途]。

## 测试
- [x] 已验证库文件存在
- [x] 已验证打包流程正常

2. KMS 含义说明

问题: "KMS" 可能有多种含义（Key Management Service? 其他？）

建议: 在描述或 commit message 中说明

3. 测试验证

建议:

• 验证打包流程包含该库
• 确认安装流程正常
• 测试运行时加载（如适用）

---

🔍 技术分析

变更详情

文件: scripts/package/module/ascend/RuntimeSo.xml

修改:

<file value="libascend_kms.so" file_type="shared" optional="true"/>

属性说明:

• value="libascend_kms.so": 库文件名
• file_type="shared": 共享库类型
• optional="true": 可选依赖（缺失不影响安装）

影响范围

• ✅ 构建: 自动包含新库到打包流程
• ✅ 安装: 作为可选组件，不影响现有安装
• ✅ 运行: 按需加载，不影响现有功能
• ✅ 兼容: 完全向后兼容

---

📝 最终建议

✅ 可以合入

理由:

1. 变更极小，风险极低
2. 所有质量检查通过
3. 已获得批准
4. 配置规范，向后兼容

建议合入后:

• 监控打包流程
• 验证库正确包含
• 如需要，补充文档说明

---

🤖 此评论由自动审查系统生成
审查时间: 2026-03-05 14:14 (Asia/Shanghai)
审查模型: zai/glm-5

🤖 CANN 代码审查报告

PR: #650 - add ascend_kms
严重性: ✅ Low
审查时间: 2026-03-06 13:05

---

📊 审查结论

✅ 建议合入

• 严重性: Low
• 代码质量: 良好
• 内存安全: ✅ 无风险
• 安全性: ✅ 无漏洞
• 测试覆盖: N/A (配置文件)
• 文档完整性: N/A

配置文件修改，添加新的共享库到打包列表，风险极低。

---

📋 修改概述

本次 PR 添加 libascend_kms.so 到 Runtime 打包配置中。

• 修改文件: 1个 (+1行, -0行)
• 核心变更:
  • scripts/package/module/ascend/RuntimeSo.xml: 添加 libascend_kms.so 到 file_info 列表

---

🔍 代码质量检查

1. 内存安全 ✅

• 内存泄漏: 无风险（配置文件）
• 指针操作: N/A
• 动态分配: N/A
• 资源管理: N/A

2. 安全性 ✅

• 输入验证: N/A
• 边界检查: N/A
• 潜在漏洞: 无

3. 可读性 ✅

• 代码清晰度: 优秀
• 命名规范: 符合
• 注释完整性: N/A

4. 逻辑正确性 ✅

• 算法逻辑: N/A
• 边界条件: N/A
• 影响范围: 明确（仅影响打包配置）

---

✅ 代码亮点

• 使用 optional="true" 标记，允许库文件不存在时不影响打包
• 遵循现有配置格式，保持一致性

---

总体评价: 简单的配置文件更新，添加新的共享库到打包列表，无安全风险，建议合入。

/lgtm

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: ykl999.
Reviewers who wrote a comment of /lgtm are: ykl999, newstarzj.