Skip to content

dirty-bitmaps: fix QMP command permissions

John Snow qemu-devel 1 version Applied

Status from this message: Thanks, applied to my bitmaps tree: [with edits suggested by Vladimir];

Get series with b4
b4 am [email protected]

Versions newest first

  1. v4 Latest6 patches 21 repliesApplied

Changes in v4 from the cover letter

- Replace "in-use" with "in use"
- Replace "user_modifiable" version with "user_locked"
- Remove more usages of frozen-and-or-locked in NBD.

Patches in v4 Applied

#Subject and filesSizeRepliesApplied
1/6block/dirty-bitmaps: add user_locked status checkerblock/dirty-bitmap.c blockdev.c include/block/dirty-bitmap.h +1 more+17 -294 replies
2/6block/dirty-bitmaps: fix merge permissionsblock/dirty-bitmap.c+3 -32 replies
3/6block/dirty-bitmaps: allow clear on disabled bitmapsblock/dirty-bitmap.c blockdev.c+0 -92 replies
4/6block/dirty-bitmaps: prohibit enable/disable on locked/frozen bitmapsblockdev.c+20 -62 replies
5/6block/backup: prohibit backup from using in use bitmapsblockdev.c+6 -64 replies
6/6nbd: forbid use of frozen bitmapsnbd/server.c+2 -22 replies

Reviews trailers per patch

All 6 patches have a Reviewed-by or Acked-by.

PatchEric BlakeVladimir Sementsov-Ogievskiy
1/6R-b R-b
2/6R-b R-b
3/6R-b R-b
4/6R-b R-b
5/6R-b
6/6R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v4 thread

Cover letter [Qemu-devel] [PATCH v4 0/6] dirty-bitmaps: fix QMP command permissions

based on: jsnow/bitmaps staging branch

This series builds on a previous standalone patch and adjusts the permission for all (or most) of the QMP bitmap commands.

V4:
- Replace "in-use" with "in use"
- Replace "user_modifiable" version with "user_locked"
- Remove more usages of frozen-and-or-locked in NBD.

John Snow (6):
block/dirty-bitmaps: add user_locked status checker block/dirty-bitmaps: fix merge permissions block/dirty-bitmaps: allow clear on disabled bitmaps block/dirty-bitmaps: prohibit enable/disable on locked/frozen bitmaps block/backup: prohibit backup from using in use bitmaps
nbd: forbid use of frozen bitmaps

 block/dirty-bitmap.c           | 13 +++++---
 blockdev.c                     | 75 +++++++++++++++++++-----------------------
 include/block/dirty-bitmap.h   |  1 +
 migration/block-dirty-bitmap.c | 10 ++----
 nbd/server.c                   |  4 +--
 5 files changed, 48 insertions(+), 55 deletions(-)
Signature · 1 line
2.14.4