[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] xen: piix reuse pci generic class init function
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH] xen: piix reuse pci generic class init function |
Date: |
Thu, 7 Apr 2016 09:58:59 -0700 (PDT) |
User-agent: |
Alpine 2.10 (DEB 1266 2009-07-14) |
On Thu, 7 Apr 2016, John Snow wrote:
> > On Wed, Apr 06, 2016 at 04:56:12PM -0700, Stefano Stabellini wrote:
> >> On Sun, 3 Apr 2016, Michael S. Tsirkin wrote:
> >>> piix3_ide_xen_class_init is identical to piix3_ide_class_init
> >>> except it's buggy as it does not set exit and does not disable
> >>> hotplug properly.
> >>>
> >>> Switch to the generic one.
> >>>
> >>> Reviewed-by: Stefano Stabellini <address@hidden>
> >>> Signed-off-by: Michael S. Tsirkin <address@hidden>
> >>
> >> Hey John,
> >>
> >> are you going to take the patch or do you want me to handle it?
> >>
> >> Cheers,
> >>
> >> Stefano
> >
> > it's in my tree already.
> >
>
> Thank you. Apologies for being asleep at the wheel.
> I've been somewhat distracted working on something else recently.
>
> I'll trust your judgment when it comes to XEN.
Thanks both of you.
Cheers,
Stefano
> >>
> >>> hw/ide/piix.c | 14 +-------------
> >>> 1 file changed, 1 insertion(+), 13 deletions(-)
> >>>
> >>> diff --git a/hw/ide/piix.c b/hw/ide/piix.c
> >>> index df46147..0a4cbcb 100644
> >>> --- a/hw/ide/piix.c
> >>> +++ b/hw/ide/piix.c
> >>> @@ -258,22 +258,10 @@ static const TypeInfo piix3_ide_info = {
> >>> .class_init = piix3_ide_class_init,
> >>> };
> >>>
> >>> -static void piix3_ide_xen_class_init(ObjectClass *klass, void *data)
> >>> -{
> >>> - DeviceClass *dc = DEVICE_CLASS(klass);
> >>> - PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
> >>> -
> >>> - k->realize = pci_piix_ide_realize;
> >>> - k->vendor_id = PCI_VENDOR_ID_INTEL;
> >>> - k->device_id = PCI_DEVICE_ID_INTEL_82371SB_1;
> >>> - k->class_id = PCI_CLASS_STORAGE_IDE;
> >>> - set_bit(DEVICE_CATEGORY_STORAGE, dc->categories);
> >>> -}
> >>> -
> >>> static const TypeInfo piix3_ide_xen_info = {
> >>> .name = "piix3-ide-xen",
> >>> .parent = TYPE_PCI_IDE,
> >>> - .class_init = piix3_ide_xen_class_init,
> >>> + .class_init = piix3_ide_class_init,
> >>> };
> >>>
> >>> static void piix4_ide_class_init(ObjectClass *klass, void *data)
> >>> --
> >>> MST
> >>>
>