[1/1] qemugl: Fix GL apps failure on Ubuntu 11 host with nVidia GLX driver

Submitted by Zhai, Edwin on April 1, 2012, 2:24 a.m. | Patch ID: 24991


Message ID eade13c5319cbe9e1a640501f8520b80a3210604.1333246894.git.edwin.zhai@intel.com
State Accepted
Commit b3ccc630e6c12a75111b1f7ca877e17d8d4e1dc7
Headers show

Commit Message

Zhai, Edwin April 1, 2012, 2:24 a.m.
From: Zhai Edwin <edwin.zhai@intel.com>

Previous version of nVidia GLX driver in Ubuntu 10 cause qemu segfault, so we
fall back to Mesa GLX driver if detecting nVidia driver installed. From Ubuntu
11, nVidia GLX driver works well, while previous work around cause GL apps
failure. So this work around is limited in Ubuntu 10 only, and will be removed
in future.

[YOCTO #1886] got fixed.

Signed-off-by: Zhai Edwin <edwin.zhai@intel.com>
 scripts/runqemu-internal |   11 ++++++++---
 1 files changed, 8 insertions(+), 3 deletions(-)

Patch hide | download patch | download mbox

diff --git a/scripts/runqemu-internal b/scripts/runqemu-internal
index 73b8b41..92f245a 100755
--- a/scripts/runqemu-internal
+++ b/scripts/runqemu-internal
@@ -467,11 +467,15 @@  cat << EOM
 WARNING: nVidia proprietary OpenGL libraries detected.
 nVidia's OpenGL libraries are known to have compatibility issues with qemu,
 resulting in a segfault. Please uninstall these drivers or ensure the mesa libGL
-libraries precede nvidia's via LD_PRELOAD(Already do it on Ubuntu).
+libraries precede nvidia's via LD_PRELOAD(Already do it on Ubuntu 10).
 # Automatically use Ubuntu system's mesa libGL, other distro can add its own path
-    if grep -i ubuntu /etc/lsb-release &> /dev/null
+if grep -i ubuntu /etc/lsb-release &> /dev/null
+    # precede nvidia's driver on Ubuntu 10
+    UBUNTU_MAIN_VERSION=`cat /etc/lsb-release |grep DISTRIB_RELEASE |cut -d=  -f 2| cut -d. -f 1`
+    if [ $UBUNTU_MAIN_VERSION -eq 10 ];
         if test -e /usr/lib/libGL.so
@@ -482,10 +486,11 @@  EOM
-        echo "Skip nVidia's libGL on Ubuntu!"
+        echo "Skip nVidia's libGL on Ubuntu 10!"
 echo "Running $QEMU..."
 # -no-reboot is a mandatory option - see bug #100