已合并
新增 Host 诊断调优工具 #109
rozwel_dx创建于 3月5日
新增 Host 诊断调优工具 #109
已合并
rozwel_dx创建于 3月5日
rozwel_dx
rozwel_dx
3月5日

PR 合入模板

注:经过自检不涉及的可标注“不涉及”或直接打勾,特殊情况请文字备注。不符合规范的 PR 不允许合入,请(后备)commit 注意。


1. 修改描述

  • 修改原因: 绑核能力生态建设(推理及自定义绑核、trace统计分析部分)
  • 修改内容: 新增绑核(支持自定义)功能

2. 功能验证


3. 分支合并要求


3. 代码检视

  • 要求:

    • 合入代码超过 200 行,需三人以上会议检视。
    • 检视密度≥1个/100行。
    • 检视缺陷密度未达要求需提供说明。
    • 大于 1000 行代码原则上不允许合入,需进行备案。
  • 检视意见数:____ 条 (请填写本次检视的意见总数,用于commit合入前审视)


4. 安全自检

Python、C++:

C++:


5. 变更知会


likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 rozwel_dx 的贡献)
ascend-robot
ascend-robot成员
3月5日 评论:

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/msprof chenhao_1209, mei-feiyao (2/2) chenhao_1209 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

likedislike
ascend-robotascend-robot成员
3月5日 添加了label:ascend-cla/no
rozwel_dxrozwel_dx
3月6日 推送  2 个提交:d28a5063-新增 Host 诊断调优工具,d3145a45-Merge branch 'master' of https://gitcode.com/rozwel_dx/msprof
ascend-robotascend-robot成员
3月6日 添加了label:stat/needs-squash
此处折叠了191条消息 查看更多
ascend-robot
ascend-robot成员
6月1日 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
xieanran
xieanran成员6月1日进行代码检视1
misc/host_analyzer/cpu_binder/cpu_binder.py
@@ -0,0 +366,4 @@
366+ return ",".join(mask_parts)
367+ 
368+ @staticmethod
369+ def get_main_pid_from_docker(pid: int) -> int:
xieanran
xieanran6月1日评论:

[codereview][一般] get_main_pid_from_docker通过grep解析文件内容额外创建进程开销较大;建议直接读取/proc//status文件解析,提高执行效率。

likedislike
xieanranxieanran成员
6月1日 解决了最后一个问题
ascend-robot
ascend-robot成员
6月1日 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
ascend-robot
ascend-robot成员
6月1日 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike