[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/6] s390x/pci: Use hotplug_dev instead of lo
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/6] s390x/pci: Use hotplug_dev instead of looking up the host bridge |
Date: |
Wed, 16 Jan 2019 10:22:39 +0100 |
On Tue, 15 Jan 2019 15:57:38 -0500
Collin Walling <address@hidden> wrote:
> On 1/14/19 5:31 AM, David Hildenbrand wrote:
> > We directly have it in our hands.
> >
> > Signed-off-by: David Hildenbrand <address@hidden>
> > ---
> > hw/s390x/s390-pci-bus.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/hw/s390x/s390-pci-bus.c b/hw/s390x/s390-pci-bus.c
> > index 7f911b216a..86dda831f9 100644
> > --- a/hw/s390x/s390-pci-bus.c
> > +++ b/hw/s390x/s390-pci-bus.c
> > @@ -826,9 +826,9 @@ static bool s390_pci_alloc_idx(S390pciState *s,
> > S390PCIBusDevice *pbdev)
> > static void s390_pcihost_plug(HotplugHandler *hotplug_dev, DeviceState
> > *dev,
> > Error **errp)
> > {
> > + S390pciState *s = S390_PCI_HOST_BRIDGE(hotplug_dev);
> > PCIDevice *pdev = NULL;
> > S390PCIBusDevice *pbdev = NULL;
> > - S390pciState *s = s390_get_phb();
> >
> > if (object_dynamic_cast(OBJECT(dev), TYPE_PCI_BRIDGE)) {
> > BusState *bus;
> > @@ -935,11 +935,11 @@ static void s390_pcihost_timer_cb(void *opaque)
> > static void s390_pcihost_unplug(HotplugHandler *hotplug_dev, DeviceState
> > *dev,
> > Error **errp)
> > {
> > + S390pciState *s = S390_PCI_HOST_BRIDGE(hotplug_dev);
> > PCIDevice *pci_dev = NULL;
> > PCIBus *bus;
> > int32_t devfn;
> > S390PCIBusDevice *pbdev = NULL;
> > - S390pciState *s = s390_get_phb();
> >
> > if (object_dynamic_cast(OBJECT(dev), TYPE_PCI_BRIDGE)) {
> > error_setg(errp, "PCI bridge hot unplug currently not supported");
> >
>
> Looks like the macro will do the same thing as the function does? I
> wonder if it makes sense to one day replace all function calls with the
> macro.
The idea behind the function was to do some caching. Opinions on that
differ :)
>
> Reviewed-by: Collin Walling <address@hidden>
>
If you like this patch, I don't really object to merging it :)
- [Qemu-devel] [PATCH v2 0/6] s390x/pci: hotplug handler fixes and reworks, David Hildenbrand, 2019/01/14
- [Qemu-devel] [PATCH v2 1/6] s390x/pci: Use hotplug_dev instead of looking up the host bridge, David Hildenbrand, 2019/01/14
- [Qemu-devel] [PATCH v2 2/6] s390x/pci: Move some hotplug checks to the pre_plug handler, David Hildenbrand, 2019/01/14
- [Qemu-devel] [PATCH v2 6/6] s390x/pci: Unplug remaining devices on pcihost reset, David Hildenbrand, 2019/01/14
- [Qemu-devel] [PATCH v2 4/6] s390x/pci: Ignore the unplug call if we already have a release_timer, David Hildenbrand, 2019/01/14
- [Qemu-devel] [PATCH v2 3/6] s390x/pci: Always delete and free the release_timer, David Hildenbrand, 2019/01/14
- [Qemu-devel] [PATCH v2 5/6] s390x/pci: Introduce unplug requests and split unplug handler, David Hildenbrand, 2019/01/14
- Re: [Qemu-devel] [PATCH v2 0/6] s390x/pci: hotplug handler fixes and reworks, Cornelia Huck, 2019/01/16