已合并
fix(aicpu): 修正 hcomm 源码目录探测候选路径 #261
fix(aicpu): 修正 hcomm 源码目录探测候选路径 #261
已合并
ooooo创建于 4 天前
ooooo
ooooo成员
4 天前

当前PR是否有AI参与:

[ ] 否
[x] 是
__1. AI Agent 平台: OpenCode
__2. AI 模型: DeepSeek v4 flash
__3. Prompt上下文 : 将本地修改推送 PR 并整理描述

PR功能描述 / 为什么需要这个合入**:

修正 AICPU 控制面源码目录探测候选路径:移除已不存在的 ../hcomm_pkg_inc,补充嵌套布局候选 ../../hcomm。

该PR关联的issue

(格式为fixes #<issue号>, 或者resolves #<issue号>): fixes #79

希望检视人员了解:

  • 仅调整候选路径集合,平铺布局(../hcomm)行为不变。

改动类型 / Change Type

测试信息 / Testing

检查清单 / Checklist

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 ooooo 的贡献)
oooooooooo成员
4 天前 创建了 pull request,commit fb25bac7
oooooooooo成员
4 天前 关联了issue:[Bug-Report|缺陷反馈]: AICPU 控制面源码目录探测候选路径与实际布局不一致
atomgit-bot
atomgit-bot
4 天前 评论:

变更摘要

本 PR 修正 AICPU 数据面控制面源码目录探测逻辑的候选路径集合。在未显式指定 ASCCOMM_CONTROL_PLANE_SOURCE_DIR 时,原候选路径在 asc-comm 处于嵌套目录、hcomm 位于其上一级的构建布局下探测失败,会回退到随仓自包含头,与真实控制面契约不一致。此次调整仅涉及候选路径的增删,未改动其它构建逻辑。

主要改动

  • 移除失效候选路径: 删除已不存在的 ../hcomm_pkg_inc 候选路径(历史 pkg_inc 单独分发形态),避免无效探测。
  • 新增嵌套布局候选路径: 补充 ../../hcomm,覆盖 asc-comm 位于嵌套目录、hcomm 位于其上一级的构建布局。
  • 回退与校验逻辑保持不变: 控制面契约头 pkg_inc/hcomm/hcomm_res_entity_defs.h 的存在性校验与回退逻辑维持原状。
  • 平铺布局行为不变: 原有平铺同级布局候选 ../hcomm 的探测行为不受影响,仅扩展了嵌套布局的命中范围。
likedislike
不准确?
atomgit-bot
atomgit-bot
4 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
4 天前 添加了label:ai-co-authored
CANN-robotCANN-robot成员
4 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
4 天前 评论:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
src ✅ chenma, jiangxinyu3 (2/2) ✅ chenma, jiangxinyu3 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
oooooooooo成员
4 天前 修改了pull request 的描述
CANN-robotCANN-robot成员
4 天前 添加了label:ai-docs-only
CANN-robotCANN-robot成员
4 天前 添加了label:ai-compliance-failed
CANN-robot
CANN-robot成员
4 天前 评论:

AI Declaration Check Failed

@ooooo-create, AI declaration is not compliant. AI tool: OpenCode

Please check:

  1. The AI tool and model declared in the PR description must be real.
  2. Commit messages must include the AI model, and it must be consistent with the PR description. ⚠️
likedislike
oooooooooo成员
4 天前 修改标题为 “fix(aicpu): 修正 hcomm 源码目录探测候选路径”,原标题为“fix(aicpu): 修正控制面源码目录探测候选路径”
oooooooooo成员
4 天前 修改了pull request 的描述
oooooooooo成员
4 天前 预合并成功(commit_id: 9d3a2a2eac460c6d83ca9298b350caa481249cbc)
oooooooooo成员
4 天前 强制推送  1 个提交:92c19e13-fix(aicpu): 修正控制面源码目录探测候选路径
oooooooooo成员
4 天前 预合并成功(commit_id: 7d7470e2ce073080492ff30d72e06f0a2e464902)
oooooooooo成员
4 天前 修改了pull request 的描述
CANN-robot
CANN-robot成员
4 天前 评论:

AI Declaration Check Failed

@ooooo-create, AI declaration is not compliant. AI tool: openCode

Please check:

  1. The AI tool and model declared in the PR description must be real.
  2. Commit messages must include the AI model, and it must be consistent with the PR description. ⚠️
likedislike
CANN-robotCANN-robot成员
4 天前 删除了label:cann-cla/yes
oooooooooo成员
4 天前 修改标题为 “fix(aicpu): 修正控制面源码目录探测候选路径”,原标题为“fix(aicpu): 修正 hcomm 源码目录探测候选路径”
CANN-robotCANN-robot成员
4 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
4 天前 评论:

AI Declaration Check Failed

@ooooo-create, AI declaration is not compliant. AI tool: OpenCode

Please check:

  1. The AI tool and model declared in the PR description must be real.
  2. Commit messages must include the AI model, and it must be consistent with the PR description. ⚠️
likedislike
CANN-robotCANN-robot成员
4 天前 删除了label:ai-compliance-failed
此处折叠了6条事件消息 查看更多
CANN-robotCANN-robot成员
4 天前 添加了label:cann-cla/yes
ooooo
ooooo成员
4 天前 评论:

compile

likedislike
oooooooooo成员
4 天前 预合并成功(commit_id: 53dcfb6f85f3d0d3d2b37690758e7166801b39c5)
oooooooooo成员
4 天前 修改标题为 “fix(aicpu): 修正 hcomm 源码目录探测候选路径”,原标题为“fix(aicpu): 修正控制面源码目录探测候选路径”
CANN-robot
CANN-robot成员
4 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
4 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
4 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
4 天前 添加了label:ci-pipeline-passed
chenma成员
3 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
3 天前 添加了label:approved
CANN-robotCANN-robot成员
3 天前 添加了label:BUILD-FAIL
ooooo
ooooo成员
3 天前 评论:

compile

likedislike
oooooooooo成员
3 天前 预合并成功(commit_id: 3b49fe2ab5606005adee48e2199a7a0bdd2be67c)
CANN-robot
CANN-robot成员
3 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
3 天前 删除了label:ci-pipeline-passed
CANN-robotCANN-robot成员
3 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
3 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
3 天前 添加了label:ci-pipeline-passed
izanaami
izanaami成员
2 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
2 天前 添加了label:lgtm
CANN-robotCANN-robot成员
2 天前 合入了pull request