[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v2 1/7] exec: Fix for qemu_ram_resize() callback
From: |
Shameerali Kolothum Thodi |
Subject: |
RE: [PATCH v2 1/7] exec: Fix for qemu_ram_resize() callback |
Date: |
Wed, 11 Mar 2020 17:28:50 +0000 |
> -----Original Message-----
> From: David Hildenbrand [mailto:address@hidden]
> Sent: 28 February 2020 18:00
> To: Shameerali Kolothum Thodi <address@hidden>;
> Igor Mammedov <address@hidden>
> Cc: address@hidden; address@hidden;
> address@hidden; address@hidden; address@hidden;
> xuwei (O) <address@hidden>; Linuxarm <address@hidden>;
> address@hidden; address@hidden; address@hidden;
> address@hidden; Juan Jose Quintela Carreira <address@hidden>
> Subject: Re: [PATCH v2 1/7] exec: Fix for qemu_ram_resize() callback
>
[...]
>
> We should instead think about
>
> 1. Migrating the actual size of the 3 memory regions separately and setting
> them via
> memory_region_ram_resize() when loading the vmstate. This will trigger
> another FW cfg
> fixup and should be fine (with the same qemu_ram_resize() above).
>
> 2. Introduce a new RAM_SAVE_FLAG_MEM_SIZE_2, that e.g., stores the
> number of ramblocks,
> not the total amount of memory of the ram blocks. But it's hacky, because we
> migrate
> something for RAM blocks, that is not a RAM block concept (sub-block sizes).
>
> I think you should look into 1. Shouldn't be too hard I think.
I have send out v3 of this series ([PATCH v3 00/10] ARM virt: Add NVDIMM
support)
with an attempt to migrate the memory regions separately. It also includes
your patch for qemu_ram_resize() callback fix. Please take a look and let me
know.
Thanks,
Shameer
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- RE: [PATCH v2 1/7] exec: Fix for qemu_ram_resize() callback,
Shameerali Kolothum Thodi <=