Re: [PATCH] allow VM configuration with both sdl=1 and vnc=1

["Yang, Xiaowei" <xiaowei.yang@intel.com>]

Hi Stefano,
HVM can't boot up with sdl this week. I found it's caused by one section 
of this patch (see below). With it Qemu exits because of unrecognized 
option (-sdl). I'm not familiar with vfb. Can you check it's necessary?

Thanks,
xiaowei


diff -r ec3d0937095d -r 0942baa2a088 tools/python/xen/xend/image.py
--- a/tools/python/xen/xend/image.py    Fri Mar 06 19:10:29 2009 +0000
+++ b/tools/python/xen/xend/image.py    Fri Mar 06 19:13:23 2009 +0000
@@ -335,11 +336,12 @@ class ImageHandler:
              if int(vnc_config.get('vncunused', 1)) != 0:
                  ret.append('-vncunused')

-        elif has_sdl:
-            # SDL is default in QEMU.
+        if has_sdl:
+            ret.append('-sdl')
              if int(vmConfig['platform'].get('opengl', opengl)) != 1 :
                  ret.append('-disable-opengl')
-        else:
+
+        if not has_sdl and not has_vnc :
              ret.append('-nographic')

          if int(vmConfig['platform'].get('monitor', 0)) != 0: