| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
【PR】: 压缩so回退 Co-authored-by: guo-yanjun<guoyanjun3@huawei.com> # message auto-generated for no-merge-commit merge: !2761 merge revert_compress_so into master 【PR】: 压缩so回退 Created-by: guo-yanjun Commit-by: guo-yanjun Merged-by: cann-robot Description: # Pull Request ## 描述 压缩aclrt接口调用路径回退 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新(格式化,局部变量) - [x] ♻️ 重构(既不修复错误也不增加功能的代码变动) - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在当前页面的右侧'关联Issue'部分添加相应Issue链接,并勾选'合并后关闭已关联的 Issue'选项。 --> ## 如何测试 描述测试此变更的步骤和前提条件: 1. 2. ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签(如: feat:, fix:) - [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 在此添加任何其他关于本次 PR 的说明。 See merge request: cann/runtime!2761 | 2 个月前 | |
fix: aclAppLog 超长日志改用截断而非丢弃整条日志(#555) Co-authored-by: GuoWenbo<guowenbo13@h-partners.com> # message auto-generated for no-merge-commit merge: !2850 merge fix/issue-555-aclapplog-truncate-910 into 9.1.0 fix: aclAppLog 超长日志改用截断而非丢弃整条日志(#555) Created-by: GuoWenbo Commit-by: GuoWenbo Merged-by: cann-robot Description: # Pull Request ## 描述 修复 issue #555(同步自 master 分支 PR #2780)。 用户通过 ACL_APP_LOG / aclAppLog 打印日志时,若格式化后的字符串总长度超过 1024 字节,会因 vsnprintf_s 在截断时返回 -1 而**丢弃整条日志**,仅输出一句不准确的 aclAppLog call vsnprintf_s failed,用户看不到任何原始内容。 本次变更: - src/acl/aclrt_impl/log.cpp: - vsnprintf_s/sprintf_s 改为 vsnprintf_truncated_s/snprintf_truncated_s,超长时截断保留内容并正常落盘,不再丢弃整条日志。 - 截断场景下在尾部覆盖写入 ...[truncated] 标记;标记连同结尾 \0 始终落在 1024 字节预算内,避免被下游 slog 二次截断。 - 保留格式串非法(含 %n)时的 aclAppLog format string is invalid 错误提示。 - include/external/acl/acl_base_rt.h:补充 aclAppLog 1024 字节长度限制及截断标记说明。 - tests/depends/slog/inc/slog_stub_log_capture.h(新增)+ slog_stub.cpp:以函数接口 DlogStubGetLastLogMsg() 暴露 stub 捕获的日志内容(vsnprintf_s 并检查返回值),支撑截断标记的 UT 断言,规避 G.EXP.05-CPP / G.FUU.11 等规则。 - tests/ut/acl/testcase/acl_common_unittest.cpp:新增截断标记存在、最终落盘总长 < 1024、短日志不带标记、非法格式提示等用例。 ## 变更类型 - [x] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新(格式化,局部变量) - [ ] ♻️ 重构(既不修复错误也不增加功能的代码变动) - [ ] 📦 构建过程或辅助工具的变动 - [x] 📝 文档内容更新 ## 关联的Issue https://gitcode.com/cann/runtime/issues/555 ## 如何测试 1. 构建 acl 单测目标 ascendcl_utest(bash tests/build_ut.sh --ut acl)。 2. 运行 ascendcl_cpp_utest --gtest_filter='*aclAppLog*:*FormatStr*',确认: - 超长日志不再触发 vsnprintf_s failed,按调用 level 落盘; - 超长日志尾部含 ...[truncated] 标记,最终落盘总长 < 1024; - 短日志不带标记;非法格式(%n)输出 format string is invalid; - 正常短日志与 FormatStr 行为不变。 3. 已用 issue #555 最小复现 demo(18 段拼接、1083 字节)端到端验证:修复后输出截断后内容 + ...[truncated],不再丢日志。 ## 核对清单 - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签(如:feat:, fix:) - [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定 ## 其他信息 本 PR 为 master 分支 PR #2780 向 9.1.0 分支的同步,代码改动完全一致。不改变 MAX_LOG_STRING(1024)上限本身,仅影响 ACL APP 日志路径(aclAppLog/aclAppLogWithArgs → aclAppLogImpl)。 See merge request: cann/runtime!2850 | 2 个月前 | |
fix: add new getPhyDevIdByUserDevId and getUserDevIdByPhyDevId interface Co-authored-by: xiao_hhh<guohui37@huawei.com> # message auto-generated for no-merge-commit merge: !3294 merge 9.1.0_merge into 9.1.0 fix: add new getPhyDevIdByUserDevId and getUserDevIdByPhyDevId interface Created-by: xiao_hhh Commit-by: xiao_hhh Merged-by: cann-robot Description: fix: add new getPhyDevIdByUserDevId and getUserDevIdByPhyDevId interface # Pull Request ## 描述 本次 PR 新增两个 ACL Runtime 设备ID转换接口: - aclrtGetUserDevIdByPhyDevId:根据物理设备ID获取对应的用户设备ID - aclrtGetPhyDevIdByUserDevId:根据用户设备ID获取对应的物理设备ID 变更内容: - 在 acl_rt.h 中声明两个新接口 - 在 device.cpp 中实现 aclrtGetUserDevIdByPhyDevIdImpl / aclrtGetPhyDevIdByUserDevIdImpl,复用既有 rtsGetLogicDevIdByPhyDevId / rtsGetPhyDevIdByLogicDevId - 在 acl_rt_wrapper.h 注册接口 wrapper - 在 prof_reporter.h 与 profiling_manager.cpp 新增 profiling 枚举项及映射 - 补充 API 参考文档 - 新增 UT 用例 ## 变更类型 请选择本次引入的变更类型: - [ ] 🐛 Bug 修复 - [x] ✨ 新功能 - [ ] 💄 代码风格更新(格式化,局部变量) - [ ] ♻️ 重构(既不修复错误也不增加功能的代码变动) - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue ## 如何测试 描述测试此变更的步骤和前提条件: 1. 新增 4 个 UT 用例(aclrtGetUserDevIdByPhyDevIdTest、aclrtGetUserDevIdByPhyDevIdNullPtrTest、aclrtGetPhyDevIdByUserDevIdTest、aclrtGetPhyDevIdByUserDevIdNullPtrTest),覆盖正常路径、runtime 失败返回、空指针入参三类分支 2. 运行 UTEST_ACL_Runtime 全量测试套(587 个用例)全部通过,新增接口对已有功能无影响 ## 核对清单 - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签(如:feat:, fix:) - [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 无 See merge request: cann/runtime!3294 | 2 个月前 | |
Initial commit | 8 个月前 | |
Initial commit | 8 个月前 | |
[refactor] add runtime code review workflow and guidelines Co-authored-by: wangtao43<wangtao43@huawei.com> # message auto-generated for no-merge-commit merge: !2394 merge feature/runtime-code-review-guidelines into master [refactor] add runtime code review workflow and guidelines Created-by: wangtao43 Commit-by: wangtao43 Merged-by: cann-robot Description: ## 变更概要 - 新增 runtime-code-review 统一代码审查能力,支持本地 diff 与 GitCode PR 审查 - 增加 PR 审查辅助脚本,包括 PR 元信息获取、文件分类、上下文聚合、summary 渲染和评论发布 - 新增并整理 docs/guidelines 下的设计模板、编码规范、UT 代码规范和 DT 指导文档 - 更新 build.sh 与 tests/build_ut.sh,对 docs/guidelines 变更跳过构建 ## 验证 - python3 -m py_compile .claude/skills/runtime-code-review/scripts/*.py - bash -n build.sh - bash -n tests/build_ut.sh - bash -n .claude/skills/default-skills/scripts/install-default-skills.sh - 已对 GitCode PR 2358 / 2362 进行 runtime-code-review 联调验证(前者跑通上下文准备链路,后者正确被 closed 状态跳过) ## 影响范围 - .claude/skills/runtime-code-review/** - docs/guidelines/** - build.sh - tests/build_ut.sh - docs/README.md - .opencode/README.md See merge request: cann/runtime!2394 | 3 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 2 个月前 | ||
| 2 个月前 | ||
| 2 个月前 | ||
| 8 个月前 | ||
| 8 个月前 | ||
| 3 个月前 |