sparc/leon3 irqmp: fix IRQ software ack
commit05b9ec96c1ae87fa2a61b2dcd62ee1263255c9e5
authorJean-Christophe Dubois <jcd@tribudubois.net>
Wed, 10 Jan 2018 20:43:27 +0000 (10 21:43 +0100)
committerMark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Wed, 24 Jan 2018 19:19:50 +0000 (24 19:19 +0000)
treed78727e7d24381b5c4482e3e7d8687f33f5db205
parent52483b067cce4a88ffbf8fbeea26de7549d2ad23
sparc/leon3 irqmp: fix IRQ software ack

With the LEON3 IRQ controller IRQs can be acknowledged 2 ways:
* Explicitly by software writing to the CLEAR_OFFSET register
* Implicitly when the procesor is done running the trap handler attached
  to the IRQ.

The actual IRQMP code only allows the implicit processor triggered IRQ ack.
If software write explicitly to the CLEAR_OFFSET register, this will clear
the pending bit in the register value but this will not lower the ongoing
raised IRQ with the processor. The IRQ will be kept raised to the LEON
processor until the related trap handler is run and the processor implicitly
ack the interrupt. So with the actual IRQMP code trap handler have to be run
even if the software has already done its job by clearing the pending bit.

This feature has been tested on another LEON3 simulator (tsim_leon3 from
Gaisler) and it turns out that the Qemu implementation is not equivalent to
the tsim one. In tsim, if software does clear a pending interrupt before
the related interrupt handler is triggered the said interrupt handler will
not be called.

This patch brings the Qemu IRQMP implementation in line with the tsim
implementation by allowing IRQ to be acknowledged by software only.

Signed-off-by: Jean-Christophe Dubois <jcd@tribudubois.net>
Reviewed-by: Fabien Chouteau <chouteau@adacore.com>
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
hw/intc/grlib_irqmp.c