-
Notifications
You must be signed in to change notification settings - Fork 5.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update stream rebind #51412
Update stream rebind #51412
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
…Paddle into update_stream_rebind
} | ||
} | ||
|
||
void ResourceManager::Increase(void* stream) { | ||
PADDLE_ENFORCE_EQ(ref_count_.count(stream), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rebind加锁后,这里的Check还是要去掉吗?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
… update_stream_rebind
Sorry to inform you that 6db7e32's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
Since you haven't replied for more than a year, we have closed this issue/pr. |
PR types
Function optimization
PR changes
Others
Describe
Update stream rebinding
Update stream rebinding to support