[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 2/6] hw/arm/virt: Add a control for the the highmem redist
From: |
Eric Auger |
Subject: |
Re: [PATCH v5 2/6] hw/arm/virt: Add a control for the the highmem redistributors |
Date: |
Tue, 18 Jan 2022 14:51:12 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 |
Hi Marc,
On 1/14/22 3:07 PM, Marc Zyngier wrote:
> Just like we can control the enablement of the highmem PCIe region
> using highmem_ecam, let's add a control for the highmem GICv3
> redistributor region.
>
> Similarily to highmem_ecam, these redistributors are disabled when
> highmem is off.
>
> Reviewed-by: Andrew Jones <drjones@redhat.com>
> Signed-off-by: Marc Zyngier <maz@kernel.org>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Eric
> ---
> hw/arm/virt-acpi-build.c | 2 ++
> hw/arm/virt.c | 2 ++
> include/hw/arm/virt.h | 4 +++-
> 3 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
> index 449fab0080..0757c28f69 100644
> --- a/hw/arm/virt-acpi-build.c
> +++ b/hw/arm/virt-acpi-build.c
> @@ -947,6 +947,8 @@ void virt_acpi_build(VirtMachineState *vms,
> AcpiBuildTables *tables)
> acpi_add_table(table_offsets, tables_blob);
> build_fadt_rev5(tables_blob, tables->linker, vms, dsdt);
>
> + vms->highmem_redists &= vms->highmem;
> +
> acpi_add_table(table_offsets, tables_blob);
> build_madt(tables_blob, tables->linker, vms);
>
> diff --git a/hw/arm/virt.c b/hw/arm/virt.c
> index ed8ea96acc..e734a75850 100644
> --- a/hw/arm/virt.c
> +++ b/hw/arm/virt.c
> @@ -2106,6 +2106,7 @@ static void machvirt_init(MachineState *machine)
> virt_flash_fdt(vms, sysmem, secure_sysmem ?: sysmem);
>
> vms->highmem_mmio &= vms->highmem;
> + vms->highmem_redists &= vms->highmem;
>
> create_gic(vms, sysmem);
>
> @@ -2805,6 +2806,7 @@ static void virt_instance_init(Object *obj)
>
> vms->highmem_ecam = !vmc->no_highmem_ecam;
> vms->highmem_mmio = true;
> + vms->highmem_redists = true;
>
> if (vmc->no_its) {
> vms->its = false;
> diff --git a/include/hw/arm/virt.h b/include/hw/arm/virt.h
> index 9c54acd10d..dc9fa26faa 100644
> --- a/include/hw/arm/virt.h
> +++ b/include/hw/arm/virt.h
> @@ -144,6 +144,7 @@ struct VirtMachineState {
> bool highmem;
> bool highmem_ecam;
> bool highmem_mmio;
> + bool highmem_redists;
> bool its;
> bool tcg_its;
> bool virt;
> @@ -190,7 +191,8 @@ static inline int
> virt_gicv3_redist_region_count(VirtMachineState *vms)
>
> assert(vms->gic_version == VIRT_GIC_VERSION_3);
>
> - return MACHINE(vms)->smp.cpus > redist0_capacity ? 2 : 1;
> + return (MACHINE(vms)->smp.cpus > redist0_capacity &&
> + vms->highmem_redists) ? 2 : 1;
> }
>
> #endif /* QEMU_ARM_VIRT_H */
- [PATCH v5 0/6] target/arm: Reduced-IPA space and highmem fixes, Marc Zyngier, 2022/01/14
- [PATCH v5 3/6] hw/arm/virt: Honor highmem setting when computing the memory map, Marc Zyngier, 2022/01/14
- [PATCH v5 1/6] hw/arm/virt: Add a control for the the highmem PCIe MMIO, Marc Zyngier, 2022/01/14
- [PATCH v5 6/6] hw/arm/virt: Drop superfluous checks against highmem, Marc Zyngier, 2022/01/14
- [PATCH v5 5/6] hw/arm/virt: Disable highmem devices that don't fit in the PA range, Marc Zyngier, 2022/01/14
- [PATCH v5 2/6] hw/arm/virt: Add a control for the the highmem redistributors, Marc Zyngier, 2022/01/14
- Re: [PATCH v5 2/6] hw/arm/virt: Add a control for the the highmem redistributors,
Eric Auger <=
- [PATCH v5 4/6] hw/arm/virt: Use the PA range to compute the memory map, Marc Zyngier, 2022/01/14
- Re: [PATCH v5 0/6] target/arm: Reduced-IPA space and highmem fixes, Peter Maydell, 2022/01/18