[Mesa-dev] [PATCH] vl: use a separate context for shader based decode v2

From: Christian König This makes VDPAU thread save again. v2: fix some memory leaks reported by Aaron Watry. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 184 ++- src/gallium/auxiliary/vl/vl_mpeg12_decoder.h | 1 + 2 files changed

Re: [Mesa-dev] [PATCH] vl: use a separate context for shader based decode

On Wed, Nov 6, 2013 at 8:13 AM, Christian König wrote: > From: Christian König > > This makes VDPAU thread save again. > > Signed-off-by: Christian König > --- > src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 180 > ++- > src/gallium/auxiliary/vl/vl_mpeg12_decoder.h |

Re: [Mesa-dev] [PATCH] vl: use a separate context for shader based decode

I haven't looked in-depth at the rest of the patch, but I don't see anywhere in vl_mpeg12_destroy that you are actually destroying the context that you create in vl_create_mpeg12_decoder. Would I be correct in assuming that this is leaked memory? --Aaron On Wed, Nov 6, 2013 at 8:13 AM, Christian

Re: [Mesa-dev] [PATCH] vl: use a separate context for shader based decode

Ups, indeed. That line is still in my code directory not amended to the patch. Thanks for the info, will send out a v2 soon. Anything else I should take care of? Christian. Am 06.11.2013 16:38, schrieb Aaron Watry: I haven't looked in-depth at the rest of the patch, but I don't see anywhere

[Mesa-dev] [PATCH] vl: use a separate context for shader based decode

From: Christian König This makes VDPAU thread save again. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 180 ++- src/gallium/auxiliary/vl/vl_mpeg12_decoder.h | 1 + 2 files changed, 120 insertions(+), 61 deletions(-) diff --git a/s