Skip to content
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

SVM bind_to_queue and unbind_queue cannot be safely used #645

Open
inducer opened this issue Oct 21, 2022 · 0 comments
Open

SVM bind_to_queue and unbind_queue cannot be safely used #645

inducer opened this issue Oct 21, 2022 · 0 comments
Labels

Comments

@inducer
Copy link
Owner

inducer commented Oct 21, 2022

Since they modify the allocation in place, they're unusable e.g. in the array. Consider the expression ary2 = ary.with_queue(None), which one might think should unbind underlying SVM from the queue. But ary is still ailve, and both ary and ary2 refer to the same allocation.

Instead, we need some sort of "holder" object that inserts a queue into a list of queues that need to be synchronized with when freeing the SVM.

cc @matthiasdiener

@inducer inducer added the bug label Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant