Browse Source

hw/intc/arm_gicv3: Implement GICv4's new redistributor frame

The GICv4 extends the redistributor register map -- where GICv3
had two 64KB frames per CPU, GICv4 has four frames. Add support
for the extra frame by using a new gicv3_redist_size() function
in the places in the GIC implementation which currently use
a fixed constant size for the redistributor register block.
(Until we implement the extra registers they will RAZ/WI.)

Any board that wants to use a GICv4 will need to also adjust
to handle the different sized redistributor register block;
that will be done separately.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20220408141550.1271295-23-peter.maydell@linaro.org
Peter Maydell 3 years ago
parent
commit
ae3b3ba15c

+ 1 - 1
hw/intc/arm_gicv3_common.c

@@ -295,7 +295,7 @@ void gicv3_init_irqs_and_mmio(GICv3State *s, qemu_irq_handler handler,
 
 
         memory_region_init_io(&region->iomem, OBJECT(s),
         memory_region_init_io(&region->iomem, OBJECT(s),
                               ops ? &ops[1] : NULL, region, name,
                               ops ? &ops[1] : NULL, region, name,
-                              s->redist_region_count[i] * GICV3_REDIST_SIZE);
+                              s->redist_region_count[i] * gicv3_redist_size(s));
         sysbus_init_mmio(sbd, &region->iomem);
         sysbus_init_mmio(sbd, &region->iomem);
         g_free(name);
         g_free(name);
     }
     }

+ 4 - 4
hw/intc/arm_gicv3_redist.c

@@ -442,8 +442,8 @@ MemTxResult gicv3_redist_read(void *opaque, hwaddr offset, uint64_t *data,
      * in the memory map); if so then the GIC has multiple MemoryRegions
      * in the memory map); if so then the GIC has multiple MemoryRegions
      * for the redistributors.
      * for the redistributors.
      */
      */
-    cpuidx = region->cpuidx + offset / GICV3_REDIST_SIZE;
-    offset %= GICV3_REDIST_SIZE;
+    cpuidx = region->cpuidx + offset / gicv3_redist_size(s);
+    offset %= gicv3_redist_size(s);
 
 
     cs = &s->cpu[cpuidx];
     cs = &s->cpu[cpuidx];
 
 
@@ -501,8 +501,8 @@ MemTxResult gicv3_redist_write(void *opaque, hwaddr offset, uint64_t data,
      * in the memory map); if so then the GIC has multiple MemoryRegions
      * in the memory map); if so then the GIC has multiple MemoryRegions
      * for the redistributors.
      * for the redistributors.
      */
      */
-    cpuidx = region->cpuidx + offset / GICV3_REDIST_SIZE;
-    offset %= GICV3_REDIST_SIZE;
+    cpuidx = region->cpuidx + offset / gicv3_redist_size(s);
+    offset %= gicv3_redist_size(s);
 
 
     cs = &s->cpu[cpuidx];
     cs = &s->cpu[cpuidx];
 
 

+ 21 - 0
hw/intc/gicv3_internal.h

@@ -489,6 +489,27 @@ FIELD(VTE, RDBASE, 42, RDBASE_PROCNUM_LENGTH)
 
 
 /* Functions internal to the emulated GICv3 */
 /* Functions internal to the emulated GICv3 */
 
 
+/**
+ * gicv3_redist_size:
+ * @s: GICv3State
+ *
+ * Return the size of the redistributor register frame in bytes
+ * (which depends on what GIC version this is)
+ */
+static inline int gicv3_redist_size(GICv3State *s)
+{
+    /*
+     * Redistributor size is controlled by the redistributor GICR_TYPER.VLPIS.
+     * It's the same for every redistributor in the GIC, so arbitrarily
+     * use the register field in the first one.
+     */
+    if (s->cpu[0].gicr_typer & GICR_TYPER_VLPIS) {
+        return GICV4_REDIST_SIZE;
+    } else {
+        return GICV3_REDIST_SIZE;
+    }
+}
+
 /**
 /**
  * gicv3_intid_is_special:
  * gicv3_intid_is_special:
  * @intid: interrupt ID
  * @intid: interrupt ID

+ 5 - 0
include/hw/intc/arm_gicv3_common.h

@@ -38,7 +38,12 @@
 
 
 #define GICV3_LPI_INTID_START 8192
 #define GICV3_LPI_INTID_START 8192
 
 
+/*
+ * The redistributor in GICv3 has two 64KB frames per CPU; in
+ * GICv4 it has four 64KB frames per CPU.
+ */
 #define GICV3_REDIST_SIZE 0x20000
 #define GICV3_REDIST_SIZE 0x20000
+#define GICV4_REDIST_SIZE 0x40000
 
 
 /* Number of SGI target-list bits */
 /* Number of SGI target-list bits */
 #define GICV3_TARGETLIST_BITS 16
 #define GICV3_TARGETLIST_BITS 16