[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v2 5/6] hw/arm: ACPI SRAT changes to accommodate n
From: |
Andrew Jones |
Subject: |
Re: [Qemu-devel] [RFC v2 5/6] hw/arm: ACPI SRAT changes to accommodate non-contiguous mem |
Date: |
Mon, 28 May 2018 19:02:26 +0200 |
User-agent: |
Mutt/1.6.0.1 (2016-04-01) |
On Wed, May 16, 2018 at 04:20:25PM +0100, Shameer Kolothum wrote:
> This is in preparation for the next patch where initial ram is split
> into a non-pluggable chunk and a pc-dimm modeled mem if the vaild
> iova regions are non-contiguous.
>
> Signed-off-by: Shameer Kolothum <address@hidden>
> ---
> hw/arm/virt-acpi-build.c | 24 ++++++++++++++++++++----
> 1 file changed, 20 insertions(+), 4 deletions(-)
>
> diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
> index c7c6a57..8d17b40 100644
> --- a/hw/arm/virt-acpi-build.c
> +++ b/hw/arm/virt-acpi-build.c
> @@ -488,7 +488,7 @@ build_srat(GArray *table_data, BIOSLinker *linker,
> VirtMachineState *vms)
> AcpiSratProcessorGiccAffinity *core;
> AcpiSratMemoryAffinity *numamem;
> int i, srat_start;
> - uint64_t mem_base;
> + uint64_t mem_base, mem_sz, mem_len;
> MachineClass *mc = MACHINE_GET_CLASS(vms);
> const CPUArchIdList *cpu_list = mc->possible_cpu_arch_ids(MACHINE(vms));
>
> @@ -505,12 +505,28 @@ build_srat(GArray *table_data, BIOSLinker *linker,
> VirtMachineState *vms)
> core->flags = cpu_to_le32(1);
> }
>
> - mem_base = vms->memmap[VIRT_MEM].base;
> + mem_base = vms->bootinfo.loader_start;
> + mem_sz = vms->bootinfo.loader_start;
mem_sz = vms->bootinfo.ram_size;
Assuming the DT generator was correct, meaning bootinfo.ram_size will
be the size of the non-pluggable dimm.
> for (i = 0; i < nb_numa_nodes; ++i) {
> numamem = acpi_data_push(table_data, sizeof(*numamem));
> - build_srat_memory(numamem, mem_base, numa_info[i].node_mem, i,
> + mem_len = MIN(numa_info[i].node_mem, mem_sz);
> + build_srat_memory(numamem, mem_base, mem_len, i,
> MEM_AFFINITY_ENABLED);
> - mem_base += numa_info[i].node_mem;
> + mem_base += mem_len;
> + mem_sz -= mem_len;
> + if (!mem_sz) {
> + break;
> + }
> + }
> +
> + /* Create table for initial pc-dimm ram, if any */
> + if (vms->bootinfo.dimm_mem) {
> + numamem = acpi_data_push(table_data, sizeof(*numamem));
> + build_srat_memory(numamem, vms->bootinfo.dimm_mem->base,
> + vms->bootinfo.dimm_mem->size,
> + vms->bootinfo.dimm_mem->node,
> + MEM_AFFINITY_ENABLED);
> +
> }
>
> build_header(linker, table_data, (void *)(table_data->data + srat_start),
> --
> 2.7.4
>
>
>
- [Qemu-devel] [RFC v2 4/6] hw/arm: Changes required to accommodate non-contiguous DT mem nodes, (continued)
- [Qemu-devel] [RFC v2 4/6] hw/arm: Changes required to accommodate non-contiguous DT mem nodes, Shameer Kolothum, 2018/05/16
- [Qemu-devel] [RFC v2 2/6] hw/arm/virt: Enable dynamic generation of guest RAM memory regions, Shameer Kolothum, 2018/05/16
- [Qemu-devel] [RFC v2 5/6] hw/arm: ACPI SRAT changes to accommodate non-contiguous mem, Shameer Kolothum, 2018/05/16
- [Qemu-devel] [RFC v2 6/6] hw/arm: Populate non-contiguous memory regions, Shameer Kolothum, 2018/05/16
- [Qemu-devel] [RFC v2 3/6] hw/arm/virt: Add pc-dimm mem hotplug framework, Shameer Kolothum, 2018/05/16
- Re: [Qemu-devel] [RFC v2 0/6] hw/arm: Add support for non-contiguous iova regions, Auger Eric, 2018/05/28