On Wed, 12 Jul 2023 22:09:37 +0530
Sunil V L <suni...@ventanamicro.com> wrote:
> PCIe High MMIO base is actually dynamic and fixed at
> run time based on the RAM configured. Currently, this is
> not part of the memmap and kept in separate static variable
> in virt.c. However, ACPI code also needs this information
> to populate DSDT. So, once the base is discovered, merge
> this into the final memmap which can be used to create
> ACPI tables later.
can ACPI code fetch virt_high_pcie_memmap at runtime from
host bridge (like we do in pc/q35)?
> Signed-off-by: Sunil V L <suni...@ventanamicro.com>
> ---
> hw/riscv/virt.c | 31 ++++++++++++++++++++++++++++++-
> include/hw/riscv/virt.h | 9 +++++++--
> 2 files changed, 37 insertions(+), 3 deletions(-)
>
> diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c
> index f6067db8ec..7aee06f021 100644
> --- a/hw/riscv/virt.c
> +++ b/hw/riscv/virt.c
> @@ -84,6 +84,22 @@ static const MemMapEntry virt_memmap[] = {
>
> static MemMapEntry virt_high_pcie_memmap;
>
> +/*
> + * virt_memmap doesn't include floating High Mem IO address entry. To enable
> + * code organization in multiple files (ex: ACPI), it is better to have
> single
> + * memmap which has complete information.
> + *
> + * VIRT_HIGH_PCIE_MMIO is always greater than the last memmap entry and hence
> + * full_virt_memmap is capable of holding both virt_memmap and
> + * VIRT_HIGH_PCIE_MMIO entry.
> + *
> + * The values for these floating entries will be updated when top of RAM is
> + * discovered.
> + */
> +static MemMapEntry full_virt_memmap[] = {
> + [VIRT_HIGH_PCIE_MMIO] = { 0x0, 0 },
> +};
> +
> #define VIRT_FLASH_SECTOR_SIZE (256 * KiB)
>
> static PFlashCFI01 *virt_flash_create1(RISCVVirtState *s,
> @@ -1444,7 +1460,20 @@ static void virt_machine_init(MachineState *machine)
> ROUND_UP(virt_high_pcie_memmap.base, virt_high_pcie_memmap.size);
> }
>
> - s->memmap = virt_memmap;
> + /*
> + * Initialize the floating values in full memory map
> + */
> + full_virt_memmap[VIRT_HIGH_PCIE_MMIO].base = virt_high_pcie_memmap.base;
> + full_virt_memmap[VIRT_HIGH_PCIE_MMIO].size = virt_high_pcie_memmap.size;
> +
> + s->memmap = full_virt_memmap;
> + /*
> + * Copy the base virt_memmap entries to full memmap
> + */
> + for (i = 0; i < ARRAY_SIZE(virt_memmap); i++) {
> + s->memmap[i] = virt_memmap[i];
> + }
> +
>
> /* register system main memory (actual RAM) */
> memory_region_add_subregion(system_memory, memmap[VIRT_DRAM].base,
> diff --git a/include/hw/riscv/virt.h b/include/hw/riscv/virt.h
> index 00c22492a7..1d7ddf5df0 100644
> --- a/include/hw/riscv/virt.h
> +++ b/include/hw/riscv/virt.h
> @@ -60,7 +60,7 @@ struct RISCVVirtState {
> char *oem_id;
> char *oem_table_id;
> OnOffAuto acpi;
> - const MemMapEntry *memmap;
> + MemMapEntry *memmap;
> PCIBus *bus;
> };
>
> @@ -84,7 +84,12 @@ enum {
> VIRT_PCIE_MMIO,
> VIRT_PCIE_PIO,
> VIRT_PLATFORM_BUS,
> - VIRT_PCIE_ECAM
> + VIRT_PCIE_ECAM,
> + VIRT_LAST_MEMMAP /* Keep this entry always last */
> +};
> +
> +enum {
> + VIRT_HIGH_PCIE_MMIO = VIRT_LAST_MEMMAP,
> };
>
> enum {
