[Qemu-devel] [PATCH v3 for-2.7 0/8] blockdev: (Nearly) free clean-up work

[Max Reitz]

After a lot has been restructed in the block layer in the past, we can
now reap at least one of the fruits: Make bdrv_open() return a BDS!


This series depends on the following series/patches:
- Revert "block: Forbid I/O throttling on nodes with multiple parents
  for 2.6"
  This is something I suppose Kevin will send when the 2.7 development
  window opens.
- "block: Move I/O throttling to BlockBackend" by Kevin
- "block: Remove BlockDriverState.blk" by Kevin


v3:
- Patch 2: Fattened commit message and comments [Kevin]
- Patch 4: Mentioned that the “inlining” involves some functional
           changes here [Kevin]
- Patch 5: Reverted to the commit message of v1. I temporarily had a
           different patch order before I sent out v2, and therefore I
           broke the commit message (and accidentally kept Berto's R-b,
           because I didn't notice the change in the commit message,
           while the patch itself did not change, sorry!). [Kevin]
- Patch 6:
  - Added hunk removing a now-superfluous ret assignment [Kevin]
  - Rebase conflicts due to the changes to patch 2


git-backport-diff against v2:

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/8:[----] [--] 'block: Drop useless bdrv_new() call'
002/8:[0020] [FC] 'block: Let bdrv_open_inherit() return the snapshot'
003/8:[----] [--] 'tests: Drop BDS from test-throttle.c'
004/8:[----] [--] 'block: Drop blk_new_with_bs()'
005/8:[----] [--] 'block: Drop bdrv_new_root()'
006/8:[0023] [FC] 'block: Make bdrv_open() return a BDS'
007/8:[----] [--] 'block: Assert !bs->refcnt in bdrv_close()'
008/8:[----] [--] 'block: Drop bdrv_parent_cb_...() from bdrv_close()'


Max Reitz (8):
  block: Drop useless bdrv_new() call
  block: Let bdrv_open_inherit() return the snapshot
  tests: Drop BDS from test-throttle.c
  block: Drop blk_new_with_bs()
  block: Drop bdrv_new_root()
  block: Make bdrv_open() return a BDS
  block: Assert !bs->refcnt in bdrv_close()
  block: Drop bdrv_parent_cb_...() from bdrv_close()

 block.c               | 146 +++++++++++++++++++++-----------------------------
 block/block-backend.c |  30 +++--------
 block/vvfat.c         |   8 +--
 blockdev.c            |  38 +++++--------
 include/block/block.h |   5 +-
 tests/test-throttle.c |   6 +--
 6 files changed, 89 insertions(+), 144 deletions(-)

-- 
2.8.0