On Wed, Mar 20, 2013 at 02:11:51AM +0000, Yin Olivia-R63875 wrote: > Hi Daniel, > > PPC also supports PCI multibus. Could you please help review this patch?
IIUC, PPC did not support PCI multibus in the past, so when did this change ? Libvirt needs to only use this for versions where it is supported, so unconditionally enabling it for all PPC is not right. > > -----Original Message----- > > From: Yin Olivia-R63875 > > Sent: Wednesday, March 13, 2013 12:44 PM > > To: libvir-list@redhat.com > > Cc: Yin Olivia-R63875 > > Subject: [PATCH] qemu: add PCI-multibus support for ppc > > > > Signed-off-by: Olivia Yin <hong-hua....@freescale.com> > > --- > > src/qemu/qemu_capabilities.c | 10 ++++++++++ > > 1 files changed, 10 insertions(+), 0 deletions(-) > > > > diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c > > index 7bc1ebc..7d7791d 100644 > > --- a/src/qemu/qemu_capabilities.c > > +++ b/src/qemu/qemu_capabilities.c > > @@ -2209,6 +2209,11 @@ virQEMUCapsInitHelp(virQEMUCapsPtr qemuCaps, uid_t > > runUid, gid_t runGid) > > virQEMUCapsClear(qemuCaps, QEMU_CAPS_NO_ACPI); > > } > > > > + /* ppc support PCI-multibus */ > > + if (qemuCaps->arch == VIR_ARCH_PPC) { > > + virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS); > > + } > > + > > /* virQEMUCapsExtractDeviceStr will only set additional caps if qemu > > * understands the 0.13.0+ notion of "-device driver,". */ > > if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE) && @@ -2450,6 +2455,11 > > @@ virQEMUCapsInitQMP(virQEMUCapsPtr qemuCaps, > > virQEMUCapsSet(qemuCaps, QEMU_CAPS_NO_ACPI); > > } > > > > + /* ppc support PCI-multibus */ > > + if (qemuCaps->arch == VIR_ARCH_PPC) { > > + virQEMUCapsSet(qemuCaps, QEMU_CAPS_PCI_MULTIBUS); > > + } > > + > > if (virQEMUCapsProbeQMPCommands(qemuCaps, mon) < 0) > > goto cleanup; > > if (virQEMUCapsProbeQMPEvents(qemuCaps, mon) < 0) > > -- 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@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list