* [PATCH 1/1] qemu: Add an option to remove host sdl/gl checking
2012-05-14 14:00 [PATCH 0/1] qemu: Add option to disable GL acceleration, Edwin, May14, 2012 edwin.zhai
@ 2012-05-14 14:00 ` edwin.zhai
2012-05-22 1:33 ` [PATCH 0/1] qemu: Add option to disable GL acceleration, Edwin, May14, 2012 Zhai, Edwin
1 sibling, 0 replies; 3+ messages in thread
From: edwin.zhai @ 2012-05-14 14:00 UTC (permalink / raw)
To: openembedded-core
From: Zhai Edwin <edwin.zhai@intel.com>
Add an PACKAGECONFIG in qemu to disable GL acceleration:
* By default configure try best to enable GL acceleration and fail when missing
host dependency(libSDL and libGL).
* End user can also choose to turn off GL capability, thus remove the host
dependence in building.
[YOCTO #2407] got fixed.
Signed-off-by: Zhai Edwin <edwin.zhai@intel.com>
---
.../qemu/qemu-0.15.1/opengl-disable-option.patch | 172 ++++++++++++++++++++
meta/recipes-devtools/qemu/qemu.inc | 23 +---
meta/recipes-devtools/qemu/qemu_0.15.1.bb | 1 +
3 files changed, 176 insertions(+), 20 deletions(-)
create mode 100644 meta/recipes-devtools/qemu/qemu-0.15.1/opengl-disable-option.patch
diff --git a/meta/recipes-devtools/qemu/qemu-0.15.1/opengl-disable-option.patch b/meta/recipes-devtools/qemu/qemu-0.15.1/opengl-disable-option.patch
new file mode 100644
index 0000000..d0c5f23
--- /dev/null
+++ b/meta/recipes-devtools/qemu/qemu-0.15.1/opengl-disable-option.patch
@@ -0,0 +1,172 @@
+Add an option gl-accel to disable GL acceleration:
+* When enabled, configure try best to enable GL acceleration and fail when
+ missing host dependency(libSDL and libGL), which is the default.
+* When disabled, end user choose to turn off GL capability, thus remove the
+ host dependence in building.
+
+Upstream-Status: Inappropriate [other] - depends on GL patch
+
+Signed-off-by: Zhai Edwin <edwin.zhai@intel.com>
+
+Index: qemu-0.15.1/Makefile.target
+===================================================================
+--- qemu-0.15.1.orig/Makefile.target 2012-05-14 21:23:36.000000000 +0800
++++ qemu-0.15.1/Makefile.target 2012-05-14 21:26:19.000000000 +0800
+@@ -79,15 +79,20 @@
+ libobj-y += cpuid.o
+ endif
+ libobj-$(CONFIG_NEED_MMU) += mmu.o
++
+ ifndef CONFIG_LINUX_USER
++ifdef CONFIG_GL_ACCEL
+ libobj-$(TARGET_I386) += helper_opengl.o opengl_exec.o
+ libobj-$(TARGET_X86_64) += helper_opengl.o opengl_exec.o
+ else
+-ifdef CONFIG_SDL
+ libobj-$(TARGET_I386) += dummygl.o
+ libobj-$(TARGET_X86_64) += dummygl.o
+-endif
++endif #CONFIG_GL_ACCEL
++else
++libobj-$(TARGET_I386) += dummygl.o
++libobj-$(TARGET_X86_64) += dummygl.o
+ endif #CONFIG_LINUX_USER
++
+ libobj-$(TARGET_ARM) += dummygl.o
+ libobj-$(TARGET_MIPS) += dummygl.o
+ libobj-$(TARGET_MIPS64) += dummygl.o
+@@ -262,8 +267,10 @@
+ obj-i386-$(CONFIG_SPICE) += qxl.o qxl-logger.o qxl-render.o
+
+ ifeq ($(TARGET_BASE_ARCH), i386)
++ifdef CONFIG_GL_ACCEL
+ QEMU_CFLAGS += -DTARGET_OPENGL_OK
+ endif
++endif
+
+ # shared objects
+ obj-ppc-y = ppc.o
+@@ -409,8 +416,6 @@
+
+ monitor.o: hmp-commands.h qmp-commands.h
+
+-LIBS += -lGL -lGLU
+-
+ $(obj-y) $(obj-$(TARGET_BASE_ARCH)-y): $(GENERATED_HEADERS)
+
+ obj-y += $(addprefix ../, $(common-obj-y))
+Index: qemu-0.15.1/configure
+===================================================================
+--- qemu-0.15.1.orig/configure 2012-05-14 21:23:34.000000000 +0800
++++ qemu-0.15.1/configure 2012-05-14 21:23:36.000000000 +0800
+@@ -179,6 +179,7 @@
+ smartcard_nss=""
+ usb_redir=""
+ opengl=""
++gl_accel="yes"
+ guest_agent="yes"
+
+ # parse CC options first
+@@ -739,6 +740,10 @@
+ ;;
+ --enable-opengl) opengl="yes"
+ ;;
++ --disable-gl-accel) gl_accel="no"
++ ;;
++ --enable-gl-accel) gl_accel="yes"
++ ;;
+ --*dir)
+ ;;
+ --disable-rbd) rbd="no"
+@@ -2016,6 +2021,39 @@
+ fi
+ fi
+
++#####################################################
++# GL acceleration probe depending on gl, glu and sdl
++if test "$gl_accel" != "no" ; then
++ if test "$sdl" = "no" ; then
++ gl_accel=no
++ echo "libSDL and header no found to build opengl acceleration for qemu-native.
++ Ubuntu package names are: libsdl1.2-dev.
++ Fedora package names are: SDL-devel."
++ exit 1;
++ fi
++
++ gl_accel_libs="-lGL -lGLU"
++ cat > $TMPC << EOF
++#include <X11/Xlib.h>
++#include <GL/gl.h>
++#include <GL/glx.h>
++#include <GL/glu.h>
++int main(void) { GL_VERSION; return 0; }
++EOF
++ if compile_prog "" "-lGL -lGLU" ; then
++ gl_accel=yes
++ libs_softmmu="$gl_accel_libs $libs_softmmu"
++ else
++ feature_not_found "gl_accel"
++ gl_accel=no
++ gl_accel_libs=
++ echo "libGL.so and libGLU.so not found to build opengl acceleration for qemu-native.
++ Ubuntu package names are: libgl1-mesa-dev, libglu1-mesa-dev and libsdl1.2-dev.
++ Fedora package names are: mesa-libGL mesa-libGLU SDL-devel."
++ exit 1;
++ fi
++fi
++
+ #
+ # Check for xxxat() functions when we are building linux-user
+ # emulator. This is done because older glibc versions don't
+@@ -2717,6 +2755,7 @@
+ echo "nss used $smartcard_nss"
+ echo "usb net redir $usb_redir"
+ echo "OpenGL support $opengl"
++echo "GL acceleration support $gl_accel"
+ echo "build guest agent $guest_agent"
+
+ if test $sdl_too_old = "yes"; then
+@@ -3020,6 +3059,10 @@
+ echo "CONFIG_OPENGL=y" >> $config_host_mak
+ fi
+
++if test "$gl_accel" = "yes" ; then
++ echo "CONFIG_GL_ACCEL=y" >> $config_host_mak
++fi
++
+ # XXX: suppress that
+ if [ "$bsd" = "yes" ] ; then
+ echo "CONFIG_BSD=y" >> $config_host_mak
+Index: qemu-0.15.1/qemu-char.c
+===================================================================
+--- qemu-0.15.1.orig/qemu-char.c 2012-05-14 21:23:35.000000000 +0800
++++ qemu-0.15.1/qemu-char.c 2012-05-14 21:23:36.000000000 +0800
+@@ -2387,7 +2387,6 @@
+ return d->outbuf_size;
+ }
+
+-#define TARGET_OPENGL_OK
+ #if defined(TARGET_OPENGL_OK)
+ static uint8_t buffer[32];
+ static int buffer_len;
+@@ -2447,7 +2446,7 @@
+ return chr;
+ }
+ #else
+-#define qemu_chr_open_opengl() 0
++#define qemu_chr_open_opengl NULL
+ #endif
+
+ QemuOpts *qemu_chr_parse_compat(const char *label, const char *filename)
+Index: qemu-0.15.1/vl.c
+===================================================================
+--- qemu-0.15.1.orig/vl.c 2012-05-14 21:23:35.000000000 +0800
++++ qemu-0.15.1/vl.c 2012-05-14 21:23:36.000000000 +0800
+@@ -2067,7 +2067,6 @@
+ return popt;
+ }
+
+-#define TARGET_OPENGL_OK
+ int main(int argc, char **argv, char **envp)
+ {
+ const char *gdbstub_dev = NULL;
diff --git a/meta/recipes-devtools/qemu/qemu.inc b/meta/recipes-devtools/qemu/qemu.inc
index 93325c3..75ff962 100644
--- a/meta/recipes-devtools/qemu/qemu.inc
+++ b/meta/recipes-devtools/qemu/qemu.inc
@@ -19,26 +19,6 @@ EXTRA_OECONF += "--target-list=${@get_qemu_target_list(d)} --disable-werror --di
inherit autotools
-# For our gl powered QEMU you need libGL and SDL headers
-do_configure_prepend_virtclass-native() {
- libgl='no'
- libsdl='no'
-
- test -e /usr/lib/libGL.so -a -e /usr/lib/libGLU.so && libgl='yes'
- test -e /usr/lib64/libGL.so -a -e /usr/lib64/libGLU.so && libgl='yes'
- test -e /usr/lib/*-linux-gnu/libGL.so -a -e /usr/lib/*-linux-gnu/libGLU.so && libgl='yes'
-
- test -e /usr/lib/pkgconfig/sdl.pc -o -e /usr/lib64/pkgconfig/sdl.pc -o -e /usr/include/SDL/SDL.h && libsdl='yes'
-
-
- if [ "$libsdl" != 'yes' -o "$libgl" != 'yes' ]; then
- echo "You need libGL.so and libGLU.so to exist in your library path and the development headers for SDL installed to build qemu-native.
- Ubuntu package names are: libgl1-mesa-dev, libglu1-mesa-dev and libsdl1.2-dev.
- Fedora package names are: mesa-libGL mesa-libGLU SDL-devel."
- exit 1;
- fi
-}
-
do_configure() {
# Handle distros such as CentOS 5 32-bit that do not have kvm support
KVMOPTS="--disable-kvm"
@@ -57,6 +37,9 @@ do_install () {
install -m 0755 ${WORKDIR}/powerpc_rom.bin ${D}${datadir}/qemu
}
+PACKAGECONFIG ??= "gl"
+PACKAGECONFIG[gl] = "--enable-gl-accel,--disable-gl-accel,,"
+
DEPENDS_virtclass-native = "zlib-native alsa-lib-native glib-2.0-native"
DEPENDS_virtclass-nativesdk = "zlib-nativesdk libsdl-nativesdk glib-2.0-nativesdk \
${@base_contains('DISTRO_FEATURES', 'x11', 'qemugl-nativesdk', '', d)}"
diff --git a/meta/recipes-devtools/qemu/qemu_0.15.1.bb b/meta/recipes-devtools/qemu/qemu_0.15.1.bb
index e90f339..bdffcf4 100644
--- a/meta/recipes-devtools/qemu/qemu_0.15.1.bb
+++ b/meta/recipes-devtools/qemu/qemu_0.15.1.bb
@@ -31,6 +31,7 @@ QEMUGLPATCHES = "\
file://qemugl-fix.patch \
file://opengl-sdl-fix.patch \
file://opengl-args-copy-fix.patch \
+ file://opengl-disable-option.patch \
"
SRC_URI_append_virtclass-native = "\
--
1.7.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH 0/1] qemu: Add option to disable GL acceleration, Edwin, May14, 2012
2012-05-14 14:00 [PATCH 0/1] qemu: Add option to disable GL acceleration, Edwin, May14, 2012 edwin.zhai
2012-05-14 14:00 ` [PATCH 1/1] qemu: Add an option to remove host sdl/gl checking edwin.zhai
@ 2012-05-22 1:33 ` Zhai, Edwin
1 sibling, 0 replies; 3+ messages in thread
From: Zhai, Edwin @ 2012-05-22 1:33 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
Ping...
On Mon, May 14, 2012 at 10:00:58PM +0800, edwin.zhai@intel.com wrote:
> From: Zhai Edwin <edwin.zhai@intel.com>
>
> All,
> This patch add an PACKAGECONFIG in qemu to disable GL acceleration:
> * By default configure try best to enable GL acceleration and fail when missing
> host dependency(libSDL and libGL).
> * End user can also choose to turn off GL capability, thus remove the host
> dependence(libSDL&libGL) in building.
>
> [YOCTO #2407] got fixed.
>
> Pls. help to review and pull
>
>
> The following changes since commit 54f38111868775c85bfe921592d8443a61952c62:
>
> pango: Fix modules load failure in multilib environment (2012-05-14 21:50:55 +0800)
>
> are available in the git repository at:
> git://git.pokylinux.org/poky-contrib gzhai/fix2
> http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=gzhai/fix2
>
> Zhai Edwin (1):
> qemu: Add an option to remove host sdl/gl checking
>
> .../qemu/qemu-0.15.1/opengl-disable-option.patch | 172 ++++++++++++++++++++
> meta/recipes-devtools/qemu/qemu.inc | 23 +---
> meta/recipes-devtools/qemu/qemu_0.15.1.bb | 1 +
> 3 files changed, 176 insertions(+), 20 deletions(-)
> create mode 100644 meta/recipes-devtools/qemu/qemu-0.15.1/opengl-disable-option.patch
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
--
best rgds,
edwin
^ permalink raw reply [flat|nested] 3+ messages in thread