[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Revert "hw/pci: disable pci-bridge's shpc by de
From: |
Marcel Apfelbaum |
Subject: |
Re: [Qemu-devel] [PATCH] Revert "hw/pci: disable pci-bridge's shpc by default" |
Date: |
Wed, 10 May 2017 13:06:42 -0400 (EDT) |
----- Original Message -----
> From: "Paolo Bonzini" <address@hidden>
> To: "Marcel Apfelbaum" <address@hidden>, address@hidden
> Cc: address@hidden, address@hidden
> Sent: Wednesday, May 10, 2017 6:38:45 PM
> Subject: Re: [Qemu-devel] [PATCH] Revert "hw/pci: disable pci-bridge's shpc
> by default"
>
>
>
> On 10/05/2017 11:29, Marcel Apfelbaum wrote:
> > +++ b/hw/pci-bridge/pci_bridge_dev.c
> > @@ -163,7 +163,7 @@ static Property pci_bridge_dev_properties[] = {
> > DEFINE_PROP_ON_OFF_AUTO(PCI_BRIDGE_DEV_PROP_MSI, PCIBridgeDev, msi,
> > ON_OFF_AUTO_AUTO),
> > DEFINE_PROP_BIT(PCI_BRIDGE_DEV_PROP_SHPC, PCIBridgeDev, flags,
> > - PCI_BRIDGE_DEV_F_SHPC_REQ, false),
> > + PCI_BRIDGE_DEV_F_SHPC_REQ, true),
> > DEFINE_PROP_END_OF_LIST(),
> > };
> >
> > diff --git a/include/hw/compat.h b/include/hw/compat.h
> > index 846b90e..eff021f 100644
> > --- a/include/hw/compat.h
> > +++ b/include/hw/compat.h
> > @@ -18,10 +18,6 @@
> > .property = "old-multiple-chip-handling",\
> > .value = "on",\
> > },{\
> > - .driver = "pci-bridge",\
> > - .property = "shpc",\
> > - .value = "on",\
> > - },{\
>
Hi Paolo,
> Unless we make a conscious decision to break PCI bridge backwards
> compatibility, you now have to disable it for 2.9 and leave this
> HW_COMPAT_2_8 hunk intact. Peter Xu has a patch to add the 2.10 machine
> types.
>
I thought is not an interesting enough scenario to keep it "broken" for 2.9,
but I'll do that, sure, I'll add it on top of Peter's patch.
Thanks,
Marcel
> Paolo
>
> > .driver = TYPE_PCI_DEVICE,\
> > .property = "x-pcie-extcap-init",\
> > .value = "off",\
> > -- 2.9.3
>