[libvirt] [PATCH] Revert "Introduce QEMU_CAPS_ARM_VIRT_PCI"
Pavel Fedin
p.fedin at samsung.com
Tue Jun 30 11:06:27 UTC 2015
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 at redhat.com]
> Sent: Tuesday, June 30, 2015 1:02 PM
> To: Peter Krempa; libvir-list at redhat.com
> Cc: Pavel Fedin; Cole Robinson
> Subject: Re: [libvirt] [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
More information about the libvir-list
mailing list