已合并
fix err: no module mem file #511
mei-feiyao创建于 3 天前
fix err: no module mem file #511
已合并
mei-feiyao创建于 3 天前
mei-feiyao
mei-feiyao
3 天前

描述

原因:prof采集host侧数据时timestamp值可能采用syscnt或monotonic(取决于是否从驱动获取host freq),工具采集module_mem数据下发的采集配置同理需要配置1或0用于通知底软组件上报syscnt或monotonic,即代码里的res1字段;
方法:调用PlatformHostFreqIsEnable,如果为true,则配置1,通知底软上报snscnt,否则下发0,通知底软上报monotonic

关联的Issue

测试

执行msprof --sys-hardware-mem=on --xxxxx的命令,采集性能数据,观察导出的数据文件里是否有module_mem.csv

文档更新

不涉及

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 mei-feiyao 的贡献)
mei-feiyaomei-feiyao
3 天前 创建了 pull request,commit fb354ada
atomgit-bot
atomgit-bot
3 天前 评论:

变更摘要

本 PR 修复“no module mem file”错误,核心改动位于 src/msprof/collector/dvvp/profimpl/collect/job_wrapper/src/prof_hardware_mem_job.cpp,将主机频率使能标志在内存采集任务之间做了对调:ProfAppMemJob::SetPeripheralConfig() 不再根据 Platform::instance()->PlatformHostFreqIsEnable() 设置 configP->res1(固定置 0),而 ProfAiStackMemJob::SetPeripheralConfig() 改为按该接口返回值设置 configP->res1

主要改动

  • 调整 ProfAppMemJob::SetPeripheralConfig()res1 配置: configP->res1 由原先的 Platform::instance()->PlatformHostFreqIsEnable() ? 1 : 0 改为固定 0,App 内存任务不再携带主机频率使能标志。
  • 调整 ProfAiStackMemJob::SetPeripheralConfig()res1 配置: configP->res1 由固定 0 改为 Platform::instance()->PlatformHostFreqIsEnable() ? 1 : 0,将主机频率使能标志迁移至 AI Stack 内存任务。
likedislike
atomgit-bot
atomgit-bot
3 天前 评论:

代码审查

✅ 未发现问题

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

CLA Signature Pass

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

likedislike
此处折叠了51条消息 查看更多
CANN-robotCANN-robot成员
18 小时前 添加了label:approved
Zzhuliangying成员
15 小时前 解决了最后一个问题
chenhao_1209成员
15 小时前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
15 小时前 添加了label:lgtm
CANN-robotCANN-robot成员
15 小时前 合入了pull request