diff options
5 files changed, 4 insertions, 218 deletions
diff --git a/libre-multilib/lib32-opencl-mesa/0001-Mesa-dev-st_glsl_to_tgsi-overlord-fix.patch b/libre-multilib/lib32-opencl-mesa/0001-Mesa-dev-st_glsl_to_tgsi-overlord-fix.patch deleted file mode 100644 index 0ba38ca09..000000000 --- a/libre-multilib/lib32-opencl-mesa/0001-Mesa-dev-st_glsl_to_tgsi-overlord-fix.patch +++ /dev/null @@ -1,17 +0,0 @@ -diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp -index 7564119..38e2c4a 100644 ---- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp -+++ b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp -@@ -6058,7 +6058,11 @@ st_translate_program( - inputSemanticName[i], inputSemanticIndex[i], - interpMode[i], 0, interpLocation[i], - array_id, array_size); -- i += array_size - 1; -+ -+ GLuint base_attr = inputSlotToAttr[i]; -+ while (i + 1 < numInputs && -+ inputSlotToAttr[i + 1] < base_attr + array_size) -+ ++i; - } - else { - t->inputs[i] = ureg_DECL_fs_input_cyl_centroid(ureg, diff --git a/libre-multilib/lib32-opencl-mesa/0001-i965-import-prime-buffers-in-the-current-context-not.patch b/libre-multilib/lib32-opencl-mesa/0001-i965-import-prime-buffers-in-the-current-context-not.patch deleted file mode 100644 index 0b37366f6..000000000 --- a/libre-multilib/lib32-opencl-mesa/0001-i965-import-prime-buffers-in-the-current-context-not.patch +++ /dev/null @@ -1,66 +0,0 @@ -From e180e9e3c830d3611a6cf7d32e988b4c28d20942 Mon Sep 17 00:00:00 2001 -From: Martin Peres <martin.peres@linux.intel.com> -Date: Wed, 3 Aug 2016 12:58:23 +0300 -Subject: [PATCH] i965: import prime buffers in the current context, not screen - -This mirrors the codepath taken by DRI2 in IntelSetTexBuffer2() and -fixes many applications when using DRI3: - - Totem with libva on hw-accelerated decoding - - obs-studio, using Window Capture (Xcomposite) as a Source - - gstreamer with VAAPI - -Cc: mesa-stable@lists.freedesktop.org -Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=71759 -Signed-off-by: Martin Peres <martin.peres@linux.intel.com> ---- - src/mesa/drivers/dri/i965/intel_screen.c | 25 +++++++++++++++++++++++-- - 1 file changed, 23 insertions(+), 2 deletions(-) - -diff --git a/src/mesa/drivers/dri/i965/intel_screen.c b/src/mesa/drivers/dri/i965/intel_screen.c -index ae51c40..169d578 100644 ---- a/src/mesa/drivers/dri/i965/intel_screen.c -+++ b/src/mesa/drivers/dri/i965/intel_screen.c -@@ -702,8 +702,11 @@ intel_create_image_from_fds(__DRIscreen *screen, - int *fds, int num_fds, int *strides, int *offsets, - void *loaderPrivate) - { -+ GET_CURRENT_CONTEXT(ctx); - struct intel_screen *intelScreen = screen->driverPrivate; -+ struct brw_context *brw = brw_context(ctx); - struct intel_image_format *f; -+ dri_bufmgr *bufmgr; - __DRIimage *image; - int i, index; - -@@ -744,8 +747,26 @@ intel_create_image_from_fds(__DRIscreen *screen, - size = end; - } - -- image->bo = drm_intel_bo_gem_create_from_prime(intelScreen->bufmgr, -- fds[0], size); -+ /* Let's import the buffer into the current context instead of the current -+ * screen as some applications like gstreamer, totem, or obs create multiple -+ * X connections which end up creating multiple screens and thus multiple -+ * buffer managers. They then proceed to use a different X connection than -+ * the one used by the currently-bound context to call GLXBindTexImageExt() -+ * which should then import the buffer in the current bound context and not -+ * the current screen. This is done properly upstairs for texture management -+ * so we need to mirror this behaviour if we don't want the kernel rejecting -+ * our pushbuffers as the buffer would not have been imported by the same -+ * buffer manager that sent the pushbuffer referencing it. -+ * -+ * If there is no context currently bound, then revert to using the screen's -+ * buffer manager and hope for the best... -+ */ -+ if (brw) -+ bufmgr = brw->bufmgr; -+ else -+ bufmgr = intelScreen->bufmgr; -+ -+ image->bo = drm_intel_bo_gem_create_from_prime(bufmgr, fds[0], size); - if (image->bo == NULL) { - free(image); - return NULL; --- -2.9.0 - diff --git a/libre-multilib/lib32-opencl-mesa/0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch b/libre-multilib/lib32-opencl-mesa/0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch deleted file mode 100644 index 3f2391dbc..000000000 --- a/libre-multilib/lib32-opencl-mesa/0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch +++ /dev/null @@ -1,76 +0,0 @@ -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 - diff --git a/libre-multilib/lib32-opencl-mesa/0002-vl-dri3-fix-a-memory-leak-from-front-buffer.patch b/libre-multilib/lib32-opencl-mesa/0002-vl-dri3-fix-a-memory-leak-from-front-buffer.patch deleted file mode 100644 index e9da7c50c..000000000 --- a/libre-multilib/lib32-opencl-mesa/0002-vl-dri3-fix-a-memory-leak-from-front-buffer.patch +++ /dev/null @@ -1,38 +0,0 @@ -From 134d6e4e4f980d738361bfcae1daedb8cc58b5df Mon Sep 17 00:00:00 2001 -From: Leo Liu <leo.liu@amd.com> -Date: Thu, 14 Jul 2016 09:19:59 -0400 -Subject: [PATCH] vl/dri3: fix a memory leak from front buffer - -Inspired by fix for mem leak of vdpau interop, resource_from_handle -set texture reference count, that need to be decreased and released, -recall there is a similar case for DRI3, that is with VA-API glx -extension, there is temporary TFP(texture from pixmap), we target it -through dma-buf. leak happens when without count down the reference. - -Checked and found with mpv vo=opengl case, there only one static TFP, -the leak happens once, but for totem player using gstreamer VA-API glx, -the dynamic TFP for each frame, so leak quite a bit. - -This fixes mem leak for mpv and totem. - -Signed-off-by: Leo Liu <leo.liu@amd.com> -Cc: "12.0" <mesa-stable@lists.freedesktop.org> ---- - src/gallium/auxiliary/vl/vl_winsys_dri3.c | 1 + - 1 file changed, 1 insertion(+) - -diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri3.c b/src/gallium/auxiliary/vl/vl_winsys_dri3.c -index f7f572e..493e645 100644 ---- a/src/gallium/auxiliary/vl/vl_winsys_dri3.c -+++ b/src/gallium/auxiliary/vl/vl_winsys_dri3.c -@@ -89,6 +89,7 @@ dri3_free_front_buffer(struct vl_dri3_screen *scrn, - { - xcb_sync_destroy_fence(scrn->conn, buffer->sync_fence); - xshmfence_unmap_shm(buffer->shm_fence); -+ pipe_resource_reference(&buffer->texture, NULL); - FREE(buffer); - } - --- -2.9.0 - diff --git a/libre-multilib/lib32-opencl-mesa/PKGBUILD b/libre-multilib/lib32-opencl-mesa/PKGBUILD index befb016c9..eea3f3286 100644 --- a/libre-multilib/lib32-opencl-mesa/PKGBUILD +++ b/libre-multilib/lib32-opencl-mesa/PKGBUILD @@ -3,7 +3,7 @@ pkgbase=mesa pkgname=lib32-opencl-$pkgbase -pkgver=12.0.1 +pkgver=12.0.2 pkgrel=1 pkgdesc='OpenCL support for mesa drivers (32-bit)' arch=('x86_64') @@ -15,18 +15,10 @@ optdepends=('opencl-headers: headers necessary for OpenCL development') # "CHROOTEXTRAPKG=(multilib-devel)" on "/etc/libretools.d/chroot.conf" file. makedepends=('python2-mako' 'dri2proto' 'dri3proto' 'glproto' 'presentproto' 'python2' 'llvm') source=(ftp://ftp.freedesktop.org/pub/$pkgbase/$pkgver/$pkgbase-$pkgver.tar.xz{,.sig} - remove-libpthread-stubs.patch - 0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch - 0002-vl-dri3-fix-a-memory-leak-from-front-buffer.patch - 0001-Mesa-dev-st_glsl_to_tgsi-overlord-fix.patch - 0001-i965-import-prime-buffers-in-the-current-context-not.patch) -sha512sums=('11898336478d1f5bd3946d695b9599647c0b9084ad8890b4fa89daa922a217e5ca6d8b72f5801e362c3dd7723382f2daaa655b54f65a6d088ad8cfdb85f10023' + remove-libpthread-stubs.patch) +sha512sums=('61fcbc3af16fff53f4877c5731c10fffa32a75c2eda9706d27ffdf00bd65a2c36b4ece165049b02c7269088f0755d93f6646ebc031d4b3fc838e316a5fcb2617' 'SKIP' - '6349de16a2a11b8540d82a78cd15574ea92e0a22f2e184b93311060f72cc3749efba67be464a1b399493035c6e389017d5631b0be2e8ef5da115b137e26c33df' - '499312eb048e32b27820eebeb243df926226f3c2247bfc9b19b28664eaffc7e524cc1dc760d701159a46f37be46e49e956b27de473c465cecbdf0db6d6b97fe0' - 'e8b6c75565b7f72f5bc073469fef50e2b04311dc235885f69ad15591e338601273dec99bfd8cbb85f0cc5b3ecf4e4ac8c6ef64225e2ed68504081af3944f88a8' - 'fd677879e97f7d30346ef9c41e89770e06e4714b0e74c7bb3d88984038e2fb3cc7f7fb6df59469527ba2c07f8bf51959f65e8dd6dfceea9e400b44157308b816' - '8b6c5eb3024ae5095fbb86e07aa578ede106844388077b5b9d2e9b5822640424be308e34d67daff44b0c25624f2b4ff1e67fc5d21cf5dfaa07b760be2cbd1531') + '6349de16a2a11b8540d82a78cd15574ea92e0a22f2e184b93311060f72cc3749efba67be464a1b399493035c6e389017d5631b0be2e8ef5da115b137e26c33df') validpgpkeys=('8703B6700E7EE06D7A39B8D6EDAE37B02CEB490D') # Emil Velikov <emil.l.velikov@gmail.com> prepare() { @@ -38,15 +30,6 @@ prepare() { # Now mesa checks for libpthread-stubs - so remove the check patch -Np1 -i ../remove-libpthread-stubs.patch - # fix memory leaks - merged upstream - patch -Np1 -i ../0001-st-mesa-fix-reference-counting-bug-in-st_vdpau.patch - patch -Np1 -i ../0002-vl-dri3-fix-a-memory-leak-from-front-buffer.patch - # fix rendering in some games - merged upstream - patch -Np1 -i ../0001-Mesa-dev-st_glsl_to_tgsi-overlord-fix.patch - # fix https://bugs.freedesktop.org/show_bug.cgi?id=71759 / FS#50240 - # reverted because of it segfault X with modesetting/intel - #patch -Np1 -i ../0001-i965-import-prime-buffers-in-the-current-context-not.patch - autoreconf -fiv } |