| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
fix:clean code Co-authored-by: liu-lu<www.liulu824910939@qq.com> # message auto-generated for no-merge-commit merge: !4869 merge master into master fix:clean code Created-by: liu-lu Commit-by: liu-lu Merged-by: cann-robot Description: # Pull Request ## 描述 本 PR 对 tsd 模块进行头文件依赖清理(clean code),主要变更如下: 1. **替换宽口径头文件**:将 #include "driver/ascend_hal.h" 替换为更精确的子模块头文件 - device_comm_agent.h:替换为 #include "driver/ascend_hal_base.h" - stub_dc.cpp:替换为 #include "driver/ascend_hal_base.h" + #include "driver/ascend_hal_error.h" 2. **移除未使用的 include**: - package_env_info.h:移除未使用的 #include "driver/ascend_hal.h" - package_manager.h:移除未使用的 #include "driver/ascend_hal.h" - package_env_info.cpp:移除未使用的 #include "driver/ascend_hal.h" 和 #include "error_manager.h" 3. **显式声明依赖**: - weak_ascend_hal.h:新增 #include "driver/ascend_hal_base.h" 和 #include "driver/ascend_hal_error.h" ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新(格式化,局部变量) - [x] ♻️ 重构(既不修复错误也不增加功能的代码变动) - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在当前页面的右侧'关联Issue'部分添加相应Issue链接,并勾选'合并后关闭已关联的 Issue'选项。 --> #965 ## 如何测试 描述测试此变更的步骤和前提条件: 1. 执行全量编译构建,验证头文件替换后编译无报错 2. 运行 tsd 模块相关单元测试,确认功能无回归 3. 验证 device_comm、package_manager 模块加载和初始化流程正常 ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签(如:feat:, fix:) - [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 本次变更仅涉及头文件 include 调整,不涉及任何功能逻辑变更。目标是降低头文件依赖耦合度,提升编译效率。 See merge request: cann/runtime!4869 | 18 天前 | |
fix:clean code Co-authored-by: liu-lu<www.liulu824910939@qq.com> # message auto-generated for no-merge-commit merge: !4869 merge master into master fix:clean code Created-by: liu-lu Commit-by: liu-lu Merged-by: cann-robot Description: # Pull Request ## 描述 本 PR 对 tsd 模块进行头文件依赖清理(clean code),主要变更如下: 1. **替换宽口径头文件**:将 #include "driver/ascend_hal.h" 替换为更精确的子模块头文件 - device_comm_agent.h:替换为 #include "driver/ascend_hal_base.h" - stub_dc.cpp:替换为 #include "driver/ascend_hal_base.h" + #include "driver/ascend_hal_error.h" 2. **移除未使用的 include**: - package_env_info.h:移除未使用的 #include "driver/ascend_hal.h" - package_manager.h:移除未使用的 #include "driver/ascend_hal.h" - package_env_info.cpp:移除未使用的 #include "driver/ascend_hal.h" 和 #include "error_manager.h" 3. **显式声明依赖**: - weak_ascend_hal.h:新增 #include "driver/ascend_hal_base.h" 和 #include "driver/ascend_hal_error.h" ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新(格式化,局部变量) - [x] ♻️ 重构(既不修复错误也不增加功能的代码变动) - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在当前页面的右侧'关联Issue'部分添加相应Issue链接,并勾选'合并后关闭已关联的 Issue'选项。 --> #965 ## 如何测试 描述测试此变更的步骤和前提条件: 1. 执行全量编译构建,验证头文件替换后编译无报错 2. 运行 tsd 模块相关单元测试,确认功能无回归 3. 验证 device_comm、package_manager 模块加载和初始化流程正常 ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签(如:feat:, fix:) - [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 本次变更仅涉及头文件 include 调整,不涉及任何功能逻辑变更。目标是降低头文件依赖耦合度,提升编译效率。 See merge request: cann/runtime!4869 | 18 天前 | |
fix: 优化 ERROR 日志枚举名称输出(#926) Co-authored-by: zhaowenrui666<zhaowenrui7@huawei.com> # message auto-generated for no-merge-commit merge: !4680 merge fix/enum-error-log-names into master fix: 优化 ERROR 日志枚举名称输出(#926) Created-by: zhaowenrui666 Commit-by: zhaowenrui666 Merged-by: cann-robot Description: # Pull Request ## 描述 部分 ERROR 日志仅输出枚举数值,定位问题时需要额外查询枚举定义,影响日志可读性和故障分析效率。本次统一优化相关日志:存在有效枚举名称时输出 枚举名称(原始值),同时保留原始数值用于识别异常输入。 主要变更如下: 1. 为 AICPU Schedule、AICPU Sharder、Queue Schedule 和 TSD 涉及的枚举补充名称转换能力。 2. 新增类型安全的 EnumNameTable Traits 注册表和统一 GetEnumName 查询接口;14 个自定义枚举由 switch 转为按枚举类型注册的静态 map,后续新增枚举只需补充对应特化。 3. 枚举名称转换接口统一返回 std::string,日志调用处通过 c_str() 传递字符串,不引入裸字符指针接口;未注册的枚举值统一返回 UNKNOWN。 4. protobuf 枚举优先复用生成的名称转换接口,避免重复维护映射关系;BQS 消息类型保留 UNKNOWN(原始值) 兜底。 5. SOMA、Dump 和 HCCL Protocol 等明确非法且无法获得有效名称的场景仅输出原始数值,避免打印无意义的枚举名称。 6. 补充 AICPU Sharder 构建及 UT/ST 的公共头文件搜索路径。 7. 精简 DgwClient::CheckConfigNum() 的冗余作用域和末尾 break,并合并 BqsServer::HandleBqsReqMsg() 的相邻日志字符串,使两个函数的 NBNC 均降至 50,不改变控制流及日志内容。 8. PR 变更已整理为单个有效提交。 本次仅整改 ERROR 级别的枚举日志,不改变业务流程、返回值及错误处理逻辑,也不扩大到协议字段等非枚举数值。 ## 变更类型 - [x] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新(格式化,局部变量) - [ ] ♻️ 重构(既不修复错误也不增加功能的代码变动) - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue 关联 Issue:https://gitcode.com/cann/runtime/issues/926 ## 如何测试 1. 对本次提交涉及的文件执行 pre-commit,检查通过。 2. tsd_client_utest、queue_schedule_main_ut、aicpu_sharder_utest 和 aicpu_aicpuschedule_utest 均编译通过。 3. 注册查表改造前新增失败用例,确认统一查询接口尚不存在;改造后该用例及未知值兜底检查通过。 4. 完整回归:AICPU Schedule 1480 个、AICPU Sharder 74 个、Queue Schedule 685 个、TSD 881 个用例均通过。 5. 使用 AICPU Sharder 的实际头文件和宏配置,以 C++11 标准对 aicpu_context.cc 执行语法编译检查,通过。 6. 移动仓内公共头后,生产目标 host_queue_schedule 的全部对象编译及可执行文件链接通过。 7. NBNC 调整后,DgwClient::CheckConfigNum() 和 BqsServer::HandleBqsReqMsg() 的 NBNC 均为 50。 ## 核对清单 - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档(本次变更不涉及用户文档,无需更新) - [x] 我在标题中使用了合适的类型标签(如:feat:, fix:) - [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 本次变更范围限定为 AICPU Schedule、AICPU Sharder、Queue Schedule 和 TSD 中的 ERROR 级别枚举日志,不涉及业务流程、返回值和资源所有权变更。 See merge request: cann/runtime!4680 | 23 天前 | |
feat:aicpu日志整改 Co-authored-by: liu-lu<www.liulu824910939@qq.com> # message auto-generated for no-merge-commit merge: !4513 merge master into master feat:aicpu日志整改 Created-by: liu-lu Commit-by: liu-lu Merged-by: cann-robot Description: ## 描述 本次 PR 对 runtime 源码中的日志文本进行规范化,并补充 UT 测试用例确保覆盖率达标: 1. **日志文本规范化**(56 个源文件):修正语法错误(如 has exist → already exists)、添加参数单位(如 us、ms)、展开缩写(如 f2nf → full-to-notfull event)、补充空日志内容(如 TSD_ERROR("") → TSD_ERROR("insert package config to map failed")) 2. **clang-format 格式修复**(6 个源文件):修复超过 ColumnLimit:120 的超长日志行换行 3. **UT 测试用例补充**(12 个测试文件,新增约 20 个测试用例):覆盖修改的日志行所在的条件分支和异常路径,使 TSD UT 和 AICPU SD UT 增量覆盖率均达到 80%+ ## 变更类型 请选择本次代码的变更类型: - [x] ✨ 新功能(日志描述规范化) - [ ] 🐛 Bug 修复 - [ ] ♻️ 性能优化(正式格式、命名规范、代码整理等不影响功能也不会添加功能的代码变动) - [ ] ♻️ 重构等不涉及修改对外接口的代码变动 - [x] 🧪 测试(引入测试流程或工具的变动) - [ ] 📦 文档内容更新 ## 关联的Issue #891 ## 如何测试 请描述你打算如何测试此次变更: 1. **编译验证**:执行 bash tests/build_ut.sh --ut=aicpusd --target=aicpu_sched_ut 和 TSD UT 编译,确认无编译错误 2. **UT 覆盖率验证**:执行 UT 并检查增量覆盖率 ≥ 80%(覆盖路径包括:路径超长、设备非空闲、参数超限、线程创建失败、文件过大、SO 名称非法、环境变量开关、Buffer 初始化失败、Profiling 响应失败、内存分配失败) 3. **clang-format 验证**:clang-format --dry-run --Werror 确认所有修改文件格式合规 4. **功能验证**:UT 测试全部通过,无回归问题 ## 确认清单 - [x] 我的代码已遵循项目的代码规范 - [x] 已完成对所有代码的测试 - [x] 已更新所有相关的文档 - [x] 我已确保使用了合适的类型和标签,如:feat:,fix: - [x] 我已经仔细阅读了贡献指南(CONTRIBUTING.md),并理解了其中的所有规定,包括对提交commit message的格式和有效commit的合并) ## 附加信息 - 源文件修改仅涉及日志文本(aicpusd_info/aicpusd_err/TSD_RUN_INFO/TSD_ERROR 等日志宏的格式字符串),不改变任何代码逻辑 - UT 测试用例通过 MOCKER(pthread_create) mock 线程创建失败、MOCKER(halBuffInit) mock buffer 初始化失败等方式覆盖异常路径 See merge request: cann/runtime!4513 | 1 个月前 | |
【feat】:tsdclient source list变量抽取 Co-authored-by: sujunwei3<sujunwei3@huawei.com> # message auto-generated for no-merge-commit merge: !4293 merge new_dev into master 【feat】:tsdclient source list变量抽取 Created-by: sujunwei3 Commit-by: sujunwei3 Merged-by: cann-robot Description: # Pull Request ## 描述 本 PR 对 tsdclient 构建文件和外部依赖桩机制做了三项改进: 1. **CMakeLists.txt 拆分**:将 tsdclient 的源文件列表和头文件包含目录从 CMakeLists.txt 内联写法抽取到独立的 tsdclient_sources.cmake 文件。CMakeLists.txt 通过 include 引用,使构建文件更简洁,源文件列表可被 UT 等其他目标复用。 2. **新增 stub_external_deps.cpp 桩文件**:为 fe::PlatformInfoManager 和 fe::PlatFormInfos::GetPlatformRes 提供空实现占位符号,用于"没有 platform target 可以链接"的场景。桩实现返回 0/false,保证链接通过但无实际行为。 3. **新增 TSD_CHECK_RET_VOID 宏 + 桩文件 UT**:补齐 TSD 错误检查宏的 void 返回分支;新增 stub_external_deps_utest UT target,不链接 platform 库,直接编译桩文件,固化桩的返回值契约(单例一致性、返回 0/false、不修改出参)。 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新(格式化,局部变量) - [x] ♻️ 重构(既不修复错误也不增加功能的代码变动) - [x] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在当前页面的右侧'关联Issue'部分添加相应Issue链接,并勾选'合并后关闭已关联的 Issue'选项。 --> #874 ## 如何测试 根据代码变更,测试场景如下: 1. **编译构建测试** - 执行整包编译,验证 tsdclient 共享库正确构建 - 验证 tsdclient_sources.cmake 中的源文件列表与重构前完全一致(无遗漏、无多余) 2. **CMake 配置验证** - 确认 tsdclient/CMakeLists.txt 正确 include tsdclient_sources.cmake - 验证 tsdclient_all_sources 和 tsdclient_include_dirs 变量定义正确 3. **桩文件 UT 验证** - 编译 stub_external_deps_utest target(不链接 platform 库) - 运行 UT,验证 4 个测试用例全部通过: - InstanceReturnsSingleton:单例一致性 - InitializePlatformInfoReturnsZero:返回 0 - GetPlatformInfosReturnsZero:返回 0,不修改出参 - GetPlatformResReturnsFalse:返回 false,不修改 val 4. **回归验证** - 运行 tsd_client_utest,验证链接真实 platform 库的场景未受影响 ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签(如:feat:, fix:) - [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 - 新增文件:src/tsd/stub/stub_external_deps.cpp、src/tsd/tsdclient_sources.cmake、tests/ut/tsd/stub_external_deps/ - TSD_CHECK_RET_VOID 宏定义在 tsd_util_func.h 中,与现有 TSD_CHECK_RET 宏风格一致 See merge request: cann/runtime!4293 | 1 个月前 | |
【PR】: [fix] cleancode Co-authored-by: FengHaozhan<fenghaozhan@huawei.com> # message auto-generated for no-merge-commit merge: !4860 merge cleancode-20260910 into master 【PR】: [fix] cleancode Created-by: FengHaozhan Commit-by: FengHaozhan Merged-by: cann-robot Description: # Pull Request ## 描述 cleancode修改,保证头文件自包含、删除不需要的头文件 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新(格式化,局部变量) - [ ] ♻️ 重构(既不修复错误也不增加功能的代码变动) - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在当前页面的右侧'关联Issue'部分添加相应Issue链接,并勾选'合并后关闭已关联的 Issue'选项。 --> ## 如何测试 描述测试此变更的步骤和前提条件: 1. 2. ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [ ] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签(如: feat:, fix:) - [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 在此添加任何其他关于本次 PR 的说明。 See merge request: cann/runtime!4860 | 18 天前 | |
add tsdclient Co-authored-by: 苏俊伟<sujunwei3@huawei.com> | 7 个月前 | |
清理cmake configure阶段多余打印 Co-authored-by: Feiteng Zheng<zhengfeiteng1@h-partners.com> # message auto-generated for no-merge-commit merge: !1269 merge 20260327-refactor into master 清理cmake configure阶段多余打印 Created-by: zhengfeiteng Commit-by: Feiteng Zheng Merged-by: cann-robot Description: ## 描述 - 清理cmake configure阶段多余打印 - acp在oam-tools中编译打包,rts删除相关编译流程 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。例如:关联Issue #000--> ## 测试 已验证修改前后runtime包二进制一致。 ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [x] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: See merge request: cann/runtime!1269 | 5 个月前 | |
【feat】:tsdclient source list变量抽取 Co-authored-by: sujunwei3<sujunwei3@huawei.com> # message auto-generated for no-merge-commit merge: !4293 merge new_dev into master 【feat】:tsdclient source list变量抽取 Created-by: sujunwei3 Commit-by: sujunwei3 Merged-by: cann-robot Description: # Pull Request ## 描述 本 PR 对 tsdclient 构建文件和外部依赖桩机制做了三项改进: 1. **CMakeLists.txt 拆分**:将 tsdclient 的源文件列表和头文件包含目录从 CMakeLists.txt 内联写法抽取到独立的 tsdclient_sources.cmake 文件。CMakeLists.txt 通过 include 引用,使构建文件更简洁,源文件列表可被 UT 等其他目标复用。 2. **新增 stub_external_deps.cpp 桩文件**:为 fe::PlatformInfoManager 和 fe::PlatFormInfos::GetPlatformRes 提供空实现占位符号,用于"没有 platform target 可以链接"的场景。桩实现返回 0/false,保证链接通过但无实际行为。 3. **新增 TSD_CHECK_RET_VOID 宏 + 桩文件 UT**:补齐 TSD 错误检查宏的 void 返回分支;新增 stub_external_deps_utest UT target,不链接 platform 库,直接编译桩文件,固化桩的返回值契约(单例一致性、返回 0/false、不修改出参)。 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新(格式化,局部变量) - [x] ♻️ 重构(既不修复错误也不增加功能的代码变动) - [x] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在当前页面的右侧'关联Issue'部分添加相应Issue链接,并勾选'合并后关闭已关联的 Issue'选项。 --> #874 ## 如何测试 根据代码变更,测试场景如下: 1. **编译构建测试** - 执行整包编译,验证 tsdclient 共享库正确构建 - 验证 tsdclient_sources.cmake 中的源文件列表与重构前完全一致(无遗漏、无多余) 2. **CMake 配置验证** - 确认 tsdclient/CMakeLists.txt 正确 include tsdclient_sources.cmake - 验证 tsdclient_all_sources 和 tsdclient_include_dirs 变量定义正确 3. **桩文件 UT 验证** - 编译 stub_external_deps_utest target(不链接 platform 库) - 运行 UT,验证 4 个测试用例全部通过: - InstanceReturnsSingleton:单例一致性 - InitializePlatformInfoReturnsZero:返回 0 - GetPlatformInfosReturnsZero:返回 0,不修改出参 - GetPlatformResReturnsFalse:返回 false,不修改 val 4. **回归验证** - 运行 tsd_client_utest,验证链接真实 platform 库的场景未受影响 ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签(如:feat:, fix:) - [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 - 新增文件:src/tsd/stub/stub_external_deps.cpp、src/tsd/tsdclient_sources.cmake、tests/ut/tsd/stub_external_deps/ - TSD_CHECK_RET_VOID 宏定义在 tsd_util_func.h 中,与现有 TSD_CHECK_RET 宏风格一致 See merge request: cann/runtime!4293 | 1 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 18 天前 | ||
| 18 天前 | ||
| 23 天前 | ||
| 1 个月前 | ||
| 1 个月前 | ||
| 18 天前 | ||
| 7 个月前 | ||
| 5 个月前 | ||
| 1 个月前 |