Re: [PATCH] Revert "Introduce QEMU_CAPS_ARM_VIRT_PCI"

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

 



 Hello!

> I'm inclined to ACK this, but I'd love to hear committer or original
> author's opinions.

 1. Does it really harm? We are going to use it soon. Unfortunately looks like we have opinion
collision with Cole who says that virtio-pci support is bad only because current distros don't
support it. Because for some weird reason, i guess, PCI support is disabled in the kernel.
 2. Unfortunately, the patch is wrong. I was setting the capability in wrong place, it should be in
virQEMUCapsInitQMPMonitor() instead of virQEMUCapsComputeCmdFlags(). My fault, and how i managed to
test this is another story. :(

Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia


> -----Original Message-----
> From: Michal Privoznik [mailto:mprivozn@xxxxxxxxxx]
> Sent: Tuesday, June 30, 2015 1:02 PM
> To: Peter Krempa; libvir-list@xxxxxxxxxx
> Cc: Pavel Fedin; Cole Robinson
> Subject: Re:  [PATCH] Revert "Introduce QEMU_CAPS_ARM_VIRT_PCI"
> 
> On 29.06.2015 09:58, Peter Krempa wrote:
> > The capability was not used up to the feature freeze.
> >
> > This reverts commit 7f3515b4bb677d0ead1887547efc844e4761268a.
> > ---
> >  src/qemu/qemu_capabilities.c | 4 ----
> >  src/qemu/qemu_capabilities.h | 1 -
> >  2 files changed, 5 deletions(-)
> >
> > diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
> > index e7002a3..27686c3 100644
> > --- a/src/qemu/qemu_capabilities.c
> > +++ b/src/qemu/qemu_capabilities.c
> > @@ -287,7 +287,6 @@ VIR_ENUM_IMPL(virQEMUCaps, QEMU_CAPS_LAST,
> >                "aarch64-off",
> >
> >                "vhost-user-multiqueue", /* 190 */
> > -              "arm-virt-pci",
> >      );
> >
> >
> > @@ -1337,9 +1336,6 @@ virQEMUCapsComputeCmdFlags(const char *help,
> >          virQEMUCapsSet(qemuCaps, QEMU_CAPS_VNC_SHARE_POLICY);
> >      }
> >
> > -    if (version >= 2003000)
> > -        virQEMUCapsSet(qemuCaps, QEMU_CAPS_ARM_VIRT_PCI);
> > -
> >      return 0;
> >  }
> >
> > diff --git a/src/qemu/qemu_capabilities.h b/src/qemu/qemu_capabilities.h
> > index f4180a8..30aa504 100644
> > --- a/src/qemu/qemu_capabilities.h
> > +++ b/src/qemu/qemu_capabilities.h
> > @@ -230,7 +230,6 @@ typedef enum {
> >      QEMU_CAPS_DEVICE_PCI_SERIAL  = 188, /* -device pci-serial */
> >      QEMU_CAPS_CPU_AARCH64_OFF    = 189, /* -cpu ...,aarch64=off */
> >      QEMU_CAPS_VHOSTUSER_MULTIQUEUE = 190, /* vhost-user with -netdev queues= */
> > -    QEMU_CAPS_ARM_VIRT_PCI       = 191, /* ARM 'virt' machine has PCI bus */
> >
> >      QEMU_CAPS_LAST,                   /* this must always be the last item */
> >  } virQEMUCapsFlags;
> >
> 
> I'm inclined to ACK this, but I'd love to hear committer or original
> author's opinions.
> 
> Michal

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