Jan Vesely <jan.ves...@rutgers.edu> writes:

> On Thu, 2017-03-16 at 19:17 -0400, Jan Vesely wrote:
>> Signed-off-by: Jan Vesely <jan.ves...@rutgers.edu>
>> ---
>>  tests/cl/custom/buffer-flags.c | 2 ++
>>  1 file changed, 2 insertions(+)
>> 
>> diff --git a/tests/cl/custom/buffer-flags.c b/tests/cl/custom/buffer-flags.c
>> index 115fb92..1429de0 100644
>> --- a/tests/cl/custom/buffer-flags.c
>> +++ b/tests/cl/custom/buffer-flags.c
>> @@ -173,6 +173,8 @@ buffer_test(piglit_cl_context *ctx,
>>  
>>      /* cleanup */
>>  cleanup:
>> +    if (result != out_buffer)
>> +            clEnqueueUnmapMemObject(context->command_queues[0], out_buffer, 
>> result, 0, NULL, NULL);
>>      clReleaseMemObject(in_buffer);
>>      clReleaseMemObject(out_buffer);
>>      clReleaseKernel(kernel);
>
> ping.
> this fixes test behaviour that was the motivation for clover pipe
> reference counting (14b543bd)
>
> are you interested in OCL piglit patches? should I just drop you from
> cc?
>

I'm not familiar with the code of this piglit test so I cannot give you
an informed review, but feel free to add my

Acked-by: Francisco Jerez <curroje...@riseup.net>

if that's useful to you...

> Jan

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to