arm_gic: gic_update should always update all cores
authorJohan Karlsson <Johan.Karlsson@enea.com>
Mon, 15 Jun 2015 17:06:07 +0000 (18:06 +0100)
committerPeter Maydell <peter.maydell@linaro.org>
Mon, 15 Jun 2015 17:06:07 +0000 (18:06 +0100)
This patch fixes so that gic_update always updates all the cores with
new pending irq states.  If the function returns early it is possible
to get interrupts that has already been acknowledged.

Signed-off-by: Johan Karlsson <johan.karlsson@enea.com>
[PMM: rebased to apply to current master]
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
hw/intc/arm_gic.c

index c1d2e704ec511307c30484eaaad5497c11b4185f..454bfd7df5449403ef3eb6091bf272208c181051 100644 (file)
@@ -71,7 +71,7 @@ void gic_update(GICState *s)
             || !(s->cpu_ctlr[cpu] & (GICC_CTLR_EN_GRP0 | GICC_CTLR_EN_GRP1))) {
             qemu_irq_lower(s->parent_irq[cpu]);
             qemu_irq_lower(s->parent_fiq[cpu]);
-            return;
+            continue;
         }
         best_prio = 0x100;
         best_irq = 1023;