From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42818) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bKSdg-0001Zb-Hp for qemu-devel@nongnu.org; Tue, 05 Jul 2016 11:51:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bKSdd-0006kq-NT for qemu-devel@nongnu.org; Tue, 05 Jul 2016 11:51:56 -0400 From: Kevin Wolf Date: Tue, 5 Jul 2016 17:50:44 +0200 Message-Id: <1467733852-27097-36-git-send-email-kwolf@redhat.com> In-Reply-To: <1467733852-27097-1-git-send-email-kwolf@redhat.com> References: <1467733852-27097-1-git-send-email-kwolf@redhat.com> Subject: [Qemu-devel] [PULL 35/43] block: Use BlockBackend for I/O in bdrv_commit() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: kwolf@redhat.com, qemu-devel@nongnu.org Just like block jobs, the HMP commit command should use its own BlockBackend for doing I/O on BlockDriverStates. Signed-off-by: Kevin Wolf Reviewed-by: Max Reitz Acked-by: Stefan Hajnoczi --- block/commit.c | 34 ++++++++++++++++++++++------------ 1 file changed, 22 insertions(+), 12 deletions(-) diff --git a/block/commit.c b/block/commit.c index 4ac3df3..379efb7 100644 --- a/block/commit.c +++ b/block/commit.c @@ -289,6 +289,7 @@ void commit_start(BlockDriverState *bs, BlockDriverState *base, /* commit COW file into the raw image */ int bdrv_commit(BlockDriverState *bs) { + BlockBackend *src, *backing; BlockDriver *drv = bs->drv; int64_t sector, total_sectors, length, backing_length; int n, ro, open_flags; @@ -316,13 +317,19 @@ int bdrv_commit(BlockDriverState *bs) } } - length = bdrv_getlength(bs); + src = blk_new(); + blk_insert_bs(src, bs); + + backing = blk_new(); + blk_insert_bs(backing, bs->backing->bs); + + length = blk_getlength(src); if (length < 0) { ret = length; goto ro_cleanup; } - backing_length = bdrv_getlength(bs->backing->bs); + backing_length = blk_getlength(backing); if (backing_length < 0) { ret = backing_length; goto ro_cleanup; @@ -332,7 +339,7 @@ int bdrv_commit(BlockDriverState *bs) * grow the backing file image if possible. If not possible, * we must return an error */ if (length > backing_length) { - ret = bdrv_truncate(bs->backing->bs, length); + ret = blk_truncate(backing, length); if (ret < 0) { goto ro_cleanup; } @@ -340,9 +347,9 @@ int bdrv_commit(BlockDriverState *bs) total_sectors = length >> BDRV_SECTOR_BITS; - /* qemu_try_blockalign() for bs will choose an alignment that works for - * bs->backing->bs as well, so no need to compare the alignment manually. */ - buf = qemu_try_blockalign(bs, COMMIT_BUF_SECTORS * BDRV_SECTOR_SIZE); + /* blk_try_blockalign() for src will choose an alignment that works for + * backing as well, so no need to compare the alignment manually. */ + buf = blk_try_blockalign(src, COMMIT_BUF_SECTORS * BDRV_SECTOR_SIZE); if (buf == NULL) { ret = -ENOMEM; goto ro_cleanup; @@ -354,12 +361,14 @@ int bdrv_commit(BlockDriverState *bs) goto ro_cleanup; } if (ret) { - ret = bdrv_read(bs, sector, buf, n); + ret = blk_pread(src, sector * BDRV_SECTOR_SIZE, buf, + n * BDRV_SECTOR_SIZE); if (ret < 0) { goto ro_cleanup; } - ret = bdrv_write(bs->backing->bs, sector, buf, n); + ret = blk_pwrite(backing, sector * BDRV_SECTOR_SIZE, buf, + n * BDRV_SECTOR_SIZE, 0); if (ret < 0) { goto ro_cleanup; } @@ -371,21 +380,22 @@ int bdrv_commit(BlockDriverState *bs) if (ret < 0) { goto ro_cleanup; } - bdrv_flush(bs); + blk_flush(src); } /* * Make sure all data we wrote to the backing device is actually * stable on disk. */ - if (bs->backing) { - bdrv_flush(bs->backing->bs); - } + blk_flush(backing); ret = 0; ro_cleanup: qemu_vfree(buf); + blk_unref(src); + blk_unref(backing); + if (ro) { /* ignoring error return here */ bdrv_reopen(bs->backing->bs, open_flags & ~BDRV_O_RDWR, NULL); -- 1.8.3.1