Re: [PATCH v2 11/17] qemu: really remove last traces of Xenner support

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

 



On Mon, Nov 09, 2015 at 06:24:37PM -0500, John Ferlan wrote:
> 
> 
> On 11/09/2015 11:24 AM, Daniel P. Berrange wrote:
> > We have twice previously attempted to remove Xenner
> > support
> > 
> >   commit de9be0ab4d7abe4c3ae5beaaff7ddb208921f1f1
> >   Author: Daniel P. Berrange <berrange@xxxxxxxxxx>
> >   Date:   Wed Aug 22 17:29:01 2012 +0100
> > 
> >     Remove xenner support
> > 
> >   commit 92572c3d71bd08e6dd1897c7b14be9d27df608e9
> >   Author: Ján Tomko <jtomko@xxxxxxxxxx>
> >   Date:   Wed Feb 18 16:33:50 2015 +0100
> > 
> >     Remove code handling the QEMU_CAPS_DOMID capability
> > 
> > This change really does remove the last traces of it
> > in the capabilities handling code
> > 
> > Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx>
> > ---
> >  src/qemu/qemu_capabilities.c |  4 ----
> >  src/qemu/qemu_capabilities.h |  4 ++--
> >  src/qemu/qemu_command.c      | 14 ++++----------
> >  tests/qemuhelptest.c         |  4 ----
> >  4 files changed, 6 insertions(+), 20 deletions(-)
> > 
> 
> How about qemuParseCommandLine
> 
>     if (strstr(path, "xenner")) {
>         def->virtType = VIR_DOMAIN_VIRT_KVM;
>         def->os.type = VIR_DOMAIN_OSTYPE_XEN;
>     } else {
> ...
> 
> Plus a few other things that show up in cscope with "xenner" string:
> 
> drvqemu.html.in
> domaincommon.rng
> qemu_command.c
> qemuargv2xmltest.c
> testutilsqemu.c

Lol, it really won't go away. 4th time lucky....

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
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]