[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 04/33] xics: Eliminate reset hook
From: |
Greg Kurz |
Subject: |
Re: [PATCH v2 04/33] xics: Eliminate reset hook |
Date: |
Fri, 27 Sep 2019 09:19:56 +0200 |
On Fri, 27 Sep 2019 15:49:59 +1000
David Gibson <address@hidden> wrote:
> Currently TYPE_XICS_BASE and TYPE_XICS_SIMPLE have their own reset methods,
> using the standard technique for having the subtype call the supertype's
> methods before doing its own thing.
>
> But TYPE_XICS_SIMPLE is the only subtype of TYPE_XICS_BASE ever
> instantiated, so there's no point having the split here. Merge them
> together into just an ics_reset() function.
>
> Signed-off-by: David Gibson <address@hidden>
> Reviewed-by: Cédric Le Goater <address@hidden>
> ---
Reviewed-by: Greg Kurz <address@hidden>
> hw/intc/xics.c | 57 ++++++++++++++++++-------------------------
> include/hw/ppc/xics.h | 1 -
> 2 files changed, 24 insertions(+), 34 deletions(-)
>
> diff --git a/hw/intc/xics.c b/hw/intc/xics.c
> index 310dc72b46..82e6f09259 100644
> --- a/hw/intc/xics.c
> +++ b/hw/intc/xics.c
> @@ -547,11 +547,28 @@ static void ics_eoi(ICSState *ics, uint32_t nr)
> }
> }
>
> -static void ics_simple_reset(DeviceState *dev)
> +static void ics_reset_irq(ICSIRQState *irq)
> {
> - ICSStateClass *icsc = ICS_BASE_GET_CLASS(dev);
> + irq->priority = 0xff;
> + irq->saved_priority = 0xff;
> +}
>
> - icsc->parent_reset(dev);
> +static void ics_reset(DeviceState *dev)
> +{
> + ICSState *ics = ICS_BASE(dev);
> + int i;
> + uint8_t flags[ics->nr_irqs];
> +
> + for (i = 0; i < ics->nr_irqs; i++) {
> + flags[i] = ics->irqs[i].flags;
> + }
> +
> + memset(ics->irqs, 0, sizeof(ICSIRQState) * ics->nr_irqs);
> +
> + for (i = 0; i < ics->nr_irqs; i++) {
> + ics_reset_irq(ics->irqs + i);
> + ics->irqs[i].flags = flags[i];
> + }
>
> if (kvm_irqchip_in_kernel()) {
> Error *local_err = NULL;
> @@ -563,9 +580,9 @@ static void ics_simple_reset(DeviceState *dev)
> }
> }
>
> -static void ics_simple_reset_handler(void *dev)
> +static void ics_reset_handler(void *dev)
> {
> - ics_simple_reset(dev);
> + ics_reset(dev);
> }
>
> static void ics_simple_realize(DeviceState *dev, Error **errp)
> @@ -580,7 +597,7 @@ static void ics_simple_realize(DeviceState *dev, Error
> **errp)
> return;
> }
>
> - qemu_register_reset(ics_simple_reset_handler, ics);
> + qemu_register_reset(ics_reset_handler, ics);
> }
>
> static void ics_simple_class_init(ObjectClass *klass, void *data)
> @@ -590,8 +607,6 @@ static void ics_simple_class_init(ObjectClass *klass,
> void *data)
>
> device_class_set_parent_realize(dc, ics_simple_realize,
> &isc->parent_realize);
> - device_class_set_parent_reset(dc, ics_simple_reset,
> - &isc->parent_reset);
> }
>
> static const TypeInfo ics_simple_info = {
> @@ -602,30 +617,6 @@ static const TypeInfo ics_simple_info = {
> .class_size = sizeof(ICSStateClass),
> };
>
> -static void ics_reset_irq(ICSIRQState *irq)
> -{
> - irq->priority = 0xff;
> - irq->saved_priority = 0xff;
> -}
> -
> -static void ics_base_reset(DeviceState *dev)
> -{
> - ICSState *ics = ICS_BASE(dev);
> - int i;
> - uint8_t flags[ics->nr_irqs];
> -
> - for (i = 0; i < ics->nr_irqs; i++) {
> - flags[i] = ics->irqs[i].flags;
> - }
> -
> - memset(ics->irqs, 0, sizeof(ICSIRQState) * ics->nr_irqs);
> -
> - for (i = 0; i < ics->nr_irqs; i++) {
> - ics_reset_irq(ics->irqs + i);
> - ics->irqs[i].flags = flags[i];
> - }
> -}
> -
> static void ics_base_realize(DeviceState *dev, Error **errp)
> {
> ICSState *ics = ICS_BASE(dev);
> @@ -726,7 +717,7 @@ static void ics_base_class_init(ObjectClass *klass, void
> *data)
>
> dc->realize = ics_base_realize;
> dc->props = ics_base_properties;
> - dc->reset = ics_base_reset;
> + dc->reset = ics_reset;
> dc->vmsd = &vmstate_ics_base;
> }
>
> diff --git a/include/hw/ppc/xics.h b/include/hw/ppc/xics.h
> index 8874bad328..7efd49c02c 100644
> --- a/include/hw/ppc/xics.h
> +++ b/include/hw/ppc/xics.h
> @@ -105,7 +105,6 @@ struct ICSStateClass {
> DeviceClass parent_class;
>
> DeviceRealize parent_realize;
> - DeviceReset parent_reset;
> };
>
> struct ICSState {
- [PATCH v2 00/33] spapr: IRQ subsystem cleanup, David Gibson, 2019/09/27
- [PATCH v2 02/33] xics: Eliminate 'reject', 'resend' and 'eoi' class hooks, David Gibson, 2019/09/27
- [PATCH v2 03/33] xics: Rename misleading ics_simple_*() functions, David Gibson, 2019/09/27
- [PATCH v2 01/33] xics: Minor fixes for XICSFabric interface, David Gibson, 2019/09/27
- [PATCH v2 07/33] spapr: Fold spapr_phb_lsi_qirq() into its single caller, David Gibson, 2019/09/27
- [PATCH v2 06/33] xics: Create sPAPR specific ICS subtype, David Gibson, 2019/09/27
- [PATCH v2 08/33] spapr: Replace spapr_vio_qirq() helper with spapr_vio_irq_pulse() helper, David Gibson, 2019/09/27
- [PATCH v2 04/33] xics: Eliminate reset hook, David Gibson, 2019/09/27
- Re: [PATCH v2 04/33] xics: Eliminate reset hook,
Greg Kurz <=
- [PATCH v2 05/33] xics: Merge TYPE_ICS_BASE and TYPE_ICS_SIMPLE classes, David Gibson, 2019/09/27
- [PATCH v2 09/33] spapr: Clarify and fix handling of nr_irqs, David Gibson, 2019/09/27
- [PATCH v2 15/33] spapr: Handle freeing of multiple irqs in frontend only, David Gibson, 2019/09/27
- [PATCH v2 11/33] spapr: Fix indexing of XICS irqs, David Gibson, 2019/09/27
- [PATCH v2 10/33] spapr: Eliminate nr_irqs parameter to SpaprIrq::init, David Gibson, 2019/09/27
- [PATCH v2 12/33] spapr: Simplify spapr_qirq() handling, David Gibson, 2019/09/27
- [PATCH v2 16/33] spapr, xics, xive: Better use of assert()s on irq claim/free paths, David Gibson, 2019/09/27