[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 4/5] hw/arm/virt-acpi-build: Present the GICR structure properly f
From: |
Peter Maydell |
Subject: |
[PULL 4/5] hw/arm/virt-acpi-build: Present the GICR structure properly for GICv4 |
Date: |
Fri, 12 Aug 2022 12:45:26 +0100 |
From: Zenghui Yu <yuzenghui@huawei.com>
With the introduction of the new TCG GICv4, build_madt() is badly broken
as we do not present any GIC Redistributor structure in MADT for GICv4
guests, so that they have no idea about where the Redistributor
register frames are. This fixes a Linux guest crash at boot time with
ACPI enabled and '-machine gic-version=4'.
While at it, let's convert the remaining hard coded gic_version into
enumeration VIRT_GIC_VERSION_2 for consistency.
Signed-off-by: Zenghui Yu <yuzenghui@huawei.com>
Message-id: 20220812022018.1069-1-yuzenghui@huawei.com
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
hw/arm/virt-acpi-build.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
index 449fab00805..9b3aee01bf8 100644
--- a/hw/arm/virt-acpi-build.c
+++ b/hw/arm/virt-acpi-build.c
@@ -732,7 +732,7 @@ build_madt(GArray *table_data, BIOSLinker *linker,
VirtMachineState *vms)
uint32_t pmu_interrupt = arm_feature(&armcpu->env, ARM_FEATURE_PMU) ?
PPI(VIRTUAL_PMU_IRQ) : 0;
- if (vms->gic_version == 2) {
+ if (vms->gic_version == VIRT_GIC_VERSION_2) {
physical_base_address = memmap[VIRT_GIC_CPU].base;
gicv = memmap[VIRT_GIC_VCPU].base;
gich = memmap[VIRT_GIC_HYP].base;
@@ -762,7 +762,7 @@ build_madt(GArray *table_data, BIOSLinker *linker,
VirtMachineState *vms)
build_append_int_noprefix(table_data, armcpu->mp_affinity, 8);
}
- if (vms->gic_version == 3) {
+ if (vms->gic_version != VIRT_GIC_VERSION_2) {
build_append_gicr(table_data, memmap[VIRT_GIC_REDIST].base,
memmap[VIRT_GIC_REDIST].size);
if (virt_gicv3_redist_region_count(vms) == 2) {
--
2.25.1
- [PULL 0/5] target-arm queue, Peter Maydell, 2022/08/12
- [PULL 3/5] tests/unit: fix a -Wformat-truncation warning, Peter Maydell, 2022/08/12
- [PULL 2/5] Fix some typos in documentation (most of them found by codespell), Peter Maydell, 2022/08/12
- [PULL 4/5] hw/arm/virt-acpi-build: Present the GICR structure properly for GICv4,
Peter Maydell <=
- [PULL 5/5] cutils: Add missing dyld(3) include on macOS, Peter Maydell, 2022/08/12
- [PULL 1/5] target/arm: Don't report Statistical Profiling Extension in ID registers, Peter Maydell, 2022/08/12
- Re: [PULL 0/5] target-arm queue, Richard Henderson, 2022/08/12