[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [for-2.11 PATCH 08/26] spapr_drc: add unrealize method to
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [for-2.11 PATCH 08/26] spapr_drc: add unrealize method to physical DRC class |
Date: |
Wed, 26 Jul 2017 14:06:29 +1000 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Tue, Jul 25, 2017 at 07:59:44PM +0200, Greg Kurz wrote:
> When hot-unplugging a PHB, all its PCI DRC connectors get unrealized. This
> patch adds an unrealize method to the physical DRC class, in order to undo
> registrations performed in realize_physical().
>
> Signed-off-by: Greg Kurz <address@hidden>
Applied to ppc-for-2.11. In a sense this is fix that could be applied
to ppc-for-2.10, but unlike the last patch, AFAICT there's no way this
bug can be triggered in the current code.
> ---
> hw/ppc/spapr_drc.c | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/hw/ppc/spapr_drc.c b/hw/ppc/spapr_drc.c
> index d72453bcb42f..105ce85b6d04 100644
> --- a/hw/ppc/spapr_drc.c
> +++ b/hw/ppc/spapr_drc.c
> @@ -626,12 +626,28 @@ static void realize_physical(DeviceState *d, Error
> **errp)
> qemu_register_reset(drc_physical_reset, drcp);
> }
>
> +static void unrealize_physical(DeviceState *d, Error **errp)
> +{
> + sPAPRDRCPhysical *drcp = SPAPR_DRC_PHYSICAL(d);
> + Error *local_err = NULL;
> +
> + unrealize(d, &local_err);
> + if (local_err) {
> + error_propagate(errp, local_err);
> + return;
> + }
> +
> + vmstate_unregister(DEVICE(drcp), &vmstate_spapr_drc_physical, drcp);
> + qemu_unregister_reset(drc_physical_reset, drcp);
> +}
> +
> static void spapr_drc_physical_class_init(ObjectClass *k, void *data)
> {
> DeviceClass *dk = DEVICE_CLASS(k);
> sPAPRDRConnectorClass *drck = SPAPR_DR_CONNECTOR_CLASS(k);
>
> dk->realize = realize_physical;
> + dk->unrealize = unrealize_physical;
> drck->dr_entity_sense = physical_entity_sense;
> drck->isolate = drc_isolate_physical;
> drck->unisolate = drc_unisolate_physical;
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- Re: [Qemu-ppc] [for-2.11 PATCH 05/26] spapr_iommu: convert TCE table object to realize(), (continued)
- [Qemu-ppc] [for-2.11 PATCH 08/26] spapr_drc: add unrealize method to physical DRC class, Greg Kurz, 2017/07/25
- Re: [Qemu-ppc] [for-2.11 PATCH 08/26] spapr_drc: add unrealize method to physical DRC class,
David Gibson <=
- [Qemu-ppc] [for-2.11 PATCH 09/26] spapr_drc: pass object ownership to parent/owner, Greg Kurz, 2017/07/25
- [Qemu-ppc] [for-2.11 PATCH 10/26] spapr_iommu: pass object ownership to parent/owner, Greg Kurz, 2017/07/25
- [Qemu-ppc] [for-2.11 PATCH 11/26] spapr_iommu: unregister vmstate at unrealize time, Greg Kurz, 2017/07/25
- [Qemu-ppc] [for-2.11 PATCH 12/26] pci: allow cleanup/unregistration of PCI buses, Greg Kurz, 2017/07/25
- [Qemu-ppc] [for-2.11 PATCH 13/26] qdev: store DeviceState's canonical path to use when unparenting, Greg Kurz, 2017/07/25