block: Use bdrv_reopen_set_read_only() in bdrv_commit()

This patch replaces the bdrv_reopen() calls that set and remove the
BDRV_O_RDWR flag with the new bdrv_reopen_set_read_only() function.

Signed-off-by: Alberto Garcia <berto@igalia.com>
Reviewed-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
Alberto Garcia 2018-11-12 16:00:36 +02:00 committed by Kevin Wolf
parent e70cdc57da
commit c742a3643f
1 changed files with 3 additions and 4 deletions

View File

@ -391,7 +391,7 @@ int bdrv_commit(BlockDriverState *bs)
BlockDriverState *commit_top_bs = NULL; BlockDriverState *commit_top_bs = NULL;
BlockDriver *drv = bs->drv; BlockDriver *drv = bs->drv;
int64_t offset, length, backing_length; int64_t offset, length, backing_length;
int ro, open_flags; int ro;
int64_t n; int64_t n;
int ret = 0; int ret = 0;
uint8_t *buf = NULL; uint8_t *buf = NULL;
@ -410,10 +410,9 @@ int bdrv_commit(BlockDriverState *bs)
} }
ro = bs->backing->bs->read_only; ro = bs->backing->bs->read_only;
open_flags = bs->backing->bs->open_flags;
if (ro) { if (ro) {
if (bdrv_reopen(bs->backing->bs, open_flags | BDRV_O_RDWR, NULL)) { if (bdrv_reopen_set_read_only(bs->backing->bs, false, NULL)) {
return -EACCES; return -EACCES;
} }
} }
@ -523,7 +522,7 @@ ro_cleanup:
if (ro) { if (ro) {
/* ignoring error return here */ /* ignoring error return here */
bdrv_reopen(bs->backing->bs, open_flags & ~BDRV_O_RDWR, NULL); bdrv_reopen_set_read_only(bs->backing->bs, true, NULL);
} }
return ret; return ret;