Skip to content

block: Error out on load_vm with active dirty bitmaps

Kevin Wolf qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 8 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3block: Add errp to bdrv_snapshot_goto()block/snapshot.c include/block/snapshot.h qemu-img.c+21 -93 replies
2/3block: Add errp to bdrv_all_goto_snapshot()block/snapshot.c include/block/snapshot.h migration/savevm.c+11 -91 reply
3/3block: Error out on load_vm with active dirty bitmapsblock/snapshot.c+3 -121 reply

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchEric BlakeDenis V. LunevVladimir Sementsov-OgievskiyJohn Snow
SeriesR-b R-b R-b
1/3R-b
2/3R-b
3/3R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 4 reviewers.

Read the v1 thread

Cover letter [Qemu-devel] [PATCH for-2.11 0/3] block: Error out on load_vm with active dirty bitmaps

Following the discussing at
https://lists.gnu.org/archive/html/qemu-devel/2017-11/msg03572.html
this implements the error return for loading a snapshot while dirty bitmaps are active.

Kevin Wolf (3):
block: Add errp to bdrv_snapshot_goto()
block: Add errp to bdrv_all_goto_snapshot()
block: Error out on load_vm with active dirty bitmaps

 include/block/snapshot.h |  6 ++++--
 block/snapshot.c         | 43 +++++++++++++++++++++++--------------------
 migration/savevm.c       |  6 +++---
 qemu-img.c               |  6 +++---
 4 files changed, 33 insertions(+), 28 deletions(-)
Signature · 1 line
2.13.6