[Qemu-devel] [PATCH 0/9] blockdev: Further BlockBackend work

[Max Reitz <mreitz@redhat.com>]

Overall, this series adds more uses for BlockBackends and makes the code
follow the "reference theory" more closely, in that any BlockBackend
created (through -drive or blockdev-add) has a reference count of 1, and
that reference should be held by the monitor. This is reflected here by
introducing an explicit list of monitor-owned BlockBackends, which in
turn allows us to remove bdrv_states, and, perhaps most importantly,
blk_hide_on_behalf_of_do_drive_del().


This series depends on v3 (or any later version) of my series
"block: Rework bdrv_close_all()" (or any later version).


Differences between the patches here and the ones included in v2 of
"block: Rework bdrv_close_all()":
- Patch 4: bdrv_close_all() looks different now
- Patch 5:
  - Fixed several bugs relating to blockdev_auto_del()
    (monitor_remove_blk() is now idempotent, so it is just called
    everywhere where we want the BB to disappear)
  - monitor_blk_unref() does no longer exist


git-backport-diff against v2 of "block: Rework bdrv_close_all()":

Key:
[----] : patches are identical
[####] : number of functional differences between upstream/downstream patch
[down] : patch is downstream-only
The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively

001/9:[----] [-C] 'block: Add blk_name_taken()'
002/9:[----] [--] 'block: Add blk_next_inserted()'
003/9:[----] [--] 'block: Add blk_commit_all() and blk_invalidate_cache_all()'
004/9:[0008] [FC] 'block: Use BlockBackend more'
005/9:[0025] [FC] 'blockdev: Add list of monitor-owned BlockBackends'
006/9:[----] [-C] 'blockdev: Remove blk_hide_on_behalf_of_do_drive_del()'
007/9:[----] [--] 'block: Make bdrv_drain_one() public'
008/9:[----] [-C] 'block: Move some bdrv_*_all() functions to BB'
009/9:[----] [-C] 'block: Remove bdrv_states'


Max Reitz (9):
  block: Add blk_name_taken()
  block: Add blk_next_inserted()
  block: Add blk_commit_all() and blk_invalidate_cache_all()
  block: Use BlockBackend more
  blockdev: Add list of monitor-owned BlockBackends
  blockdev: Remove blk_hide_on_behalf_of_do_drive_del()
  block: Make bdrv_drain_one() public
  block: Move some bdrv_*_all() functions to BB
  block: Remove bdrv_states

 block.c                        | 138 +++------------------------
 block/block-backend.c          | 211 +++++++++++++++++++++++++++++++++--------
 block/qapi.c                   |  13 ++-
 block/snapshot.c               |   3 +-
 blockdev.c                     |  19 ++--
 cpus.c                         |   7 +-
 include/block/block.h          |   5 -
 include/block/block_int.h      |   4 +-
 include/sysemu/block-backend.h |   8 +-
 migration/block.c              |  10 +-
 migration/migration.c          |   4 +-
 monitor.c                      |  13 ++-
 qemu-char.c                    |   3 +-
 qemu-io.c                      |   2 +-
 qmp.c                          |   9 +-
 savevm.c                       |  66 +++++++------
 stubs/Makefile.objs            |   2 +-
 stubs/bdrv-commit-all.c        |   7 --
 stubs/blk-commit-all.c         |   7 ++
 xen-mapcache.c                 |   3 +-
 20 files changed, 293 insertions(+), 241 deletions(-)
 delete mode 100644 stubs/bdrv-commit-all.c
 create mode 100644 stubs/blk-commit-all.c

-- 
2.1.0