[libvirt] [RFC v3 PATCH 5/5] PowerPC : Add address-type "spapr-vio"

[Michael Ellerman]

On Tue, 2011-11-29 at 12:16 -0500, Stefan Berger wrote:
> On 11/29/2011 10:03 AM, Prerna Saxena wrote:
> > @@ -3048,6 +3087,7 @@ virDomainControllerDefParseXML(xmlNodePtr node,
> >       }
> >
> >       if (def->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_NONE &&
> > +         def->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_SPAPRVIO &&
> >           def->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI) {
> >           virDomainReportError(VIR_ERR_INTERNAL_ERROR, "%s",
> >                                _("Controllers must use the 'pci' address type"));
> Is this still the proper error message for this type of bus? You may 
> want to look at def->os.arch / def->os.machine to see what is expected 
> here in terms of bus and then have a more specific error message.

No it's not the right message, which is why we're adding that check to
skip the message if the controller has an spapr-vio address. So I don't
think we need to anything else with arch/machine.

cheers

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20111201/1aacc162/attachment-0001.sig>