Star87
33
代码
代码
Issues13
Pull Requests4
项目讨论
Wiki
分析
Star87
33
已合并
[feature] 支持core文件展示&切换线程信息 #106
wiyr0创建于 4月23日
[feature] 支持core文件展示&切换线程信息 #106
已合并
wiyr0创建于 4月23日
从wiyr0/msdebugfix_coredump_register合入到Ascend/msdebugmaster
wiyr0
wiyr0成员
4月23日

1. 修改描述

  • 修改原因： 如果coredump位置是在simt_vf里，工具需要展示、切换线程信息
  • 修改方案： 给ProcessElfCoreDevice类实现正确的GetCoresInfo, GetWarpsInfo接口。通过判断VEC_ERR_INFO寄存器，识别当前pc所在的是否是simt
  • 修改内容：
  1. ProcessElfCoreDevice类新增ProcessElfCoreDevice::GetThreadDim接口，用于从寄存器里拿thread_dim信息，从而获取warp_num的值。
    新增ProcessElfCoreDevice::FocusToActiveThreadInWarp，用于识别到当前error发生在simt_vf时，聚焦到对应的warp_id
    实现ProcessElfCoreDevice::GetWarpsInfo接口，获取当前的所有warp信息
  2. 修正pc替换时所用的mask数值类型为uint64_t
  3. 使用clang-format，自动修改了很多枚举定义的格式。
  4. Coredump的寄存器展示的顺序和上板保持一致
  5. 移除DeviceContext系列的类的无用函数GetRegisterAddr和GetRegisterList

2. 功能验证

3. 代码检视

  • 要求：
    • 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。
    • committer评估是否需要在sig会议进行代码检视。
    • 参与检视的committer人员名单与检视时间。
    • 大于 1000 行代码原则上不允许合入，需进行备案。
  • 检视committer人员名单与检视时间：

4. 资料修改自检

  • 资料修改： NA
likedislike
Pull Request已成功合入, 合并人@ascend-robot
（感谢 wiyr0 的贡献）
ascend-robotascend-robot成员
4月23日 添加了label：stat/needs-squash
ascend-robot
ascend-robot成员
4月23日 评论：

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/msdebug ✅ 龚思维, 门飞飞 (2/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

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

likedislike
ascend-robotascend-robot成员
4月23日 添加了label：ascend-cla/yes
wiyr0wiyr0成员
4月23日 关联了issue：[Feature]: 支持simd_vf上的断点代码行显示
wiyr0wiyr0成员
4月23日 修改标题为 “[feature] 支持core文件展示&切换线程信息”，原标题为“fix coredump thread register”
此处折叠了107条消息 查看更多
went_code
went_code成员
4月25日 评论：

/lgtm

likedislike
G
gong-siwei成员
4月25日 评论：

/lgtm
/approve

likedislike
ascend-robotascend-robot成员
4月25日 添加了label：approvedlgtm
ascend-robot
ascend-robot成员
4月25日 评论：

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: 龚思维.
Reviewers who wrote a comment of /lgtm are: 龚思维, 门飞飞.

likedislike
ascend-robotascend-robot成员
4月25日 合入了pull request