已合并
[fix]修复 dump core-id 参数在没有timelineDetail时不拦截的问题 #250
wanghaopeng创建于 14 天前
[fix]修复 dump core-id 参数在没有timelineDetail时不拦截的问题 #250
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 wanghaopeng 的贡献）
atomgit-bot
14 天前 评论：
14 天前 评论：
变更摘要
此 PR 修复了 --dump 和 --core-id 参数在 device 模式下未校验 --aic-metrics=TimelineDetail 依赖关系的问题。修改集中在 ArgChecker::CheckDump 与 ArgChecker::CheckCoreId 两个方法中：在芯片支持校验之后，新增对 config.argAicMetrics.isDeviceToSimulator 的判断，当该字段为 false（即未指定 TimelineDetail）时，直接拦截并返回明确的错误提示；同时将 ASCEND950_SERIES 补充到 --dump 的芯片支持列表中。
主要改动
CheckDump新增TimelineDetail依赖校验：在 device 模式下，芯片支持校验通过后，增加对config.argAicMetrics.isDeviceToSimulator的检查，若为false则返回错误"--dump requires --aic-metrics=TimelineDetail."，阻止无TimelineDetail时使用--dump。CheckCoreId新增TimelineDetail依赖校验：与CheckDump一致，在 device 模式下增加对isDeviceToSimulator的检查，若未启用则返回"--core-id requires --aic-metrics=TimelineDetail."，确保--core-id仅在指定TimelineDetail时可用。CheckDump扩展芯片支持范围：在 device 模式的芯片白名单中新增ChipProductType::ASCEND950_SERIES，使ASCEND950系列芯片也能使用--dump功能。
atomgit-bot
14 天前 评论：
14 天前 评论：
14 天前 添加了label：ascend-cla/yes
ascend-robot
14 天前 评论：
14 天前 评论：
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 |
|---|---|---|
| repo-Ascend/msopprof | ✅ yan_yw, zzzsss1234 (2/2) | ✅ yan_yw (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
RainbowWhp, thanks for your pull request. All authors of the commits have signed the CLA. 👍
此处折叠了68条消息 查看更多
13 天前 添加了label：approved
流栎
13 天前 评论：
13 天前 评论：
/lgtm
13 天前 添加了label：lgtm
13 天前 合入了pull request
1. 修改描述
在 ArgChecker 参数初始化阶段增加依赖与冲突校验，使无效组合在应用启动前失败。
按产品区分参数支持范围：--dump：仅支持 A2/A3。
--core-id：支持 A2/A3/A5。
对 A5 的 range 重放增加专属指标冲突拦截，同时不改变 A2/A3 的既有行为。
同步英文资料中的产品支持范围。
增加 --dump 和 --core-id 对 TimelineDetail 的依赖校验：
上板模式使用 --dump=on 时，必须配置 --aic-metrics=TimelineDetail。
上板模式使用非空 --core-id 时，必须配置 --aic-metrics=TimelineDetail。
保持先检查芯片支持范围，再检查指标依赖。
明确产品支持范围：
--dump 仅支持 A2/A3，不支持 A5。
--core-id 支持 A2/A3/A5。
simulator 模式原有行为保持不变。
增加 A5 range 重放冲突校验：
A5 使用 --replay-mode=range 时，禁止同时配置 Source、PcSampling、Roofline、PipeTimeline 和 InstrTimeline。
保留 range 与 TimelineDetail、Source、MemoryDetail 的公共冲突校验。
A2/A3 的 Roofline + range 行为保持不变。
同步英文资料：
明确 A5 不支持 --dump。
明确 A5 支持 --core-id。
补充回归测试：
覆盖 A5 dump 不支持场景。
覆盖 A5 core-id 缺少 TimelineDetail 失败、配置 TimelineDetail 成功场景。
覆盖 A5 range 与各冲突指标组合失败场景。
覆盖 A2/A3 Roofline + range 兼容场景。
2. 功能验证
3. 代码检视
4. 资料修改自检