On Tue, Nov 16, 2010 at 05:09:35PM +0900, Isaku Yamahata wrote: > On Mon, Nov 15, 2010 at 04:30:55PM +0200, Gleb Natapov wrote: > > diff --git a/hw/piix_pci.c b/hw/piix_pci.c > > index b5589b9..38f9d9e 100644 > > --- a/hw/piix_pci.c > > +++ b/hw/piix_pci.c > > @@ -365,6 +365,7 @@ static PCIDeviceInfo i440fx_info[] = { > > static SysBusDeviceInfo i440fx_pcihost_info = { > > .init = i440fx_pcihost_initfn, > > .qdev.name = "i440FX-pcihost", > > + .qdev.fw_name = "pci", > > .qdev.size = sizeof(I440FXState), > > .qdev.no_user = 1, > > }; > > There are other pci host bridges for not pc architecture > which needs similar modification. Yes, I know. This patch doesn't try to add fw_name everywhere it is needed, just for x86 for now. Things can be added incrementally. > Please grep by pci_register_bus(). > - apb_pci.c > - bonito.c > - grackle_pci.c > - unin_pci.c > - versatile_pci.c I'll look into using BusInfo->name if fw_name is not available. Then all pci buses will work. > > BTW, what happens for non-qdevfied pci host bridge? They should be converted someday. -- Gleb. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html