[Mesa-dev] [PATCH] clover: Update the wait_count of the correct event when chaining events

Francisco Jerez currojerez at riseup.net
Wed Mar 25 12:08:41 PDT 2015


Tom Stellard <thomas.stellard at amd.com> writes:

> Cc: 10.5 10.4 <mesa-stable at lists.freedesktop.org>
> ---
>  src/gallium/state_trackers/clover/core/event.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/state_trackers/clover/core/event.cpp b/src/gallium/state_trackers/clover/core/event.cpp
> index 58de888..9d78b48 100644
> --- a/src/gallium/state_trackers/clover/core/event.cpp
> +++ b/src/gallium/state_trackers/clover/core/event.cpp
> @@ -67,7 +67,7 @@ event::signalled() const {
>  void
>  event::chain(event &ev) {
>     if (wait_count) {
> -      ev.wait_count++;
> +      wait_count++;

ev0->chain(ev1) establishes a dependency between ev0 and ev1 so that ev1
is signaled after ev0, while doing that only the number of unmet
dependencies of ev1 (wait_count) can be incremented by one, so the code
was already doing the right thing.

>        _chain.push_back(ev);
>     }
>     ev.deps.push_back(*this);
> -- 
> 2.0.4
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 212 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150325/17037438/attachment.sig>


More information about the mesa-dev mailing list