Star85
32
代码
代码
Issues14
Pull Requests4
流水线
项目讨论
Wiki
分析
Star85
32
已合并
[fixbug] 多线程下调试器错停于内部断点&构建问题 #45
wangyixian创建于 3月17日
[fixbug] 多线程下调试器错停于内部断点&构建问题 #45
已合并
wangyixian创建于 3月17日
从wiyr0/msdebugfix_bug合入到Ascend/msdebugmaster
W
wangyixian成员
3月17日

1. 修改描述

  • 修改原因：
  1. 多线程的场景里，内部断点触发时，出现了某个线程也是sigtrap的中断，此时client端会认为多个线程出现sigtrap中断，需要stop。
  2. 算子多卡场景里，若劫持函数是多线程被调用下，发送kernel信息的劫持函数，易出现覆盖现象，导致该发送kernel的卡没发送kernel信息
  3. 某些构建环境下，构建出来的liblldb.so一来libform.so
  • 修改方案：
  1. 强行修改非主线程的中断信号
  2. 加锁
  3. 增加libform.so.5的交付件
  • 修改内容：
  1. 当前为内部断点事件时，强行修改非主线程的中断信号为0号
  2. 给sentBinaries map加个局部静态锁。
  3. 参考libpanel.so的加法，增加libform.so.5的交付件
  4. 删除不再使用到的函数
  • 涉及代码双合(贴上另一个PR链接)：NA

2. 功能验证

  • 功能自验截图（请确保不体现个人信息）

image.png
image.png

  • 冒烟是否通过

3. 代码检视

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

4. 资料修改自检

  • 资料修改： NA
likedislike
Pull Request已成功合入, 合并人@ascend-robot
（感谢 wangyixian 的贡献）
ascend-robot
ascend-robot成员
3月17日 评论：

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 ✅ gong-siwei, went_code (2/2) ✅ gong-siwei (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成员
3月17日 添加了label：ascend-cla/yes
Wwangyixian成员
3月17日 关联了里程碑：MindStudio 26.0.0
ascend-robot
ascend-robot成员
3月17日 评论：

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
3月17日 添加了label：docs-ci-pipeline-running
此处折叠了44条消息 查看更多
went_code
went_code成员
3月17日 评论：

/lgtm

likedislike
G
gong-siwei成员
3月17日 评论：

/lgtm
/approve

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

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: gong-siwei.
Reviewers who wrote a comment of /lgtm are: went_code, gong-siwei.

likedislike
ascend-robotascend-robot成员
3月17日 合入了pull request