[Mesa-dev] [PATCH 7/7] clover: Flush the command queue in clReleaseCommandQueue()

Francisco Jerez currojerez at riseup.net
Fri Aug 8 08:39:16 PDT 2014


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

> This is required by the spec.
>
> CC: "10.2" <mesa-stable at lists.freedesktop.org>
> ---
>  src/gallium/state_trackers/clover/api/queue.cpp | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/state_trackers/clover/api/queue.cpp b/src/gallium/state_trackers/clover/api/queue.cpp
> index a136018..06a2863 100644
> --- a/src/gallium/state_trackers/clover/api/queue.cpp
> +++ b/src/gallium/state_trackers/clover/api/queue.cpp
> @@ -58,7 +58,11 @@ clRetainCommandQueue(cl_command_queue d_q) try {
>  
>  CLOVER_API cl_int
>  clReleaseCommandQueue(cl_command_queue d_q) try {
> -   if (obj(d_q).release())
> +   auto &q = obj(d_q);
> +
> +   q.flush();
> +
> +   if (q.release())
>        delete pobj(d_q);
>  
>     return CL_SUCCESS;

For this patch:
Reviewed-by: Francisco Jerez <currojerez at riseup.net>

Thanks.

> -- 
> 1.8.1.5
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 212 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140808/df3acaad/attachment.sig>


More information about the mesa-dev mailing list