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

Concurrency check #431

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from
Draft

Concurrency check #431

wants to merge 13 commits into from

Conversation

inducer
Copy link
Owner

@inducer inducer commented Feb 22, 2021

inducer and others added 13 commits October 22, 2019 14:18
Added the following logic:
* in `enqueue_nd_range_kernel`, we catch all generated events for each
queue.
* those events get cleared when calling `queue.finish` or
`cl.wait_for_events`.
* if a kernel argument's queue's events are not in the current `wait_for`,
we complain.
Uses WeakSet to store events so we don't have to worry when they get
destroyed.

The check for out of sync events is also more strict: we now store the
event per (buffer, op) as well as per queue and only check events that
are still valid in both and only for the current buffer.
# Conflicts:
#   pyopencl/invoker.py
#   test/test_wrapper.py
Base automatically changed from master to main March 8, 2021 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants