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
signature.asc
Description: PGP signature
_______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/piglit