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