mbox series

[v2,0/3] migration: Fixes to the 'background-snapshot' code

Message ID 20210331154809.86052-1-andrey.gruzdev@virtuozzo.com (mailing list archive)
Headers show
Series migration: Fixes to the 'background-snapshot' code | expand

Message

Andrey Gruzdev March 31, 2021, 3:48 p.m. UTC
Changes v1->v2:
 * Added comment over the overlooked qemu_flush() in bg_migration_thread

Changes v0->v1:
 * Using qemu_real_host_page_size instead of TARGET_PAGE_SIZE for host
   page size in ram_block_populate_pages()
 * More elegant implementation of ram_block_populate_pages()

This patch series contains:
 * Fix to the issue with occasionally truncated non-iterable device state
 * Solution to compatibility issues with virtio-balloon device
 * Fix to the issue when discarded or never populated pages miss UFFD
   write protection and get into migration stream in dirty state

Andrey Gruzdev (3):
  migration: Fix missing qemu_fflush() on buffer file in
    bg_migration_thread
  migration: Inhibit virtio-balloon for the duration of background
    snapshot
  migration: Pre-fault memory before starting background snasphot

 hw/virtio/virtio-balloon.c |  8 +++++--
 include/migration/misc.h   |  2 ++
 migration/migration.c      | 22 ++++++++++++++++-
 migration/ram.c            | 48 ++++++++++++++++++++++++++++++++++++++
 migration/ram.h            |  1 +
 5 files changed, 78 insertions(+), 3 deletions(-)

Comments

Peter Xu March 31, 2021, 4:02 p.m. UTC | #1
On Wed, Mar 31, 2021 at 06:48:06PM +0300, Andrey Gruzdev wrote:
> Changes v1->v2:
>  * Added comment over the overlooked qemu_flush() in bg_migration_thread
> 
> Changes v0->v1:
>  * Using qemu_real_host_page_size instead of TARGET_PAGE_SIZE for host
>    page size in ram_block_populate_pages()
>  * More elegant implementation of ram_block_populate_pages()
> 
> This patch series contains:
>  * Fix to the issue with occasionally truncated non-iterable device state
>  * Solution to compatibility issues with virtio-balloon device
>  * Fix to the issue when discarded or never populated pages miss UFFD
>    write protection and get into migration stream in dirty state
> 
> Andrey Gruzdev (3):
>   migration: Fix missing qemu_fflush() on buffer file in
>     bg_migration_thread
>   migration: Inhibit virtio-balloon for the duration of background
>     snapshot
>   migration: Pre-fault memory before starting background snasphot

Should this be 6.0-rc material, maybe?
Andrey Gruzdev March 31, 2021, 5:21 p.m. UTC | #2
On 31.03.2021 19:02, Peter Xu wrote:
> On Wed, Mar 31, 2021 at 06:48:06PM +0300, Andrey Gruzdev wrote:
>> Changes v1->v2:
>>   * Added comment over the overlooked qemu_flush() in bg_migration_thread
>>
>> Changes v0->v1:
>>   * Using qemu_real_host_page_size instead of TARGET_PAGE_SIZE for host
>>     page size in ram_block_populate_pages()
>>   * More elegant implementation of ram_block_populate_pages()
>>
>> This patch series contains:
>>   * Fix to the issue with occasionally truncated non-iterable device state
>>   * Solution to compatibility issues with virtio-balloon device
>>   * Fix to the issue when discarded or never populated pages miss UFFD
>>     write protection and get into migration stream in dirty state
>>
>> Andrey Gruzdev (3):
>>    migration: Fix missing qemu_fflush() on buffer file in
>>      bg_migration_thread
>>    migration: Inhibit virtio-balloon for the duration of background
>>      snapshot
>>    migration: Pre-fault memory before starting background snasphot
> Should this be 6.0-rc material, maybe?
>
Ah, yes)