On Fri, Feb 21, 2020 at 05:42:01PM +0100, David Hildenbrand wrote: > Resizing while migrating is dangerous and does not work as expected. > The whole migration code works on the usable_length of ram blocks and does > not expect this to change at random points in time. > > In the case of postcopy, relying on used_length is racy as soon as the > guest is running. Also, when used_length changes we might leave the > uffd handler registered for some memory regions, reject valid pages > when migrating and fail when sending the recv bitmap to the source. > > Resizing can be trigger *after* (but not during) a reset in > ACPI code by the guest > - hw/arm/virt-acpi-build.c:acpi_ram_update() > - hw/i386/acpi-build.c:acpi_ram_update() > > Let's remember the original used_length in a separate variable and > use it in relevant postcopy code. Make sure to update it when we resize > during precopy, when synchronizing the RAM block sizes with the source. > > Cc: "Dr. David Alan Gilbert" <[email protected]> > Cc: Juan Quintela <[email protected]> > Cc: Eduardo Habkost <[email protected]> > Cc: Paolo Bonzini <[email protected]> > Cc: Igor Mammedov <[email protected]> > Cc: "Michael S. Tsirkin" <[email protected]> > Cc: Richard Henderson <[email protected]> > Cc: Shannon Zhao <[email protected]> > Cc: Alex Bennée <[email protected]> > Cc: Peter Xu <[email protected]> > Signed-off-by: David Hildenbrand <[email protected]>
Reviewed-by: Peter Xu <[email protected]> -- Peter Xu
