| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
fix: improve log usability Co-authored-by: m0_50621083<hongyuecheng@huawei.com> # message auto-generated for no-merge-commit merge: !4556 merge CANN-log-evaluation-develop into develop fix: improve log usability Created-by: m0_50621083 Commit-by: m0_50621083 Merged-by: cann-robot Description: # Pull Request ## 描述 本次修改针对 GE 代码中的日志易用性问题进行整改，主要包括： - 修复日志中的英文拼写及语法问题； - 优化日志提示语义和表达； - 补充部分日志中的单位及上下文信息； - 统一部分日志及注释的英文表达； - 优化个别不规范的日志输出方式。 本次修改主要涉及日志和提示文本，不改变核心功能逻辑。 ## 变更类型 请选择本次引入的变更类型： <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新（格式化，局部变量） - [ ] ♻️ 重构（既不修复错误也不增加功能的代码变动） - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue 无关联Issue <!-- 如果这个PR是为了解决特定的Issue，请在当前页面的右侧'关联Issue'部分添加相应Issue链接，并勾选'合并后关闭已关联的 Issue'选项。 --> ## 如何测试 描述测试此变更的步骤和前提条件： 1. 使用 git diff 对本次修改内容进行检查，确认修改范围均属于日志易用性整改。 2. 使用 git diff --check 检查代码格式，确认不存在 trailing whitespace 等格式问题。 3. 检查日志字符串修改前后的参数及格式化占位符，确认日志参数与原有逻辑保持一致。 4. 确认本次修改主要涉及日志、提示信息及相关文本，不影响原有核心功能逻辑。 ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [ ] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签（如： feat:, fix:） - [ ] 我已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 本次为日志易用性专项整改，修改范围涉及多个模块，主要针对现有日志中的拼写、语法、语义表达、单位信息以及日志输出规范进行优化。 相关修改不涉及对外接口变更，不改变现有业务流程及核心功能逻辑。 See merge request: cann/ge!4556 | 2 个月前 | |
!4829 merge fix/issue-551-model-extend-boundary-check into master fix: 校验模型扩展分区长度(#551) Created-by: ningjingyuan Commit-by: ningjingyuan Merged-by: cann-robot Description: # Pull Request ## 描述 修复 ParseModelDescExtend 未校验模型扩展分区内部长度的问题。 OM 文件中的 ModelExtendHead::len 来自外部输入。原实现直接将该长度传入 ParseTlvList，当声明长度超过扩展分区实际剩余空间时，子解析器可能读取分区边界之外的数据。 本次变更： - 在解析 TLV 列表前校验 extendDesc->len <= size - sizeof(ModelExtendHead)，越界时返回 ACL_ERROR_GE_LOAD_MODEL。 - 为普通内存加载路径增加最小一字节越界回归用例，通过缩短分区表中的声明长度验证 GE 在解析阶段拒绝非法模型且不调用 Runtime。 - 为 LiteOS 文件查询路径增加最小一字节越界回归用例，验证按分区长度读取的缓冲区受到相同边界保护。 - 将既有 AbnormalInvalidLen 用例调整为 FifoMallocFailed：把扩展区长度由非法的 52 改为与实际 TLV 一致的 36，使其继续稳定覆盖 ProcFifoInfo 的 FIFO 地址数组分配失败分支。 ## 变更类型 - [x] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新（格式化，局部变量） - [ ] ♻️ 重构（既不修复错误也不增加功能的代码变动） - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue Closes #551 https://gitcode.com/cann/ge/issues/551 ## 如何测试 1. 增量编译 ut_ge_executor_c_utest，目标编译成功。 2. 执行 UtestGEExecutorTest.GeExecutorCaseParseModelDescExtend*，10/10 通过。 3. 执行 UtestGEExecutorLiteOSTest.GeExecutorCaseParseModelDescExtend*，3/3 通过。 4. 对变更测试文件执行 pre-commit 检查，clang-format、codespell、私钥检测和 OAT 等检查全部通过。 ## 核对清单 - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 本次变更无需更新文档（不改变公开 API、ABI 或 OM 格式） - [x] 我在标题中使用了合适的类型标签（如：feat:, fix:） - [x] 我已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 本次修改仅在解析不合法 OM 扩展分区时提前返回错误，不改变合法模型的加载行为。 See merge request: cann/ge!4829 | 1 个月前 | |
fix: precommit整改 Co-authored-by: yelongjian<yelongjian1@huawei.com> # message auto-generated for no-merge-commit merge: !3726 merge dev-precommit into develop fix: precommit整改 Created-by: yelongjian Commit-by: yelongjian Merged-by: cann-robot Description: # Pull Request ## 描述 precommit整改 ## 变更类型 请选择本次引入的变更类型： <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新功能 - [x] 💄 代码风格更新（格式化，局部变量） - [ ] ♻️ 重构（既不修复错误也不增加功能的代码变动） - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在当前页面的右侧'关联Issue'部分添加相应Issue链接，并勾选'合并后关闭已关联的 Issue'选项。 --> ## 如何测试 描述测试此变更的步骤和前提条件： 1.NA ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签（如： feat:, fix:） - [x] 我已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 在此添加任何其他关于本次 PR 的说明。 See merge request: cann/ge!3726 | 4 个月前 |