[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 32/36] spapr/rtas: modify spapr_rtas_register
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH v5 32/36] spapr/rtas: modify spapr_rtas_register() to remove RTAS handlers |
Date: |
Fri, 30 Nov 2018 09:44:16 +1100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Thu, Nov 29, 2018 at 05:40:18PM +0100, Cédric Le Goater wrote:
> On 11/29/18 5:12 AM, David Gibson wrote:
> > On Fri, Nov 16, 2018 at 11:57:25AM +0100, Cédric Le Goater wrote:
> >> Removing RTAS handlers will become necessary when the new pseries
> >> machine supporting multiple interrupt mode is introduced.
> >
> > I'd prefer this to be done as a separate spapr_rtas_unregister()
> > helper, just to improve greppability.
>
> ok. I should propose an inline :
>
> static inline void spapr_rtas_unregister(int token)
> {
> spapr_rtas_register(token, NULL, NULL);
> }
Fair enough.
>
> >
> >>
> >> Signed-off-by: Cédric Le Goater <address@hidden>
> >> ---
> >> hw/ppc/spapr_rtas.c | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/hw/ppc/spapr_rtas.c b/hw/ppc/spapr_rtas.c
> >> index d6a0952154ac..e005d5d08151 100644
> >> --- a/hw/ppc/spapr_rtas.c
> >> +++ b/hw/ppc/spapr_rtas.c
> >> @@ -404,7 +404,7 @@ void spapr_rtas_register(int token, const char *name,
> >> spapr_rtas_fn fn)
> >>
> >> token -= RTAS_TOKEN_BASE;
> >>
> >> - assert(!rtas_table[token].name);
> >> + assert(!name || !rtas_table[token].name);
> >>
> >> rtas_table[token].name = name;
> >> rtas_table[token].fn = fn;
> >
>
--
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