sdl2: add patch to fix busy-loop in C library
authorPaul Cercueil <paul@crapouillou.net>
Fri, 20 Dec 2019 16:04:38 +0000 (17:04 +0100)
committerYann E. MORIN <yann.morin.1998@free.fr>
Sun, 22 Dec 2019 10:27:05 +0000 (11:27 +0100)
For some obscure reason, the order in which the libdrm/libgbm libraries
are loaded matters.

Without this fix, the first call to check_modesetting() will work and
load then unload all symbols properly, but the second call to this
function will lock up as soon as dlopen() is called on libdrm.

Swapping the order in which the libdrm and libgbm libraries are loaded
is enough to fix (or work around?) this issue.

Signed-off-by: Paul Cercueil <paul@crapouillou.net>
[yann.morin.1998@free.fr: add upstream commit URL]
Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
package/sdl2/0002-fix-busy-loop-in-dlopen.patch [new file with mode: 0644]

diff --git a/package/sdl2/0002-fix-busy-loop-in-dlopen.patch b/package/sdl2/0002-fix-busy-loop-in-dlopen.patch
new file mode 100644 (file)
index 0000000..e335f95
--- /dev/null
@@ -0,0 +1,41 @@
+
+# HG changeset patch
+# User Paul Cercueil <paul@crapouillou.net>
+# Date 1576820863 28800
+# Node ID 4f5bef55183c677d12a7da8f3392879ed50670a3
+# Parent  01f16d7f36cb9f4fa02016e57fbe915fdea71cc8
+kmsdrm: Fix busy-loop within libc's dlopen()
+For some obscure reason, the order in which the libdrm/libgbm libraries
+are loaded matters.
+
+Without this fix, the first call to check_modesetting() will work and
+load then unload all symbols properly, but the second call to this
+function will lock up as soon as dlopen() is called on libdrm.
+
+Swapping the order in which the libdrm and libgbm libraries are loaded
+is enough to fix (or work around?) this issue.
+
+Fixes #4891:
+https://bugzilla.libsdl.org/show_bug.cgi?id=4891
+
+Upstream: https://hg.libsdl.org/SDL/rev/4f5bef55183c
+
+Signed-off-by: Paul Cercueil <paul@crapouillou.net>
+[yann.morin.1998@free.fr: add upstream commit URL]
+Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
+
+diff -r 01f16d7f36cb -r 4f5bef55183c src/video/kmsdrm/SDL_kmsdrmdyn.c
+--- a/src/video/kmsdrm/SDL_kmsdrmdyn.c Thu Dec 19 21:45:44 2019 -0800
++++ b/src/video/kmsdrm/SDL_kmsdrmdyn.c Thu Dec 19 21:47:43 2019 -0800
+@@ -50,8 +50,8 @@
+ #endif
+ static kmsdrmdynlib kmsdrmlibs[] = {
+-    {NULL, SDL_VIDEO_DRIVER_KMSDRM_DYNAMIC},
+-    {NULL, SDL_VIDEO_DRIVER_KMSDRM_DYNAMIC_GBM}
++    {NULL, SDL_VIDEO_DRIVER_KMSDRM_DYNAMIC_GBM},
++    {NULL, SDL_VIDEO_DRIVER_KMSDRM_DYNAMIC}
+ };
+ static void *
+