已合并
【bugfix】tsd Close 时清理 hasSendConfigFile_ 与 pkg hash 缓存 #3671
【bugfix】tsd Close 时清理 hasSendConfigFile_ 与 pkg hash 缓存 #3671
已合并
Ding_Jing创建于 7月15日
Ding_Jing
Ding_Jing成员
7月15日

Pull Request

描述

ProcessModeManager::Close() 关闭 hccp / computer 进程时,未重置 hasSendConfigFile_,也未清理 pkgHostHashValue_ / pkgDeviceHashValue_ 缓存,导致会话关闭后这些状态残留,可能影响下一次启动时“是否需要重新下发配置文件”的判断。本 PR 在 Close() 流程末尾补充清理这三处状态。

变更类型

关联的Issue

#743 (建议在右侧“关联 Issue”面板绑定,并勾选“合并后关闭已关联的 Issue”)

如何测试

  1. 启动 tsd 会话并触发一次配置文件下发,确认 hasSendConfigFile_ 置位、pkgHostHashValue_ / pkgDeviceHashValue_ 已记录包哈希。
  2. 调用 ProcessModeManager::Close() 关闭会话。
  3. 再次在同一对象上启动 / 下发,确认 hasSendConfigFile_ 已复位为 false、两个 pkg hash 已清空,配置文件能被正确重新下发。

核对清单

其他信息

改动仅 3 行,位于 src/tsd/tsdclient/src/process_mode_manager.cppProcessModeManager::Close(),不涉及对外接口变更,无需文档更新。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 Ding_Jing 的贡献)
Ding_JingDing_Jing成员
7月15日 添加了label:enhancement
Ding_JingDing_Jing成员
7月15日 创建了 pull request,commit 713a78bf
Ding_JingDing_Jing成员
7月15日 关联了issue:[Bug-Report|缺陷反馈]: tsd ProcessModeManager::Close 未清理 hasSendConfigFile_ 与 pkg hash 缓存
atomgit-bot
atomgit-bot
7月15日 评论:

变更摘要

本次修复针对 ProcessModeManager::Close() 在关闭会话时未清理配置文件下发相关状态的问题:当 hccp / computer 进程关闭后,hasSendConfigFile_ 标志位及 pkgHostHashValue_ / pkgDeviceHashValue_ 两个包哈希缓存未被重置,导致会话关闭后状态残留,可能使下一次启动时错误地跳过配置文件重新下发。改动在 Close() 返回前补充了这三项状态的清理逻辑。

主要改动

  • 重置 hasSendConfigFile_ 标志: 在 ProcessModeManager::Close() 中将 hasSendConfigFile_ 置为 false,确保下次会话启动时不会因为残留的已下发标记而跳过配置文件下发流程。
  • 清空 pkgHostHashValue_ 缓存: 在 ProcessModeManager::Close() 中调用 pkgHostHashValue_.clear(),清理主机侧包哈希缓存,避免旧哈希值影响新会话的配置变更判断。
  • 清空 pkgDeviceHashValue_ 缓存: 在 ProcessModeManager::Close() 中调用 pkgDeviceHashValue_.clear(),清理设备侧包哈希缓存,与主机侧保持一致的清理策略。
likedislike
atomgit-bot
atomgit-bot
7月15日 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
7月15日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
7月15日 评论:

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


PR Approval Progress

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

Module Approval Details

module lgtm status approve status
** gcw_kUomxQ2l, yanmingxiang (2/2) gcw_kUomxQ2l (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
7月15日 将wangtao43,zhangpengpeng8,yanmingxiang,Reyn52166,jiangtao_rts,ykl999,houyanbao,hanxiaolong,gcw_kUomxQ2l,Andy-lb,tingwood设为评审人
CANN-robotCANN-robot成员
7月15日 将jiangtao_rts,hanxiaolong,gcw_kUomxQ2l,Andy-lb设为审查人
Ding_Jing
Ding_Jing成员
7月15日 评论:

compile

likedislike
Ding_JingDing_Jing成员
7月15日 update merge request[project id: 8810053, iid: 3671, commit_id: 3f6090aa68260cf73204ba44b5ab7a7e21c4b6a4] virtual merging success
CANN-robotCANN-robot成员
7月15日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
7月15日 评论:

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_acl ✅ SUCCESS >>>>>
UT_Test_rts_v201 ✅ SUCCESS >>>>>
UT_Test_rts_c ✅ SUCCESS >>>>>
UT_Test_rts_david ✅ SUCCESS >>>>>
UT_Test_rts_910b ✅ SUCCESS >>>>>
UT_Test_rts_common ✅ SUCCESS >>>>>
UT_Test_platform ✅ SUCCESS >>>>>
UT_Test_qs ✅ SUCCESS >>>>>
UT_Test_aicpusd ✅ SUCCESS >>>>>
UT_Test_tsd ✅ SUCCESS >>>>>
UT_Test_error_manager ✅ SUCCESS >>>>>
UT_Test_slog ✅ SUCCESS >>>>>
UT_Test_atrace ✅ SUCCESS >>>>>
UT_Test_adump ✅ SUCCESS >>>>>
UT_Test_mmpa ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>
UT_Test_msprof ✅ SUCCESS >>>>>
UT_Test_rts_report ✅ SUCCESS >>>>>
UT_Test_msprof_report ✅ SUCCESS >>>>>

[2026-07-15 17:43:59]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月15日 添加了label:SC-SUCC
CANN-robot
CANN-robot成员
7月15日 评论:

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Check_Pr ✅ SUCCESS
codecheck_style ✅ SUCCESS >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-07-15 17:41:02]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月15日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
7月15日 添加了label:ci-pipeline-passed
CANN-robotCANN-robot成员
7月15日 添加了label:BUILD-SUCC
CANN-robotCANN-robot成员
7月18日 删除了label:BUILD-SUCC
CANN-robotCANN-robot成员
7月18日 添加了label:BUILD-SUCC
Ding_Jing
Ding_Jing成员
7月20日 评论:

compile

likedislike
Ding_JingDing_Jing成员
7月20日 update merge request[project id: 8810053, iid: 3671, commit_id: b7dfed02d9aa44ef8c10ebb15a8fc4f1bdc0c448] virtual merging success
CANN-robotCANN-robot成员
7月20日 删除了label:ci-pipeline-passed
CANN-robotCANN-robot成员
7月20日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
7月20日 评论:

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_acl ✅ SUCCESS >>>>>
UT_Test_rts_v201 ✅ SUCCESS >>>>>
UT_Test_rts_c ✅ SUCCESS >>>>>
UT_Test_rts_david ✅ SUCCESS >>>>>
UT_Test_rts_910b ✅ SUCCESS >>>>>
UT_Test_rts_common ✅ SUCCESS >>>>>
UT_Test_platform ✅ SUCCESS >>>>>
UT_Test_qs ✅ SUCCESS >>>>>
UT_Test_aicpusd ✅ SUCCESS >>>>>
UT_Test_tsd ✅ SUCCESS >>>>>
UT_Test_error_manager ✅ SUCCESS >>>>>
UT_Test_slog ✅ SUCCESS >>>>>
UT_Test_atrace ✅ SUCCESS >>>>>
UT_Test_adump ✅ SUCCESS >>>>>
UT_Test_mmpa ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>
UT_Test_msprof ✅ SUCCESS >>>>>
UT_Test_rts_report ✅ SUCCESS >>>>>
UT_Test_msprof_report ✅ SUCCESS

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

likedislike
CANN-robot
CANN-robot成员
7月20日 评论:

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Check_Pr ✅ SUCCESS
codecheck_style ✅ SUCCESS >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-07-20 08:36:07]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月20日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
7月20日 添加了label:ci-pipeline-passed
ZhaiPeiChao
ZhaiPeiChao成员
7月20日 评论:

/lgtm
/approve

likedislike
CANN-robot
CANN-robot成员
7月20日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
ZhaiPeiChao

likedislike
CANN-robot
CANN-robot成员
7月20日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
ZhaiPeiChao

likedislike
yanmingxiang成员
7月20日 评论:

/lgtm

likedislike
CANN-robot
CANN-robot成员
7月20日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
ZhaiPeiChao

likedislike
liulun0308成员
7月20日 评论:

/approve

likedislike
CANN-robot
CANN-robot成员
7月20日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
ZhaiPeiChao

likedislike
CANN-robotCANN-robot成员
7月20日 添加了label:lgtmapproved
CANN-robotCANN-robot成员
7月20日 合入了pull request