Star679
458
代码
代码
Issues29
Pull Requests100
Actions
项目讨论
Wiki
分析
Star679
458
已合并
【fix】: 防止重复AdxDataDumpServerInit初始化 #4620
lidaoming1创建于 6 天前
【fix】: 防止重复AdxDataDumpServerInit初始化 #4620
已合并
lidaoming1创建于 6 天前
从lidaoming1/gedump_dts合入到cann/gedevelop
L
lidaoming1成员
6 天前

Pull Request

描述

增加锁来控制adx的初始化

变更类型

请选择本次引入的变更类型：

关联的Issue

如何测试

描述测试此变更的步骤和前提条件：
1.
2.

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
（感谢 lidaoming1 的贡献）
Llidaoming1成员
6 天前 添加了label：enhancement
Llidaoming1成员
6 天前 创建了 pull request，commit 8c47cf1c
atomgit-bot
atomgit-bot
6 天前 评论：

变更摘要

本 PR 是一个 bugfix，旨在防止 AdxDataDumpServerInit 被重复初始化。核心改动是为 InnerSession 新增静态互斥锁 dump_server_mutex_，并在 AddDumpProperties 与 RemoveDumpProperties 中通过 std::lock_guard<std::mutex> 保护静态标志 is_dump_server_inited_ 的检查与赋值，从而避免多会话/多线程并发场景下 dump 服务器的重复初始化和重复反初始化。

主要改动

  • 新增静态互斥锁成员: 在 InnerSession 中新增 static std::mutex dump_server_mutex_（inner_session.h 声明、inner_session.cc 定义），用于串行化 dump 服务器初始化/反初始化流程。
  • 保护 AddDumpProperties 初始化逻辑: 将 is_dump_server_inited_ 的判断及 AdxDataDumpServerInit() 调用包裹在 std::lock_guard<std::mutex> 中，确保并发调用时只有一个线程执行初始化并将标志置为 true，避免重复 AdxDataDumpServerInit。
  • 保护 RemoveDumpProperties 反初始化逻辑: 将 AdxDataDumpServerUnInit() 及 is_dump_server_inited_ 复位同样置于锁保护之下，保证反初始化与初始化操作的互斥。
likedislike
atomgit-bot
atomgit-bot
6 天前 评论：

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
6 天前 添加了label：cann-cla/yes
CANN-robot
CANN-robot成员
6 天前 评论：

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
repo-cann/ge ✅ 张德鹏, ClarkXie, yangyongqiang0606 (3/2) ✅ 张德鹏 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
6 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
6 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
L
lidaoming1成员
6 天前 评论：

compile

likedislike
Llidaoming1成员
6 天前 update merge request[project id: 8824148, iid: 4620, commit_id: 735b2b09f81019712d00bcd5f13cf30dace68bb6] virtual merging success
CANN-robotCANN-robot成员
6 天前 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
6 天前 评论：

流水线任务触发成功
任务链接 [746536a35bb545559a25b672c11a479f][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_X86_executor_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_X86_dflow_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_ARM_compiler_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_ARM_executor_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_ARM_dflow_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_ARM_compiler ✅ SUCCESS >>>>> >>>>>
Compile_ARM_executor ✅ SUCCESS >>>>> >>>>>
Compile_ARM_dflow ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-08-28 16:52:36]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6 天前 评论：

流水线任务触发成功
任务链接 [4c8efcd0761b4b2a8547ceda224f815d][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler ✅ SUCCESS >>>>> >>>>>
Compile_X86_executor ✅ SUCCESS >>>>> >>>>>
Compile_X86_dflow ✅ SUCCESS >>>>> >>>>>
UT_Test_dflow ✅ SUCCESS
UT_Test_parser ✅ SUCCESS
UT_Test_python ✅ SUCCESS >>>>>
UT_Test_rt ✅ SUCCESS
UT_Test_acl ✅ SUCCESS
UT_Test_autofuse ✅ SUCCESS
UT_Test_executor ✅ SUCCESS
UT_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_dflow ✅ SUCCESS
ST_Test_ge_common ✅ SUCCESS
ST_Test_parser ✅ SUCCESS
ST_Test_python ✅ SUCCESS >>>>>
ST_Test_rt ✅ SUCCESS
ST_Test_autofuse ✅ SUCCESS
ST_Test_executor ✅ SUCCESS
ST_Test_hetero ✅ SUCCESS
ST_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_autofuse_e2e ✅ SUCCESS
UT_Test_fe ✅ SUCCESS
UT_Test_tefusion ✅ SUCCESS
ST_Test_fe ✅ SUCCESS
ST_Test_tefusion ✅ SUCCESS

[2026-08-28 17:17:07]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6 天前 评论：

流水线任务触发成功
任务链接 [fa50ad66f241415c9ffed1623b4bdcd3][流水线指导]

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-08-28 16:49:25]    CI执行结束

likedislike
CANN-robotCANN-robot成员
6 天前 添加了label：api-check-pass
Llidaoming1成员
6 天前 update merge request[project id: 8824148, iid: 4620, commit_id: 91c18d8fffbcf1a03c2fa1599c7270d79d421ca5] virtual merging success
Llidaoming1成员
6 天前 强制推送  1 个提交:77ec51b2-fix mutex for is_dump_server_inited_
Llidaoming1成员
6 天前 update merge request[project id: 8824148, iid: 4620, commit_id: 4199556a5b88c16420caeb11dcf33cebd716453a] virtual merging success
L
lidaoming1成员
6 天前 评论：

compile

likedislike
Llidaoming1成员
6 天前 update merge request[project id: 8824148, iid: 4620, commit_id: 4af21c22d795b610dcedb416ec38ee8b499521ad] virtual merging success
CANN-robotCANN-robot成员
6 天前 删除了label：cann-cla/yes
CANN-robotCANN-robot成员
6 天前 添加了label：cann-cla/yes
CANN-robotCANN-robot成员
6 天前 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
6 天前 删除了label：api-check-pass
CANN-robotCANN-robot成员
6 天前 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
6 天前 评论：

流水线任务触发成功
任务链接 [a8db7ddf0b5e4139bc097adba9a5cee0][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler ✅ SUCCESS >>>>> >>>>>
Compile_X86_executor ✅ SUCCESS >>>>> >>>>>
Compile_X86_dflow ✅ SUCCESS >>>>> >>>>>
UT_Test_dflow ✅ SUCCESS
UT_Test_ge_common ✅ SUCCESS
UT_Test_parser ✅ SUCCESS
UT_Test_python ✅ SUCCESS >>>>>
UT_Test_rt ✅ SUCCESS
UT_Test_acl ✅ SUCCESS
UT_Test_autofuse ✅ SUCCESS
UT_Test_executor ✅ SUCCESS
UT_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_dflow ✅ SUCCESS
ST_Test_ge_common ✅ SUCCESS
ST_Test_parser ✅ SUCCESS
ST_Test_python ✅ SUCCESS >>>>>
ST_Test_rt ✅ SUCCESS
ST_Test_autofuse ✅ SUCCESS
ST_Test_executor ✅ SUCCESS
ST_Test_hetero ✅ SUCCESS
ST_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_autofuse_e2e ✅ SUCCESS
UT_Test_fe ✅ SUCCESS
UT_Test_tefusion ✅ SUCCESS
ST_Test_fe ✅ SUCCESS
ST_Test_tefusion ✅ SUCCESS
ST_Test_ge_common_atc ✅ SUCCESS
UT_Test_Report_graphengine ✅ SUCCESS >>>>>
UT_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
UT_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
ST_Test_Report_graphengine ✅ SUCCESS >>>>>
ST_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
ST_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
UT_Test_Report_fe ✅ SUCCESS >>>>>
ST_Test_Report_fe ✅ SUCCESS >>>>>

[2026-08-28 17:37:41]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6 天前 评论：

流水线任务触发成功
任务链接 [077ff24f1c14449a9c72090f4a5056ae][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_X86_executor_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_X86_dflow_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_ARM_compiler_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_ARM_executor_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_ARM_dflow_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_ARM_compiler ✅ SUCCESS >>>>> >>>>>
Compile_ARM_executor ✅ SUCCESS >>>>> >>>>>
Compile_ARM_dflow ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-08-28 17:22:04]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6 天前 评论：

流水线任务触发成功
任务链接 [5b99c06a97b04b07bcda7177067a7584][流水线指导]

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-08-28 17:19:54]    CI执行结束

likedislike
CANN-robotCANN-robot成员
6 天前 添加了label：api-check-pass
此处折叠了10条事件消息 查看更多
CANN-robotCANN-robot成员
6 天前 添加了label：ci-pipeline-passed
C
ClarkXie成员
6 天前 评论：

/lgtm

likedislike
yangyongqiang
yangyongqiang成员
5 天前 评论：

/lgtm

likedislike
CANN-robotCANN-robot成员
5 天前 添加了label：lgtm
张
张德鹏成员
5 天前 评论：

/approve

likedislike
CANN-robotCANN-robot成员
5 天前 添加了label：approved
CANN-robotCANN-robot成员
5 天前 合入了pull request