已合并
[motor]日志整改 #959
[motor]日志整改 #959
已合并
lichunyang创建于 7月15日
lichunyang
lichunyang成员
7月15日

合入来源

问题/功能描述

[motor]日志整改

修改方案描述

1、日志信息不合理
2、修改日志等级不合理,warn级别日志中,有error、failed等error级别字段信息
3、重要流程日志缺失

是否涉及UT/ST

开发自检

likedislike
Pull Request已成功合入, 合并人@chenxin
(感谢 lichunyang 的贡献)
lichunyanglichunyang成员
7月15日 关联了issue:[Bug]:warn级别日志有error、failed字段信息
atomgit-bot
atomgit-bot
7月15日 评论:

变更摘要

本次 PR 主要对 motor 项目中多个模块的日志进行规范化整改,核心目标是调整日志级别和统一日志措辞。大量原本使用 LOG_WARN / BM_LOG_WARN / SM_LOG_WARN 输出的关键失败场景被提升为 LOG_ERROR / BM_LOG_ERROR / SM_LOG_ERROR,同时将日志消息中的 "Failed to" 统一替换为 "Unable to" 或 "unable to",部分 INFO 级别日志也根据实际严重程度调整为 WARN 或 ERROR。

主要改动

  • 日志级别提升(WARN→ERROR):在 acc_links、hybm、smem 三个子系统的数十处关键路径中,将网络收发失败、内存分配失败、证书校验失败、RA/QP 初始化失败、etcd 读写失败、群组操作失败等场景的日志从 WARN 提升为 ERROR,确保这些实质性错误能被更有效地捕捉和告警。

  • 日志措辞统一("Failed to"→"Unable to"):跨 20 余个文件将日志消息前缀从 "Failed to" 改为 "Unable to" 或 "unable to",覆盖 acc_tcp_listener、acc_tcp_worker、hybm_va_manager、hybm_dev_user_legacy_segment、host_hcom_transport_manager、smem_shm_entry 等模块,统一了错误日志的表达风格。

  • host_hcom_transport_manager.cpp 日志级别调整:4 处远程读写操作因 channel == 0(未连接)的日志从 BM_LOG_INFO 提升为 BM_LOG_WARN,使其与实际错误严重程度匹配。

  • hybm_stream.cpp 日志增强:AllocSqcq、AllocLogicCq、Destroy 等函数的日志从 INFO/WARN 提升为 ERROR,并补充了 deviceId、tsId 等上下文信息,便于问题定位。

likedislike
不准确?
atomgit-bot
atomgit-bot
7月15日 评论:

代码审查

审查总结

本次 PR(#959,日志整改)对 37 个文件进行了日志级别和措辞的调整,主要目标是消除 WARN 级别日志中包含 "error"/"failed" 等字段的问题(关联 issue #290)。审查发现 4 个 P2 问题 和 4 个 P3 问题,无 P0/P1 级别问题。

P2 问题(4 个)

# 文件 问题
1 src/hybm/csrc/common/hybm_define.h:165 DL_LOAD_SYM_OPTIONAL 宏(可选符号加载)从 WARN 升级为 ERROR,影响 20+ 处调用点,会在可选符号缺失时产生大量 ERROR 噪音
2 src/hybm/csrc/mm/hybm_host_shm_segment.cpp:46-87 5 处 hugepage 检测回退路径从 WARN 升级为 ERROR——这些都是预期内的回退(fallback to /dev/shm),不是错误
3 src/hybm/csrc/data_operation/host/hybm_data_op_host_rdma.cpp:49-51 hugepage 分配失败的回退路径从 WARN 升级为 ERROR——代码会回退到普通页分配,日志信息也明确写了 "fallback to mmap with regular pagesize"
4 src/smem/csrc/net/smem_net_group_engine.cpp:1212 SM_LOG_WARN → SM_LOG_AND_SET_LAST_ERROR 不仅是日志级别变化,还会设置全局 SmLastError 状态,引入了超出日志整改范围的行为变更

P3 问题(4 个)

# 文件 问题
5 src/hybm/csrc/transport/device/device_rdma_indirect_transport_manager.cpp:1124 拼写错误 "unbale" 应为 "unable"
6 src/smem/csrc/smem_bm/smem_bm.cpp:88-89 冗余措辞 "unable to ... failed"
7 src/hybm/csrc/transport/device/fixed_ranks_qp_manager.cpp:575 冗余措辞 "unable to ... failed"
8 src/acc_links/csrc/acc_tcp_listener.cpp:249 文件末尾缺少换行符(POSIX 规范)

逐文件审查结果

以下文件经审查无问题(多为措辞变更 "Failed"→"Unable" 或合理的日志级别提升):

  • src/acc_links/csrc/acc_tcp_link_complex_default.cpp — WARN→ERROR(入队失败),合理
  • src/acc_links/csrc/acc_tcp_link_complex_default.h — WARN→ERROR(接收头失败),合理
  • src/acc_links/csrc/acc_tcp_link_default.h — WARN→ERROR(发送/读取失败),合理
  • src/acc_links/csrc/acc_tcp_worker.cpp — 措辞变更,无问题
  • src/acc_links/csrc/security/acc_tcp_ssl_helper.cpp — INFO/WARN→ERROR(证书验证失败),合理
  • src/hybm/csrc/driver/userspace/devmm_svm_gva.cpp — 措辞变更,无问题
  • src/hybm/csrc/entity/hybm_entity_default.cpp — 措辞变更,无问题
  • src/hybm/csrc/hybm_entry.cpp — WARN→ERROR(ptracer 初始化),合理
  • src/hybm/csrc/mm/hybm_conn_based_segment.cpp — 措辞变更,无问题
  • src/hybm/csrc/mm/hybm_dev_user_legacy_segment.cpp — 措辞变更,无问题
  • src/hybm/csrc/mm/hybm_va_manager.cpp — 措辞变更,无问题
  • src/hybm/csrc/mm/hybm_vmm_based_segment.cpp — WARN→ERROR(析构清理)+措辞变更,合理
  • src/hybm/csrc/transport/compose/compose_transport_manager.cpp — 措辞变更,无问题
  • src/hybm/csrc/transport/device/device_rdma_transport_manager.cpp — WARN→ERROR(RA 初始化),合理
  • src/hybm/csrc/transport/device/indirect/async_socket_queue.cpp — WARN→ERROR(epoll 创建),合理
  • src/hybm/csrc/transport/device/joinable_ranks_qp_manager.cpp — 措辞变更,无问题
  • src/hybm/csrc/transport/device/urma/device_urma_transport_manager.cpp — 措辞变更,无问题
  • src/hybm/csrc/transport/host/host_hcom_transport_manager.cpp — INFO→WARN + 措辞变更,合理
  • src/hybm/csrc/ts_engine/hybm_stream.cpp — INFO/WARN→ERROR,合理
  • src/hybm/csrc/under_api/dl_hcom_api.cpp — INFO→WARN,合理
  • src/smem/csrc/config_store/backend/smem_etcd_client.h — WARN→ERROR(etcd 操作),合理
  • src/smem/csrc/config_store/backend/smem_etcd_store_backend.cpp — WARN→ERROR(etcd 操作),合理
  • src/smem/csrc/config_store/ha/smem_ha_config_store.cpp — WARN→ERROR + 措辞变更,合理
  • src/smem/csrc/config_store/tcp_store/smem_tcp_config_store_server.cpp — 措辞变更 + WARN→ERROR,合理
  • src/smem/csrc/smem_bm/smem_bm_entry.cpp — 措辞变更,无问题
  • src/smem/csrc/smem_bm/smem_bm_entry_manager.cpp — WARN→ERROR(重复创建),合理
  • src/smem/csrc/smem_shm/smem_shm_entry.cpp — 措辞变更,无问题
  • src/smem/csrc/smem_shm/smem_shm_entry_manager.cpp — WARN→ERROR(重复创建),合理
  • src/smem/csrc/smem_trans/smem_trans_entry.cpp — INFO→WARN + 增加调试信息,合理

整体风险评估

中等风险。主要风险集中在:将回退/可选路径的日志级别从 WARN 提升为 ERROR 会导致生产环境中产生大量误报告警(P2 问题 #1-#3),以及 SM_LOG_AND_SET_LAST_ERROR 的行为变更可能影响下游错误诊断(P2 问题 #4)。建议在合入前修复 4 个 P2 问题。P3 问题(typo、措辞冗余、缺少换行符)影响较小,可在后续迭代中修复。

类型 数量
🔴 阻塞 1
🟡 建议 3

⛔ 需要修改

likedislike
不准确?
ascend-robot
ascend-robot成员
7月15日 评论:
流水线 PR-pipeline_memfabric_hybrid#4589 [ commitID:c6af5498 ] 已完成
阶段 任务名 状态 详情
编译构建 Build_memfabric-hybrid ✅ >>>
Build_memfabric-hybrid-bazel ✅ >>>
恶意代码检查 Antipoison_mf ✅ >>>
编码安全与规范检查 pre-commit ✅ >>>
CodeCheck_mf ✅ >>>
开源片段检查 SCA_mf ✅ >>>
开发者测试 UT_memfabric-hybrid ✅ >>>
流水线 PR-pipeline_memfabric_hybrid ✅ >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
此处折叠了147条消息 查看更多
ascend-robotascend-robot成员
7月21日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
7月21日 添加了label:ci-pipeline-passed
chenxinchenxin成员
7月21日 通过审查
chenxinchenxin成员
7月21日 关闭了关联的issue
chenxinchenxin成员
7月21日 合入了pull request