diff options
author | André Fabian Silva Delgado <emulatorman@parabola.nu> | 2016-08-07 21:20:27 -0300 |
---|---|---|
committer | André Fabian Silva Delgado <emulatorman@parabola.nu> | 2016-08-07 21:20:27 -0300 |
commit | dfccf4f7b761b2570ff373e7e31988e18edf5988 (patch) | |
tree | f1581337a75203851f044f84172f780cdd08374a /libre/mesa-libcl/0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch | |
parent | aef55ac8b4d632b9b320a0e97aa280fc2dca2dcb (diff) |
mesa-libcl-12.0.1-1: updating version
Diffstat (limited to 'libre/mesa-libcl/0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch')
-rw-r--r-- | libre/mesa-libcl/0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch | 76 |
1 files changed, 76 insertions, 0 deletions
diff --git a/libre/mesa-libcl/0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch b/libre/mesa-libcl/0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch new file mode 100644 index 000000000..3f2391dbc --- /dev/null +++ b/libre/mesa-libcl/0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch @@ -0,0 +1,76 @@ +From 9ce52baf7f479d506bcea17a371dcf2fad92f139 Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Christian=20K=C3=B6nig?= <christian.koenig@amd.com> +Date: Wed, 13 Jul 2016 14:54:31 +0200 +Subject: [PATCH] st/mesa: fix reference counting bug in st_vdpau +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Otherwise we leak the resources created for the DMA-buf descriptors. + +Signed-off-by: Christian König <christian.koenig@amd.com> +Cc: 12.0 <mesa-stable@lists.freedesktop.org> +Tested-and-Reviewed by: Leo Liu <leo.liu@amd.com> +Ack-by: Tom St Denis <tom.stdenis@amd.com> +--- + src/mesa/state_tracker/st_vdpau.c | 10 ++++++++-- + 1 file changed, 8 insertions(+), 2 deletions(-) + +diff --git a/src/mesa/state_tracker/st_vdpau.c b/src/mesa/state_tracker/st_vdpau.c +index dffa52f..4f599dd 100644 +--- a/src/mesa/state_tracker/st_vdpau.c ++++ b/src/mesa/state_tracker/st_vdpau.c +@@ -65,6 +65,7 @@ st_vdpau_video_surface_gallium(struct gl_context *ctx, const void *vdpSurface, + + struct pipe_video_buffer *buffer; + struct pipe_sampler_view **samplers; ++ struct pipe_resource *res = NULL; + + getProcAddr = (void *)ctx->vdpGetProcAddress; + if (getProcAddr(device, VDP_FUNC_ID_VIDEO_SURFACE_GALLIUM, (void**)&f)) +@@ -82,7 +83,8 @@ st_vdpau_video_surface_gallium(struct gl_context *ctx, const void *vdpSurface, + if (!sv) + return NULL; + +- return sv->texture; ++ pipe_resource_reference(&res, sv->texture); ++ return res; + } + + static struct pipe_resource * +@@ -90,13 +92,15 @@ st_vdpau_output_surface_gallium(struct gl_context *ctx, const void *vdpSurface) + { + int (*getProcAddr)(uint32_t device, uint32_t id, void **ptr); + uint32_t device = (uintptr_t)ctx->vdpDevice; ++ struct pipe_resource *res = NULL; + VdpOutputSurfaceGallium *f; + + getProcAddr = (void *)ctx->vdpGetProcAddress; + if (getProcAddr(device, VDP_FUNC_ID_OUTPUT_SURFACE_GALLIUM, (void**)&f)) + return NULL; + +- return f((uintptr_t)vdpSurface); ++ pipe_resource_reference(&res, f((uintptr_t)vdpSurface)); ++ return res; + } + + static struct pipe_resource * +@@ -208,6 +212,7 @@ st_vdpau_map_surface(struct gl_context *ctx, GLenum target, GLenum access, + /* do we have different screen objects ? */ + if (res->screen != st->pipe->screen) { + _mesa_error(ctx, GL_INVALID_OPERATION, "VDPAUMapSurfacesNV"); ++ pipe_resource_reference(&res, NULL); + return; + } + +@@ -241,6 +246,7 @@ st_vdpau_map_surface(struct gl_context *ctx, GLenum target, GLenum access, + stObj->surface_format = res->format; + + _mesa_dirty_texobj(ctx, texObj); ++ pipe_resource_reference(&res, NULL); + } + + static void +-- +2.9.0 + |