Thanks to visit codestin.com
Credit goes to github.com

Skip to content

vmm: Reject snapshot/migration if on-demand restore still active - #8556

Merged
likebreath merged 3 commits into
cloud-hypervisor:mainfrom
rbradford:202607/prefault-snapshot-migrate-gate
Jul 8, 2026
Merged

likebreath merged 3 commits into
cloud-hypervisor:mainfrom
rbradford:202607/prefault-snapshot-migrate-gate

Conversation

@rbradford

Copy link
Copy Markdown
Member
  • vmm: Record when the on-demand memory restore is done
  • vmm: Error out on migration & snapshot if on-demand restoring
  • tests: Add integration tests to snapshot after/during restore

rbradford added 3 commits July 8, 2026 06:09
In order to deny migration or snapshot when currently doing an on-demand
restore it is necessary to track whether the prefaulting is completed.
This is a proxy for on-demand restoring being completed as pages that
have been restored by a userfaultfd request will excluded from the
prefault set.

Fixes: cloud-hypervisor#8525

Signed-off-by: Rob Bradford <[email protected]>
If there is an active on-demand restoration then reject any requests to
migrate or snapshot this VM as the memory will not be available for the
snapshot.

Signed-off-by: Rob Bradford <[email protected]>
Check that we can make a successful snapshot (and restore it) after
another restore. Also check that snapshot it refused until restore is
complete.

Assisted-by: Claude:Opus-4.8
Signed-off-by: Rob Bradford <[email protected]>
@rbradford
rbradford marked this pull request as ready for review July 8, 2026 14:31
@rbradford
rbradford requested a review from a team as a code owner July 8, 2026 14:31
@rbradford rbradford added the bug-fix Bug fix to include in release notes label Jul 8, 2026
@likebreath
likebreath added this pull request to the merge queue Jul 8, 2026
Merged via the queue into cloud-hypervisor:main with commit 19289a3 Jul 8, 2026
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Bug fix to include in release notes

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants