master

ascend-robot[bugfix] 多卡调试屏蔽设备同步超时并补充消息长度日志

453f94e3创建于 10 天前历史提交

文件	最后提交记录	最后更新时间
API	[feature] add clang-tidy hook to pre-commit config Co-authored-by: wiyr0<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !163 merge feature/add-clang-tidy-hook into master [feature] add clang-tidy hook to pre-commit config Created-by: wiyr0 Commit-by: wiyr0 Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** https://gitcode.com/Ascend/msot/issues/54 - **修改方案：** 新增 clang-tidy hook，使用 lldb/.clang-tidy 作为配置（通过 InheritParentConfig 继承根配置并禁用 readability-identifier-naming） - **修改内容：**  - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- ### 2. 功能验证 - [x] **功能自验截图**（请确保不体现个人信息） - [x] **冒烟是否通过**  ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** ---- See merge request: Ascend/msdebug!163	11 天前
Breakpoint	[bugfix] 修复host断点与device断点重复时,host断点取消失败导致算子断不到device侧的问题 Co-authored-by: wiyr0<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !181 merge test_host_bkpt into master [bugfix] 修复host断点与device断点重复时,host断点取消失败导致算子断不到device侧的问题 Created-by: wiyr0 Commit-by: wiyr0 Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** https://gitcode.com/Ascend/msdebug/issues/99 断点设置同时匹配host和device侧时，比如动态库和kernel侧都匹配到了。原本逻辑会去除host侧的断点，但是host侧其实运行后几乎没停下来，取消host断点就会失败，但是实际的算子跑的host断点还在，调试器管理的断点信息不在了。这就会导致host断点命中时，被误认为SIGTRAP类型的中断 - **修改方案：** 当算子运行后，不再对host断点做取消的操作，保持两个host侧和device侧的断点都在 - **修改内容：** BreakpointLocationSP BreakpointLocationList::AddLocation函数里，对host断点删除操作前，先判断process_sp是否为空，为空说明进程还没启动，可以做取消。 - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- ### 2. 功能验证 - [ ] **功能自验截图**（请确保不体现个人信息）   - [x] **冒烟是否通过**  ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** ---- See merge request: Ascend/msdebug!181	1 个月前
Commands	[feature] add clang-tidy hook to pre-commit config Co-authored-by: wiyr0<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !163 merge feature/add-clang-tidy-hook into master [feature] add clang-tidy hook to pre-commit config Created-by: wiyr0 Commit-by: wiyr0 Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** https://gitcode.com/Ascend/msot/issues/54 - **修改方案：** 新增 clang-tidy hook，使用 lldb/.clang-tidy 作为配置（通过 InheritParentConfig 继承根配置并禁用 readability-identifier-naming） - **修改内容：**  - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- ### 2. 功能验证 - [x] **功能自验截图**（请确保不体现个人信息） - [x] **冒烟是否通过**  ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** ---- See merge request: Ascend/msdebug!163	11 天前
Core	[bugfix] 修复host侧的uint8_t指针变量会展示乱码的问题 & 增加超时等待时间 Co-authored-by: wiyr0<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !176 merge fix_var into master [bugfix] 修复host侧的uint8_t指针变量会展示乱码的问题 & 增加超时等待时间 Created-by: wiyr0 Commit-by: wiyr0 Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** 原生lldb打印host uint8_t* 的变量时，会认为是char* 指针，从而打印成字符串，有概率显示成乱码。大算子传输的速度慢，会导致kernel.o还没到client侧算子下发了，从而断点不生效。 - **修改方案：** 1. 展示不展示uint8_t* 指向的内容；2. 增加超时到30s - **修改内容：** 1. 去掉只对device上的uint8_t* 的限制； 2.修改连接的超时字段 - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- ### 2. 功能验证 - [x] **功能自验截图**（请确保不体现个人信息）  - [x] **冒烟是否通过**  ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** ---- See merge request: Ascend/msdebug!176	1 个月前
DataFormatters	added all modifications made for msdebug	7 个月前
Expression	added all modifications made for msdebug	7 个月前
Host	[bugfix]修复msdebug易用性安全问题 Co-authored-by: zzzsss1234<zhanglei802@h-partners.com> # message auto-generated for no-merge-commit merge: !166 merge fix_usage into master [bugfix]修复msdebug易用性安全问题 Created-by: zzzsss1234 Commit-by: zzzsss1234 Merged-by: ascend-robot Description: ### 1. 修改描述 https://gitcode.com/Ascend/msot/issues/59 - **修改原因：** 安全易用性整改，需要符合标准，放开相关限制 - **修改方案：** skill扫描代码仓，目前测试msdebug仓库问题确认情况，扫描问题23个，易用性改进2个，确认问题2个，修复问题1个，其余问题已确认无需修改，判断结果见附件 - **修改内容：** 1.socket链接使用accept的fd 2.易用性校验时去掉属主权限的限制 - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- ### 2. 功能验证 - [ ] **功能自验截图**（请确保不体现个人信息） reakpoint 1: no locations (pending on future shared library load). WARNING: Unable to resolve breakpoint to any actual locations. (msdebug) r Process 1629062 launched: '/home/project/samples/operator/ascendc/0_introduction/3_add_kernellaunch/AddKernelInvocationNeo/ascendc_kernels_bbit' (aarch64) [Launch of Kernel add_custom_0 on Device 0] 1 location added to breakpoint 1 Process 1629062 stopped [Switching to focus on Kernel add_custom_0, CoreId 41, Type aiv] * thread #1, name = 'ascendc_kernels', stop reason = breakpoint 1.1 frame #0: 0x0000124000000624 device_debugdata_0`::add_custom_0(uint8_t *__gm__, uint8_t *__gm__, uint8_t *__gm__, uint8_t *__gm__) [inlined] KernelAdd::Compute(this=0x0000000000000000, progress=<unavailable>) at add_custom.cpp:56:9 53 AscendC::LocalTensor<half> xLocal = inQueueX.DeQue<half>(); 54 AscendC::LocalTensor<half> yLocal = inQueueY.DeQue<half>(); 55 AscendC::LocalTensor<half> zLocal = outQueueZ.AllocTensor<half>(); -> 56 AscendC::Add(zLocal, xLocal, yLocal, TILE_LENGTH); 57 outQueueZ.EnQue<half>(zLocal); 58 inQueueX.FreeTensor(xLocal); 59 inQueueY.FreeTensor(yLocal); (msdebug) - [ ] **冒烟是否通过** ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [ ] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** ---- See merge request: Ascend/msdebug!166	1 个月前
Initialization	Revert "[lldb] Implement coalescing of disjoint progress events (#84854)" This reverts commit 930f64689c1fb487714c3836ffa43e49e46aa488 as it's failing on the Linux bots.	2 年前
Interpreter	Revert "[feature] Attempt to achieve more stable interruption of a deferred process that actively utilizes the NPU" Co-authored-by: wangyixian<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !138 merge fix_ctrlc into master Revert "[feature] Attempt to achieve more stable interruption of a deferred process that actively utilizes the NPU" Created-by: wiyr0 Commit-by: wangyixian Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** 这个pr合入后，ascendc用例的ctrl-c功能失效；该pr本身是优化triton ctrl-c的功能，目前triton调试尚未支持，revert影响不大 - **修改方案：** 先revert这个pr，后续待完整修复 - **修改内容：** 直接revert指定hash - [ ] **涉及代码双合**(贴上另一个PR链接)：NA ---- ### 2. 功能验证 - [x] **功能自验截图**（请确保不体现个人信息）  - [x] **冒烟是否通过**  ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** NA - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** NA ---- ### 4. 资料修改自检 - **资料修改：** NA ---- See merge request: Ascend/msdebug!138	2 个月前
Plugins	[bugfix] 多卡调试屏蔽设备同步超时并补充消息长度日志 Co-authored-by: wiyr0<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !175 merge support_mc2 into master [bugfix] 多卡调试屏蔽设备同步超时并补充消息长度日志 Created-by: wiyr0 Commit-by: wiyr0 Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** 调试器在核函数断点处长时间停留时,上层传入的设备同步超时时间会被触发,产生同步超时告警,导致多卡调试被中断,影响调试体验。同时,接收 device 消息的日志缺少消息长度信息, 排查消息截断/异常等问题时不便。 - **修改方案：** 1. 屏蔽设备同步超时: 新增 aclrtSynchronizeDeviceWithTimeout 接口的 stub 实现,在调试期间忽略上层传入的 timeout,统一以 -1(永不超时)调用真实接口,从根本规避同步超时告警, 保证断点暂停期间不会被超时中断。 2. 补充消息长度日志: 在 AscendProcessLinux::HandleMsg 处理 device id 消息的日志中追加 size 字段输出 msg.length(),便于定位消息长度相关问题。 - **修改内容：**  - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- ### 2. 功能验证 - [ ] **功能自验截图**（请确保不体现个人信息） NA - [x] **冒烟是否通过**  ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** ---- See merge request: Ascend/msdebug!175	10 天前
Symbol	[feature] simd上支持断点类型识别问题 Co-authored-by: wangyixian<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !165 merge fix_bt into master [feature] simd上支持断点类型识别问题 Created-by: wiyr0 Commit-by: wiyr0;wangyixian Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** simd_vf上的函数都是inlined行为，不能直接通过函数类型去识别 - **修改方案：** 通过pc对应的block块，为每个block设置函数类型，若当前block没有函数类型信息，则通过父节点寻找 - **修改内容：** 为lldb/include/lldb/Symbol/Block.h里的Block类增加m_function_class字段；SymbolFileDWARF::ParseBlocksRecursive函数里为Block 设置函数类型。 - [ ] **涉及代码双合**(贴上另一个PR链接)：NA ---- ### 2. 功能验证 - [ ] **功能自验截图**（请确保不体现个人信息） - [x] **冒烟是否通过**  ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** NA - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** NA ---- ### 4. 资料修改自检 - **资料修改：** NA ---- See merge request: Ascend/msdebug!165	1 个月前
Target	[bugfix] 修复 simt_vf上的调用栈功能 Co-authored-by: wiyr0<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !188 merge fix_bt_01 into master [bugfix] 修复 simt_vf上的调用栈功能 Created-by: wiyr0 Commit-by: wiyr0 Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** simt vf里，函数调用栈深度大于等3时，调用栈功能不准。 根因： 1. 读取simt stack栈数据为空 2. 编译器的simt debug_frame信息不完整 - **修改方案：** 1. simt stack内存的读取，会从dcache先读，这里有地址映射的逻辑，调试器必须传对地址， 且每次只能读4字节，因为接下来的4个字节，需要重新映射地址。这里我们重新对SIMT_STACK做一个读取内存的实现。 2. 编译器simt debug_frame缺少lr寄存器的描述，缺少sp寄存器的描述。 - **修改内容：** 1. 新增DeviceContext::ReadSIMTStack函数，每次读取内存只读4字节。 2. RegisterContext::ReadRegisterValueFromMemory函数里，从内存里读取PC 时，如果当前是在simt vf里，那么就对PC值前后4字节反转。因为编译器存PC的时候，高地址存了PC低位，低地址存了PC高位。而我们是从低地址开始读取的。 - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- ### 2. 功能验证 - [x] **功能自验截图**（请确保不体现个人信息）  - [x] **冒烟是否通过**  ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** ---- See merge request: Ascend/msdebug!188	1 个月前
Utility	[feature] 支持simt step over功能&修复ascend info devices输出展示 Co-authored-by: wangyixian<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !146 merge fix_info_devices into master [feature] 支持simt step over功能&修复ascend info devices输出展示 Created-by: wiyr0 Commit-by: wangyixian Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** https://gitcode.com/Ascend/msdebug/issues/89 - **修改方案：** 1. vf内是可以不使用stack的，这个意味着当执行call指令后，新的栈和老的栈的cfa是一致的，此时调试协议无法区分当前pc所在位置是执行n的位置的下一行还是函数内。这里只能通过当前frame和上一个frame是否在同一个函数内，如是，则说明当前位置的frame是step in的 2. simt vf内的sp ra寄存器分别是r2, (r0,r1)，这里需要构造个虚拟的寄存器，表示(r0,r1)的组合 3. 950 simt上，每个指令最小的size是8字节，这里就需要新增一个专门解析950 simt上的指令大小的逻辑，所以需要有当前soc_type，pos_type，这些数据就直接放ArchSpec类里，简易点 3. ascend info devices命令的输出，vector core数量已经超过64，需要用两个u64表示, 这里参考友商，直接展示128bit。 - **修改内容：** 1. 新增 Ascend 950 反汇编器：添加 AscendDisassembler950 类，支持 950 芯片的分支/调用/返回指令识别，处理变长指令（4/8/16 字节），并根据 InterruptPosType 调整 SIMT 模式下指令大小。 2. InterruptPosType (pos_type) 传递链路：在 ArchSpec 中新增 m_pos_type 成员及其 getter/setter；在反汇编、步进（StepRange/StepOver）、Process 等多处从 DeviceStopInfo获取并设置 pos_type，传递给反汇编器用于判断中断位置类型。 3. SIMT 帧比较修复（ThreadPlanStepRange）：SIMT 内核下多线程共享 CFA，新增符号上下文/函数对比逻辑，避免帧被误判 4. SIMT LR 寄存器：新增 dwarf_simt_lr_ascend / lldb_simt_lr_ascend，ReadSimtLr 读取 r0+r1 组成链接寄存器；在 RegisterContextUnwind 中，SIMT 模式下覆盖 RA 寄存器为 dwarf_simt_lr_ascend。 5. ascend info devices 命令重构：改用 PrettyPrintTable 表格化输出，高亮当前设备行；移除旧的 WriteLine 方法。 6. 反汇编接口扩展：所有 Ascend 反汇编器的 GetInstruction 方法新增 InterruptPosType pos_type 参数；GetBit 校验从 != 4 改为 < 4 以兼容变长指令。 7. 其他：hiipu64 默认 cal 对齐从 4 改为 16；950 设备信息忽略 aic_bitmap1；新增若干调试 打印。 - [x] **涉及代码双合**(贴上另一个PR链接)：不涉及. ---- ### 2. 功能验证 - [x] **功能自验截图**（请确保不体现个人信息）   - [x] **冒烟是否通过**  ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [x] **是否具备UT测试用例看护** 不影响 - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** 不影响 ---- See merge request: Ascend/msdebug!146	1 个月前
Version	[refactor] version info redesigned Co-authored-by: sedoykin<sedoykin.alexey@huawei.com> # message auto-generated for no-merge-commit merge: !100 merge version_info_redesign into master [refactor] version info redesigned Created-by: sedoykin Commit-by: sedoykin Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：Improving user experience** - **修改方案：Changed start banner for regular run of msdebug, changed visual layout for --version mode run of msdebug** - **修改内容：** - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- ### 2. 功能验证 - [ ] **功能自验截图**   - [ ] **冒烟是否通过** ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [ ] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** ---- **IR:** [https://gitcode.com/Ascend/msdebug/issues/41](url) See merge request: Ascend/msdebug!100	3 个月前
CMakeLists.txt	[lldb] Make lldbVersion a full fledged library Because of its dependency on clang (and potentially other compilers downstream, such as swift) lldb_private::GetVersion already lives in its own library called lldbBase. Despite that, its implementation was spread across unrelated files. This patch improves things by introducing a Version library with its own directory, header and implementation file. The benefits of this patch include: - We can get rid of the ugly quoting macros. - Other parts of LLDB can read the version number from lldb/Version/Version.inc. - The implementation can be swapped out for tools like lldb-server than don't need to depend on clang at all. Differential revision: https://reviews.llvm.org/D115211	4 年前