已合并
[feature] 950 coredump解析功能-线程展示优化 #134
wangyixian创建于 5月13日
[feature] 950 coredump解析功能-线程展示优化 #134
已合并
wangyixian创建于 5月13日
wangyixian成员
5月13日

1. 修改描述

  • 修改原因: 线程展示相比友商缺少文件名和行号的展示;当前只有异常的warp的pc是可以推算的,其他warp的pc是未知,不应该展示;加载core文件后,可以直接展示异常代码位置,提高易用性。
  • 修改方案: 通过pc获取行号信息,若失败则打印NA; 加载device core文件后,获取默认线程,打印线程的位置信息;
  • 修改内容:
  1. 实现PrettyPrintTable函数,打印一个表格:
  • 用于获取每一列的最大宽度
  • 列与列之前只相隔1个空格
  • 每列都右对齐
  1. GetLineEntryForPC实现从pc拿到line信息
  • 涉及代码双合(贴上另一个PR链接):NA

2. 功能验证

  • 功能自验截图(请确保不体现个人信息)
    image.png
  • 冒烟是否通过
    image.png

3. 代码检视

  • 要求:
    • 合入功能代码大于 200 行,需要sig会议申报代码检视议题,并在PR中标注会议。
    • committer评估是否需要在sig会议进行代码检视。
    • 参与检视的committer人员名单与检视时间。
    • 大于 1000 行代码原则上不允许合入,需进行备案。
  • 是否经过代码检视
  • 是否具备UT测试用例看护
  • 是否需要在sig会议中进行代码检视
  • 检视committer人员名单与检视时间:

4. 资料修改自检

  • 资料修改:

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 wangyixian 的贡献)
ascend-robotascend-robot成员
5月13日 添加了label:stat/needs-squash
ascend-robot
ascend-robot成员
5月13日 评论:

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) 门飞飞, 龚思维 (2/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成员
5月13日 添加了label:ascend-cla/yes
Wwangyixian成员
5月13日 设置为草稿状态
ascend-robot
ascend-robot成员
5月13日 评论:
流水线 PR-pipeline_msdebug#342 [ commitID:c4fbc135 ] 运行中
>>>代码风格自动修复执行失败,具体请查看日志,不影响流水线执行及PR合入
阶段 任务名 状态 详情
编译构建 Build_msdebug_x86 🕚 >>>
Build_msdebug_arm 🕚 >>>
恶意代码检查 Antipoison_msdebug 🕚 >>>
编码安全与规范检查 CodeCheck_msdebug 🕚 >>>
PreCommit_gitcode 🕚 >>>
开源片段检查 SCA_msdebug 🕚 >>>
开发者测试 PreSmoke_msdebug 🕚 >>>
流水线 PR-pipeline_msdebug 🕚 >>>
此流水线已支持下列评论快捷指令,仅PR创建者评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
此处折叠了160条消息 查看更多
ascend-robotascend-robot成员
5月19日 添加了label:approved
went_code
went_code成员
5月19日 评论:

/approve

likedislike
ascend-robotascend-robot成员
5月19日 添加了label:lgtm
ascend-robot
ascend-robot成员
5月19日 评论:

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成员
5月19日 合入了pull request