diff options
author | Greg Kurz <groug@kaod.org> | 2019-02-12 19:24:06 +0100 |
---|---|---|
committer | David Gibson <david@gibson.dropbear.id.au> | 2019-02-17 21:54:02 +1100 |
commit | 0afed8c8195886111dd8ab0d078b189c55949521 (patch) | |
tree | 5dcd48a555d73d8410c69dac175a1934ae4c86ad /hw/intc | |
parent | e150ac89fbcf4ea15535f6718d0194310c39280d (diff) | |
download | qemu-0afed8c8195886111dd8ab0d078b189c55949521.zip qemu-0afed8c8195886111dd8ab0d078b189c55949521.tar.gz qemu-0afed8c8195886111dd8ab0d078b189c55949521.tar.bz2 |
xive: Only set source type for LSIs
MSI is the default and LSI specific code is guarded by the
xive_source_irq_is_lsi() helper. The xive_source_irq_set()
helper is a nop for MSIs.
Simplify the code by turning xive_source_irq_set() into
xive_source_irq_set_lsi() and only call it for LSIs. The
call to xive_source_irq_set(false) in spapr_xive_irq_free()
is also a nop. Just drop it.
Signed-off-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Message-Id: <154999584656.690774.18352404495120358613.stgit@bahia.lan>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'hw/intc')
-rw-r--r-- | hw/intc/spapr_xive.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/hw/intc/spapr_xive.c b/hw/intc/spapr_xive.c index a0f5ff9..290a290 100644 --- a/hw/intc/spapr_xive.c +++ b/hw/intc/spapr_xive.c @@ -489,20 +489,19 @@ bool spapr_xive_irq_claim(sPAPRXive *xive, uint32_t lisn, bool lsi) } xive->eat[lisn].w |= cpu_to_be64(EAS_VALID); - xive_source_irq_set(xsrc, lisn, lsi); + if (lsi) { + xive_source_irq_set_lsi(xsrc, lisn); + } return true; } bool spapr_xive_irq_free(sPAPRXive *xive, uint32_t lisn) { - XiveSource *xsrc = &xive->source; - if (lisn >= xive->nr_irqs) { return false; } xive->eat[lisn].w &= cpu_to_be64(~EAS_VALID); - xive_source_irq_set(xsrc, lisn, false); return true; } |