From 51022bfe0eeaef5f08d6ace98ebfb1917d1b3331 Mon Sep 17 00:00:00 2001 From: Giacomo Travaglini Date: Tue, 20 Aug 2019 13:53:39 +0100 Subject: [PATCH] dev-arm: Add GICv3 unimplemented Hyp Active Priorities Group regs ICH_APxR1, ICH_APxR2, ICH_APxR3 are implemented only if supporting more than 6 bits of priority. Since this is not the case, they are currently unimplemented. According to spec, unimplemented registers are RAZ/WI. Change-Id: Ifd7f7a3d42b4575c2f7aff3b95d5a47ac1e61842 Signed-off-by: Giacomo Travaglini Reviewed-by: Andreas Sandberg Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/20619 Maintainer: Andreas Sandberg Tested-by: kokoro --- src/dev/arm/gic_v3_cpu_interface.cc | 58 +++++++++++++++++++++++++++-- 1 file changed, 54 insertions(+), 4 deletions(-) diff --git a/src/dev/arm/gic_v3_cpu_interface.cc b/src/dev/arm/gic_v3_cpu_interface.cc index c6c1b142e..b8752dd91 100644 --- a/src/dev/arm/gic_v3_cpu_interface.cc +++ b/src/dev/arm/gic_v3_cpu_interface.cc @@ -675,11 +675,35 @@ Gicv3CPUInterface::readMiscReg(int misc_reg) case MISCREG_ICH_AP0R0_EL2: break; + // only implemented if supporting 6 or more bits of priority + case MISCREG_ICH_AP0R1: + case MISCREG_ICH_AP0R1_EL2: + // only implemented if supporting 7 or more bits of priority + case MISCREG_ICH_AP0R2: + case MISCREG_ICH_AP0R2_EL2: + // only implemented if supporting 7 or more bits of priority + case MISCREG_ICH_AP0R3: + case MISCREG_ICH_AP0R3_EL2: + // Unimplemented registers are RAZ/WI + return 0; + // Hyp Active Priorities Group 1 Registers case MISCREG_ICH_AP1R0: case MISCREG_ICH_AP1R0_EL2: break; + // only implemented if supporting 6 or more bits of priority + case MISCREG_ICH_AP1R1: + case MISCREG_ICH_AP1R1_EL2: + // only implemented if supporting 7 or more bits of priority + case MISCREG_ICH_AP1R2: + case MISCREG_ICH_AP1R2_EL2: + // only implemented if supporting 7 or more bits of priority + case MISCREG_ICH_AP1R3: + case MISCREG_ICH_AP1R3_EL2: + // Unimplemented registers are RAZ/WI + return 0; + // Maintenance Interrupt State Register case MISCREG_ICH_MISR: case MISCREG_ICH_MISR_EL2: @@ -1614,13 +1638,39 @@ Gicv3CPUInterface::setMiscReg(int misc_reg, RegVal val) } // Hyp Active Priorities Group 0 Registers - case MISCREG_ICH_AP0R0 ... MISCREG_ICH_AP0R3: - case MISCREG_ICH_AP0R0_EL2 ... MISCREG_ICH_AP0R3_EL2: + case MISCREG_ICH_AP0R0: + case MISCREG_ICH_AP0R0_EL2: + break; + + // only implemented if supporting 6 or more bits of priority + case MISCREG_ICH_AP0R1: + case MISCREG_ICH_AP0R1_EL2: + // only implemented if supporting 7 or more bits of priority + case MISCREG_ICH_AP0R2: + case MISCREG_ICH_AP0R2_EL2: + // only implemented if supporting 7 or more bits of priority + case MISCREG_ICH_AP0R3: + case MISCREG_ICH_AP0R3_EL2: + // Unimplemented registers are RAZ/WI + return; + // Hyp Active Priorities Group 1 Registers - case MISCREG_ICH_AP1R0 ... MISCREG_ICH_AP1R3: - case MISCREG_ICH_AP1R0_EL2 ... MISCREG_ICH_AP1R3_EL2: + case MISCREG_ICH_AP1R0: + case MISCREG_ICH_AP1R0_EL2: break; + // only implemented if supporting 6 or more bits of priority + case MISCREG_ICH_AP1R1: + case MISCREG_ICH_AP1R1_EL2: + // only implemented if supporting 7 or more bits of priority + case MISCREG_ICH_AP1R2: + case MISCREG_ICH_AP1R2_EL2: + // only implemented if supporting 7 or more bits of priority + case MISCREG_ICH_AP1R3: + case MISCREG_ICH_AP1R3_EL2: + // Unimplemented registers are RAZ/WI + return; + default: panic("Gicv3CPUInterface::setMiscReg(): unknown register %d (%s)", misc_reg, miscRegName[misc_reg]); -- 2.30.2