[libvirt] [PATCH] qemu: add PCI-multibus support for ppc

[Daniel P. Berrange]

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 at redhat.com
> > Cc: Yin Olivia-R63875
> > Subject: [PATCH] qemu: add PCI-multibus support for ppc
> > 
> > Signed-off-by: Olivia Yin <hong-hua.yin at 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 :|