Re: [PATCH] qemu: Reject SDL graphic if it's not supported by qemu

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



ä 2011å01æ11æ 19:29, Daniel P. Berrange åé:
On Tue, Jan 11, 2011 at 03:43:07PM +0800, Osier Yang wrote:
If the emulator doesn't support SDL graphic, we should reject
the use of SDL graphic xml with error messages, but not ignore
it silently.

* src/qemu/qemu_command.c

NACK.

Most previous versions of QEMU don't have any explicit -sdl
flag. You got SDL automagically when *no* -vnc flag was
given. So this change breaks many old QEMU versions.


I didn't known the older versions of QEMU don't have explicit
-sdl flag before, thanks for the knowledge, but the problem is
that qemu-kvm of RHEL removed sdl support.

* Fri Jan 15 2010 Eduardo Habkost <ehabkost@xxxxxxxxxx> -
qemu-kvm-0.12.1.2-2.5.el6
- Remove unneeded/unsupported features: [bz#555336]
  - make default options explicit
  - remove sdl support

So, it means the newer qemu-kvm (of RHEL) doesn't support sdl
anymore. Didn't send the patch to Red Hat internal list,
as the patch won't conflict with upstream.

IMHO it's not reasonable that we have "sdl" xml in domain
config, but no according command line of qemu, and pretend
everything is fine.

Any good idea? Thanks

Regards
Osier

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]