[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 63/86] ppc:pnv: use memdev for RAM
From: |
Cédric Le Goater |
Subject: |
Re: [PATCH v2 63/86] ppc:pnv: use memdev for RAM |
Date: |
Thu, 16 Jan 2020 09:16:02 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2 |
On 1/15/20 4:07 PM, Igor Mammedov wrote:
> memory_region_allocate_system_memory() API is going away, so
> replace it with memdev allocated MemoryRegion. The later is
> initialized by generic code, so board only needs to opt in
> to memdev scheme by providing
> MachineClass::default_ram_id
> and using MachineState::ram instead of manually initializing
> RAM memory region.
>
> Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Cédric Le Goater <address@hidden>
C.
> ---
> CC: address@hidden
> CC: address@hidden
> CC: address@hidden
> ---
> hw/ppc/pnv.c | 8 ++------
> 1 file changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
> index e2735bb..a85a5fc 100644
> --- a/hw/ppc/pnv.c
> +++ b/hw/ppc/pnv.c
> @@ -676,7 +676,6 @@ static void pnv_init(MachineState *machine)
> {
> PnvMachineState *pnv = PNV_MACHINE(machine);
> MachineClass *mc = MACHINE_GET_CLASS(machine);
> - MemoryRegion *ram;
> char *fw_filename;
> long fw_size;
> int i;
> @@ -688,11 +687,7 @@ static void pnv_init(MachineState *machine)
> if (machine->ram_size < (1 * GiB)) {
> warn_report("skiboot may not work with < 1GB of RAM");
> }
> -
> - ram = g_new(MemoryRegion, 1);
> - memory_region_allocate_system_memory(ram, NULL, "pnv.ram",
> - machine->ram_size);
> - memory_region_add_subregion(get_system_memory(), 0, ram);
> + memory_region_add_subregion(get_system_memory(), 0, machine->ram);
>
> /*
> * Create our simple PNOR device
> @@ -1785,6 +1780,7 @@ static void pnv_machine_class_init(ObjectClass *oc,
> void *data)
> * enough to fit the maximum initrd size at it's load address
> */
> mc->default_ram_size = INITRD_LOAD_ADDR + INITRD_MAX_SIZE;
> + mc->default_ram_id = "pnv.ram";
> ispc->print_info = pnv_pic_print_info;
> }
>
>