已合并
lazy query events #30998
liubuyu1创建于 2月25日
lazy query events #30998
已合并
Pull Request已成功合入, 合并人@ascend-robot
(感谢 liubuyu1 的贡献)AtlasAccount
2月25日 评论:
2月25日 评论:
ascend-robot
2月25日 评论:
2月25日 评论:
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 |
|---|---|---|
| docs | ✅ liujunzhu, molly123321 (2/2) | ✅ liujunzhu (1/1) |
| repo-Ascend/pytorch | ✅ liujunzhu, hbhu_bin (2/2) | ✅ liujunzhu (1/1) |
| test | ✅ liujunzhu, hbhu_bin (2/2) | ✅ liujunzhu (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)


2月25日 添加了label:ascend-cla/no
liubuyu1
2月25日 评论:
2月25日 评论:
compile


此处折叠了282条消息 查看更多
3月4日 合入了pull request
ascend-robot
3月4日 评论:
3月4日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: liujunzhu.
Reviewers who wrote a comment of /lgtm are: liujunzhu, hbhu_bin, molly123321.


ascend-robot
3月4日 评论:
3月4日 评论:
Pull Request 已合并或已关闭。
If you want to solve this problem, you can click here to do it in the FAQs.


ascend-robot
3月4日 评论:
3月4日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: liujunzhu.
Reviewers who wrote a comment of /lgtm are: molly123321, liujunzhu, hbhu_bin.


ascend-robot
3月4日 评论:
3月4日 评论:
Pull Request 已合并或已关闭。
If you want to solve this problem, you can click here to do it in the FAQs.


【合入来源】
【修改方案】
内存申请流程中,将原来的每次申请即查询event,修改为event数量达到上限后再查询,降低查询次数,提升host性能。
【资料变更】
文档中新增环境变量
PYTORCH_NPU_ALLOC_CONF=multi_stream_lazy_reclaim:True说明。【接口变更】
新增环境变量
export PYTORCH_NPU_ALLOC_CONF=multi_stream_lazy_reclaim:True,控制event延迟查询功能开启。【功能验证】
新增测试用例,使用profiling功能对比功能开启前后event查询次数

【CheckList】