已合并
bugfix:统一db问题修正 #487
wangzixuan创建于 15 天前
bugfix:统一db问题修正 #487
已合并
wangzixuan创建于 15 天前
wangzixuan成员
15 天前

PR 合入模板

注:经过自检不涉及的可标注“不涉及”或直接打勾,特殊情况请文字备注。不符合规范的 PR 不允许合入,请(后备)commit 注意。


1. 修改描述

  • 修改原因:
  • 1、DPUTask表中包含globalTid,但是未遵循统一的globalTid的呈现逻辑。
  • 2、静态图场景,无法区分device数据对应关系,globalTaskId重复
  • 修改内容:
  • 1、dpuTask表修改对应的globalTid计算规则为 高位pid 低位tid
  • 2、python侧 7 8 11 表pmuCalculator新增endTime
  • 3、c++侧pmuCalculator新增endtime字段
  • 4、unifiedPmuProcessor增加读取endTime字段。
  • 5、PmuTaskInfo表新增timestampNs字段
  • 6、CommunicationTaskInfo表同样新增timestampNs字段

2. 功能验证


3. 分支合并要求


3. 代码检视

  • 要求:

    • 合入代码超过 200 行,需三人以上会议检视。
    • 检视密度≥1个/100行。
    • 检视缺陷密度未达要求需提供说明。
    • 大于 1000 行代码原则上不允许合入,需进行备案。
  • 检视意见数:____ 条 (请填写本次检视的意见总数,用于commit合入前审视)


4. 安全自检

Python、C++

C++


5. 变更知会


likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 wangzixuan 的贡献)
Wwangzixuan成员
15 天前 创建了 pull request,commit 04534404
atomgit-bot
atomgit-bot
15 天前 评论:

变更摘要

本 PR 为 bugfix,修正 DPUTask 表写入时 globalTid 的生成逻辑:原实现直接将 threadId 作为 globalTid,未遵循统一的 globalTid 呈现规则。改动后统一按「高位 pid、低位 tid」计算 globalTid,即由 pidthreadId 拼接(Utils::Contact(pid, data.threadId))得到,并相应扩大该字段的数据类型。核心逻辑位于 analysis/csrc/application/database/db_assembler.cppSaveDPUData 函数,UT 用例在 db_assembler_utest.cpp 中同步更新。

主要改动

  • globalTid 计算规则修正:在 SaveDPUData 中新增通过 Environment::Context::GetInstance().GetPidFromInfoJson(HOST_ID, profPath) 获取 pid,并将入表字段由 data.threadId 改为 Utils::Contact(pid, data.threadId)(高位 pid、低位 tid)拼接到 64 位 globalTid
  • 字段类型扩展DPUTaskFormat 元组中对应 globalTid 列的元素类型由 uint32_t 调整为 uint64_t,以容纳拼接后的 64 位取值。
  • UT 用例同步更新db_assembler_utest.cppTestRunSaveDPUDataShouldReturnTrueWhenRunSuccessTestRunSaveDPUDataShouldReturnFalseWhenReserveFailedDPUDataFormat 类型声明同步改为 uint64_t,断言值由 123/124 更新为 Contact(1, 123)/Contact(1, 124)
likedislike
不准确?
atomgit-bot
atomgit-bot
15 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
ascend-robotascend-robot成员
15 天前 添加了label:ascend-cla/yes
此处折叠了172条消息 查看更多
panyj1993
panyj1993成员
10 天前 评论:

/approve

likedislike
ascend-robotascend-robot成员
10 天前 添加了label:approved
ascend-robotascend-robot成员
10 天前 合入了pull request
ascend-robot
ascend-robot成员
10 天前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
ascend-robot
ascend-robot成员
10 天前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike