[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 0/6] virtio-mem: Handle preallocation with migration
From: |
David Hildenbrand |
Subject: |
[PATCH v3 0/6] virtio-mem: Handle preallocation with migration |
Date: |
Thu, 22 Dec 2022 12:02:09 +0100 |
While playing with migration of virtio-mem with an ordinary file backing,
I realized that migration and prealloc doesn't currently work as expected
for virtio-mem. Further, Jing Qi reported that setup issues (insufficient
huge pages on the destination) result in QEMU getting killed with SIGBUS
instead of failing gracefully.
In contrast to ordinary memory backend preallocation, virtio-mem
preallocates memory before plugging blocks to the guest. Consequently,
when migrating we are not actually preallocating on the destination but
"only" migrate pages. Fix that be migrating the bitmap early, before any
RAM content, and use that information to preallocate memory early, before
migrating any RAM.
Postcopy needs some extra care, and I realized that prealloc+postcopy is
shaky in general. Let's at least try to mimic what ordinary
prealloc+postcopy does: temporarily allocate the memory, discard it, and
cross fingers that we'll still have sufficient memory when postcopy
actually tries placing pages.
Cc: Dr. David Alan Gilbert <dgilbert@redhat.com>
Cc: Juan Quintela <quintela@redhat.com>
Cc: Peter Xu <peterx@redhat.com>
Cc: Michael S. Tsirkin <mst@redhat.com>
Cc: Michal Privoznik <mprivozn@redhat.com>
v2 -> v3:
- New approach/rewrite, drop RB and TB of last patch
v1 -> v2:
- Added RBs and Tested-bys
- "virtio-mem: Fail if a memory backend with "prealloc=on" is specified"
-- Fail instead of warn
-- Adjust subject/description
David Hildenbrand (6):
migration: Allow immutable device state to be migrated early (i.e.,
before RAM)
migration/vmstate: Introduce VMSTATE_WITH_TMP_TEST() and
VMSTATE_BITMAP_TEST()
migration: Factor out checks for advised and listening incomming
postcopy
virtio-mem: Fail if a memory backend with "prealloc=on" is specified
virtio-mem: Migrate bitmap, size and sanity checks early
virtio-mem: Proper support for preallocation with migration
hw/core/machine.c | 4 +-
hw/virtio/virtio-mem.c | 154 ++++++++++++++++++++++++++++++++-
include/hw/virtio/virtio-mem.h | 8 ++
include/migration/misc.h | 6 +-
include/migration/vmstate.h | 19 +++-
migration/migration.c | 27 ++++++
migration/migration.h | 4 +
migration/ram.c | 16 +---
migration/savevm.c | 104 ++++++++++++++++------
migration/savevm.h | 1 +
10 files changed, 298 insertions(+), 45 deletions(-)
--
2.38.1
- [PATCH v3 0/6] virtio-mem: Handle preallocation with migration,
David Hildenbrand <=
- [PATCH v3 1/6] migration: Allow immutable device state to be migrated early (i.e., before RAM), David Hildenbrand, 2022/12/22
- [PATCH v3 3/6] migration: Factor out checks for advised and listening incomming postcopy, David Hildenbrand, 2022/12/22
- [PATCH v3 2/6] migration/vmstate: Introduce VMSTATE_WITH_TMP_TEST() and VMSTATE_BITMAP_TEST(), David Hildenbrand, 2022/12/22
- [PATCH v3 4/6] virtio-mem: Fail if a memory backend with "prealloc=on" is specified, David Hildenbrand, 2022/12/22
- [PATCH v3 5/6] virtio-mem: Migrate bitmap, size and sanity checks early, David Hildenbrand, 2022/12/22
- [PATCH v3 6/6] virtio-mem: Proper support for preallocation with migration, David Hildenbrand, 2022/12/22