[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v2 2/6] hw/arm/virt: Enable dynamic generation of
From: |
Andrew Jones |
Subject: |
Re: [Qemu-devel] [RFC v2 2/6] hw/arm/virt: Enable dynamic generation of guest RAM memory regions |
Date: |
Mon, 28 May 2018 18:47:12 +0200 |
User-agent: |
Mutt/1.6.0.1 (2016-04-01) |
On Wed, May 16, 2018 at 04:20:22PM +0100, Shameer Kolothum wrote:
> Register ram_memory_region_init notifier to allocate memory region
> from system memory.
>
> Signed-off-by: Zhu Yijun <address@hidden>
> Signed-off-by: Shameer Kolothum <address@hidden>
> ---
> hw/arm/virt.c | 28 ++++++++++++++++++++++------
> include/hw/arm/virt.h | 1 +
> 2 files changed, 23 insertions(+), 6 deletions(-)
>
> diff --git a/hw/arm/virt.c b/hw/arm/virt.c
> index 94dcb12..05fcb62 100644
> --- a/hw/arm/virt.c
> +++ b/hw/arm/virt.c
> @@ -1171,6 +1171,19 @@ void virt_machine_done(Notifier *notifier, void *data)
> virt_build_smbios(vms);
> }
>
> +static void virt_ram_memory_region_init(Notifier *notifier, void *data)
> +{
> + MemoryRegion *sysmem = get_system_memory();
> + MemoryRegion *ram = g_new(MemoryRegion, 1);
> + VirtMachineState *vms = container_of(notifier, VirtMachineState,
> + ram_memory_region_init);
> + MachineState *machine = MACHINE(vms);
> +
> + memory_region_allocate_system_memory(ram, NULL, "mach-virt.ram",
> + machine->ram_size);
> + memory_region_add_subregion(sysmem, vms->memmap[VIRT_MEM].base, ram);
> +}
> +
> static uint64_t virt_cpu_mp_affinity(VirtMachineState *vms, int idx)
> {
> uint8_t clustersz = ARM_DEFAULT_CPUS_PER_CLUSTER;
> @@ -1204,7 +1217,6 @@ static void machvirt_init(MachineState *machine)
> MemoryRegion *sysmem = get_system_memory();
> MemoryRegion *secure_sysmem = NULL;
> int n, virt_max_cpus;
> - MemoryRegion *ram = g_new(MemoryRegion, 1);
> bool firmware_loaded = bios_name || drive_get(IF_PFLASH, 0, 0);
>
> /* We can probe only here because during property set
> @@ -1361,10 +1373,6 @@ static void machvirt_init(MachineState *machine)
> fdt_add_timer_nodes(vms);
> fdt_add_cpu_nodes(vms);
>
> - memory_region_allocate_system_memory(ram, NULL, "mach-virt.ram",
> - machine->ram_size);
> - memory_region_add_subregion(sysmem, vms->memmap[VIRT_MEM].base, ram);
> -
> create_flash(vms, sysmem, secure_sysmem ? secure_sysmem : sysmem);
>
> create_gic(vms, pic);
> @@ -1405,15 +1413,23 @@ static void machvirt_init(MachineState *machine)
> vms->bootinfo.loader_start = vms->memmap[VIRT_MEM].base;
> vms->bootinfo.get_dtb = machvirt_dtb;
> vms->bootinfo.firmware_loaded = firmware_loaded;
> +
> + /* Register notifiers. They are executed in registration reverse order */
> arm_load_kernel(ARM_CPU(first_cpu), &vms->bootinfo);
>
> /*
> * arm_load_kernel machine init done notifier registration must
> * happen before the platform_bus_create call. In this latter,
> * another notifier is registered which adds platform bus nodes.
> - * Notifiers are executed in registration reverse order.
> */
> create_platform_bus(vms, pic);
> +
> + /*
> + * Register memory region notifier last as this has to be executed
> + * first.
> + */
> + vms->ram_memory_region_init.notify = virt_ram_memory_region_init;
> + qemu_add_machine_init_done_notifier(&vms->ram_memory_region_init);
I realize we've been slow to get to this for reviewing, but a lot has
changed here since this patch was written. virt now only has a single
machine done notifier. On rebase we should attempt to integrate with
that one rather than introduce a new one.
Thanks,
drew
> }
>
> static bool virt_get_secure(Object *obj, Error **errp)
> diff --git a/include/hw/arm/virt.h b/include/hw/arm/virt.h
> index ba0c1a4..fc24f3a 100644
> --- a/include/hw/arm/virt.h
> +++ b/include/hw/arm/virt.h
> @@ -91,6 +91,7 @@ typedef struct {
> typedef struct {
> MachineState parent;
> Notifier machine_done;
> + Notifier ram_memory_region_init;
> FWCfgState *fw_cfg;
> bool secure;
> bool highmem;
> --
> 2.7.4
>
>
>
- [Qemu-devel] [RFC v2 0/6] hw/arm: Add support for non-contiguous iova regions, Shameer Kolothum, 2018/05/16
- [Qemu-devel] [RFC v2 1/6] hw/vfio: Retrieve valid iova ranges from kernel, Shameer Kolothum, 2018/05/16
- [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