Pull Request已成功合入, 合并人@CANN-robot
（感谢 fanhong 的贡献）
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 |
|---|---|---|
| src/dfx | ✅ zhuliangying, chenhao_1209 (2/2) | ✅ zhuliangying (1/1) |
| tests/ut/msprof | ✅ zhuliangying, chenhao_1209 (2/2) | ✅ zhuliangying (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
fanhong, thanks for your pull request. All authors of the commits have signed the CLA. 👍
CANN-robot
compile
流水线任务触发成功
任务链接 [2ef27b09181b4060bfa3f2bd38e26a5a][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| 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 | >>>>> | |
| Compile_Ascend_ARM_ubuntu24 | ✅ SUCCESS | >>>>> | |
| UT_Test_acl | ❌ FAILED | >>>>> | |
| UT_Test_rts | ❌ FAILED | >>>>> | |
| 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 | >>>>> |
compile
流水线任务触发成功
任务链接 [da688076e31a4d06888516cf012520de][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| 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 | >>>>> | |
| Compile_Ascend_ARM_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-17 15:58:23] CI执行结束
🤖 CANN 代码审查报告
PR: #950 - [PROFILING]change sys-lp switch default value
严重性: ✅ Low
审查时间: 2026-03-17 16:05
📊 审查结论
✅ 建议合入
- 严重性: Low
- 代码质量: 优秀
- 内存安全: ✅ 无风险
- 安全性: ✅ 无漏洞
- 测试覆盖: 完整
- 文档完整性: 完整
这是一个简单的配置默认值修改，代码质量优秀，可以合入。
📋 修改概述
将 profiling 的 sys-lp（低功耗数据采集）参数默认值从 "off" 改为 "on"，帮助用户更好地定位性能问题是否由降频引起。
- 修改文件: 5个 (+9行, -4行)
- 核心变更:
prof_params.h: 将 sysLp 默认值从 "off" 改为 "on"input_parser_closed.cpp: 更新参数说明，默认值改为 on- 3个测试文件: 更新测试用例以适应新默认值
🔍 代码质量检查
1. 内存安全 ✅
- 内存泄漏: 无风险（仅字符串默认值修改）
- 指针操作: 不涉及
- 动态分配: 不涉及
- 资源管理: 不涉及
2. 安全性 ✅
- 输入验证: 不涉及
- 边界检查: 不涉及
- 潜在漏洞: 无
3. 可读性 ✅
- 代码清晰度: 优秀
- 命名规范: 符合项目规范
- 注释完整性: 完整（更新了参数说明）
4. 逻辑正确性 ✅
- 算法逻辑: 正确
- 边界条件: 已处理（测试用例已更新）
- 影响范围: 明确（仅影响默认配置）
💡 改进建议
无需改进，代码质量优秀。
✅ 代码亮点
- ✅ 测试覆盖完整: 同步更新了所有相关测试用例
- ✅ 文档同步: 参数说明文档已同步更新
- ✅ 变更最小化: 仅修改必要的默认值，影响范围明确
- ✅ 向后兼容: 用户仍可手动关闭此功能
总体评价: 这是一个高质量的配置修改 PR，代码清晰、测试完整、文档同步，建议合入。
/approve
/lgtm
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: zhuliangying.
Reviewers who wrote a comment of /lgtm are: zhuliangying, chenhao_1209.
描述
原因：torch_npu.profiler等方式下未默认采集低功耗数据，影响用户定位性能问题是否是降频引起的
修改：经过对齐，默认打开低功耗数据采集能力，用户可手动关闭
关联的Issue
测试
文档更新
类型标签