master

cann-robotfix "jetty SQ overflow check"

0efba3f5创建于 3 天前历史提交

文件	最后提交记录	最后更新时间
common	【feat】集合通信域管理层和基础通信层二进制拆分 第二阶段反向依赖解耦 Co-authored-by: zjw22<zhujiawei29@huawei.com> # message auto-generated for no-merge-commit merge: !4916 merge 930 into master 【feat】集合通信域管理层和基础通信层二进制拆分 第二阶段反向依赖解耦 Created-by: zjw22 Commit-by: zjw22 Merged-by: cann-robot Description: ## 描述 本 PR 完成 base_comm(L3)对 coll_communicator_mgr(L2)的编译期反向依赖消减 主要改动: 1. **头文件下沉**:将 base_comm 引用的上层头文件逐个消减/下沉至 base_comm 或 pkg_inc,包括 hccl_comm_pub.h、hcom_common.h、hcclCommDfx.h、dfx_profiling_handler(_lite).h、coll_comm_aicpu_kernel_adpt.h、aicpu_init_param.h、comm_mems.h、notify_manager.h、hccl_independent_common.h、hcclCommOp.h 等; 2. **dfx 代码归位**:将 profiling/taskException 相关实现迁入 src/base_comm/dfx/(dfx_profiling_handler.cc、dfx_profiling_handler_lite.cc、dfx_dlprof_function.cc、hcclCommOp.cc、channel_profiling_adpt.cc 等),device 侧代码归入 device 编译目标;通过 std::function 回调注册 + DfxDlProfFunction(dlopen)桥接替代对上层头文件的直接依赖; 3. **构建收口**:删除 hcomm_base_obj.cmake 中全部 16 个 coll_communicator_mgr include 目录,base_comm 编译期不再可见上层头文件; 4. **配套修复**:修复消减过程中暴露的编译问题、UT/ST 失败及 pre-commit 问题;新增 opbase_adpt 收敛 hcom_common.h 中的线程设备 ID 声明。 变更后经全仓检查,src/base_comm 下无任何对 coll_communicator_mgr 的 #include 与符号引用,分层依赖方向(L2→L3)成立。 ## 变更类型 - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他，请描述：分层架构整改（base_comm 反向依赖消减，代码迁移与构建收口） ## 关联的Issue NA <!-- 若有拆解的消减任务 Issue 请替换 --> ## 测试 已完成的测试用例和场景： 1. bash build.sh --pkg --full 全量编译通过（host + device），无编译错误/告警； 2. bash build.sh -u UT 全量通过（含修复的 ut/st 用例）； 3. bash build.sh -s ST 二级冒烟通过； 4. HCCL_TEST用例样子通过 5. HIXL Benchmark 单机双机用例通过 6. HCCL 图模式用例通过 7. checker用例 7.1 CCU_SCHED  7.2 AI_CPU  7.3 AIV  补充的UT用例： - opbase_adpt 适配层（新增 test/ut/base_comm/common/，target hccl_ut_base_comm_common）：HcclGetThreadDeviceId 覆盖缓存命中、首次调用成功、hrtGetDevice 失败、GetMaxDevNum 失败、设备数超限返回 INVALID_INT 共 5 条路径；HcclDeviceRefresh 覆盖成功（出参刷新+thread_local 缓存更新）/失败透传错误码 2 条路径；依赖 hrtGetDevice/hccl::GetMaxDevNum/hrtGetDeviceRefresh 均走 mockcpp 打桩； - channel_profiling_adpt 注册/回调机制（test/ut/dfx/ut_channel_profiling_adpt.cc，并入 target hccl_utest_dfx）：未注册时 GetChannelRemoteRankIdFuncImpl 返回 nullptr、注册后返回注册函数指针、注册后回调可正常调用（覆盖检视意见要求的注册前/注册后场景）、双线程并发注册+读取冒烟（验证 g_getChannelRemoteRankIdFunc 原子化改造无数据竞争）； ## 文档更新 无 <!-- 若 dfx 目录迁移影响架构文档，同步更新 docs/zh/architecture/architecture-brief.md --> ## 合入检查 - [x] 🧐 已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论/lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签（如：[feat], [fix]） <!-- 建议标题：[refactor] base_comm 依赖 coll_communicator_mgr 消减 --> - [x] 📄 代码修改内容已简要描述，相关文档已更新 - [x] 📝 代码注释已更新，代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新，覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查，无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲，确保代码质量 - [x] ✅ 代码检视意见已处理或答复，无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [ ] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile，编译无错误，无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试，确保基本功能正常 - [ ] 🎯 预约 approver评论/approve，正式合入前的必要检查 - [ ] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!4916	8 天前
primitives	fix "jetty SQ overflow check" Co-authored-by: richRachel<yinjingcheng@h-partners.com> # message auto-generated for no-merge-commit merge: !5394 merge master into master fix "jetty SQ overflow check" Created-by: booleanwater Commit-by: richRachel Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 解决在jetty被多个channel共享是，ci得不到更新，导致jetty溢出的问题 ## 变更类型 请选择本次引入的变更类型： <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他，请描述： ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue，可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景： 1. 单边通信hixl用例 2. 集合通信用例 补充的UT用例： ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前，请做好必要的代码测试，用例补充，软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论/lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签（如：[feat], [fix]） - [x] 📄 代码修改内容已简要描述，相关文档已更新 - [x] 📝 代码注释已更新，代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新，覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查，无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲，确保代码质量 - [x] ✅ 代码检视意见已处理或答复，无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile，编译无错误，无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试，确保基本功能正常 - [x] 🎯 预约 approver评论/approve，正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!5394	3 天前