From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54191) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1anrhk-0000YG-6Y for qemu-devel@nongnu.org; Wed, 06 Apr 2016 13:57:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1anrhj-0006St-68 for qemu-devel@nongnu.org; Wed, 06 Apr 2016 13:57:24 -0400 From: Max Reitz Date: Wed, 6 Apr 2016 19:57:06 +0200 Message-Id: <1459965434-21531-1-git-send-email-mreitz@redhat.com> Subject: [Qemu-devel] [PATCH v2 for-2.7 0/8] blockdev: (Nearly) free clean-up work List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: Kevin Wolf , Alberto Garcia , qemu-devel@nongnu.org, 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 v2: - Patch 1: bdrv_open_backing_file() has been changed already, so that part can be dropped from the patch - Patch 2: - bdrv_append_temp_snapshot() now takes an additional argument (snapshot_options) - Drop a superfluous assignment to ret [Berto] - The function is now also static already, so we don't need to do that - Patch 3: Added because this test has been added in the meantime - Patch 4: Rebase conflicts - Patch 6: - Rebase conflicts - A whole lot of places we no longer need to touch because they use blk_new_open() instead of bdrv_open() - Patch 8: Added because patch 7 makes this possible. git-backport-diff against v1: 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:[down] 'block: Drop useless bdrv_new() call' ^^^^ Actually [0006] [FC] with a changed commit title 002/8:[0015] [FC] 'block: Let bdrv_open_inherit() return the snapshot' 003/8:[down] 'tests: Drop BDS from test-throttle.c' 004/8:[0019] [FC] 'block: Drop blk_new_with_bs()' 005/8:[----] [-C] 'block: Drop bdrv_new_root()' 006/8:[0135] [FC] 'block: Make bdrv_open() return a BDS' 007/8:[----] [-C] 'block: Assert !bs->refcnt in bdrv_close()' 008/8:[down] '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 | 139 ++++++++++++++++++++------------------------------ 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, 83 insertions(+), 143 deletions(-) -- 2.8.0