[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v10 05/11] block: Add bdrv_set_backing_hd()
From: |
Fam Zheng |
Subject: |
[Qemu-devel] [PATCH v10 05/11] block: Add bdrv_set_backing_hd() |
Date: |
Wed, 8 Jan 2014 11:42:44 +0800 |
This is the common but non-trivial steps to assign or change the
backing_hd of BDS.
Signed-off-by: Fam Zheng <address@hidden>
---
block.c | 34 ++++++++++++++++++++++++++++------
include/block/block.h | 1 +
2 files changed, 29 insertions(+), 6 deletions(-)
diff --git a/block.c b/block.c
index b122154..ff25749 100644
--- a/block.c
+++ b/block.c
@@ -958,6 +958,29 @@ fail:
return ret;
}
+void bdrv_set_backing_hd(BlockDriverState *bs, BlockDriverState *backing_hd)
+{
+ if (bs->backing_hd) {
+ bdrv_unref(bs->backing_hd);
+ }
+
+ bs->backing_hd = backing_hd;
+ if (!backing_hd) {
+ bs->backing_file[0] = '\0';
+ bs->backing_format[0] = '\0';
+ return;
+ }
+ pstrcpy(bs->backing_file, sizeof(bs->backing_file), backing_hd->filename);
+ pstrcpy(bs->backing_format, sizeof(bs->backing_file),
+ backing_hd->drv ? backing_hd->drv->format_name : "");
+ bdrv_ref(bs->backing_hd);
+
+ pstrcpy(bs->backing_file, sizeof(bs->backing_file),
+ bs->backing_hd->file->filename);
+ pstrcpy(bs->backing_format, sizeof(bs->backing_format),
+ bs->backing_hd->drv->format_name);
+}
+
/*
* Opens the backing file for a BlockDriverState if not yet open
*
@@ -971,6 +994,7 @@ int bdrv_open_backing_file(BlockDriverState *bs, QDict
*options, Error **errp)
char backing_filename[PATH_MAX];
int back_flags, ret;
BlockDriver *back_drv = NULL;
+ BlockDriverState *backing_hd;
Error *local_err = NULL;
if (bs->backing_hd != NULL) {
@@ -994,7 +1018,7 @@ int bdrv_open_backing_file(BlockDriverState *bs, QDict
*options, Error **errp)
sizeof(backing_filename));
}
- bs->backing_hd = bdrv_new("");
+ backing_hd = bdrv_new("");
if (bs->backing_format[0] != '\0') {
back_drv = bdrv_find_format(bs->backing_format);
@@ -1004,20 +1028,18 @@ int bdrv_open_backing_file(BlockDriverState *bs, QDict
*options, Error **errp)
back_flags = bs->open_flags & ~(BDRV_O_RDWR | BDRV_O_SNAPSHOT |
BDRV_O_COPY_ON_READ);
- ret = bdrv_open(bs->backing_hd,
+ ret = bdrv_open(backing_hd,
*backing_filename ? backing_filename : NULL, options,
back_flags, back_drv, &local_err);
if (ret < 0) {
- bdrv_unref(bs->backing_hd);
- bs->backing_hd = NULL;
+ bdrv_unref(backing_hd);
bs->open_flags |= BDRV_O_NO_BACKING;
error_setg(errp, "Could not open backing file: %s",
error_get_pretty(local_err));
error_free(local_err);
return ret;
}
- pstrcpy(bs->backing_file, sizeof(bs->backing_file),
- bs->backing_hd->file->filename);
+ bdrv_set_backing_hd(bs, backing_hd);
return 0;
}
diff --git a/include/block/block.h b/include/block/block.h
index 5339f92..4ec0069 100644
--- a/include/block/block.h
+++ b/include/block/block.h
@@ -204,6 +204,7 @@ int bdrv_parse_cache_flags(const char *mode, int *flags);
int bdrv_parse_discard_flags(const char *mode, int *flags);
int bdrv_file_open(BlockDriverState **pbs, const char *filename,
QDict *options, int flags, Error **errp);
+void bdrv_set_backing_hd(BlockDriverState *bs, BlockDriverState *backing_hd);
int bdrv_open_backing_file(BlockDriverState *bs, QDict *options, Error **errp);
int bdrv_open(BlockDriverState *bs, const char *filename, QDict *options,
int flags, BlockDriver *drv, Error **errp);
--
1.8.5.1
- [Qemu-devel] [PATCH v10 00/11] Drop in_use from BlockDriverState and enable point-in-time snapshot exporting over NBD, Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 01/11] block: Add BlockOpType enum, Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 02/11] block: Introduce op_blockers to BlockDriverState, Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 03/11] block: Replace in_use with operation blocker, Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 04/11] block: Move op_blocker check from block_job_create to its caller, Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 05/11] block: Add bdrv_set_backing_hd(),
Fam Zheng <=
- [Qemu-devel] [PATCH v10 06/11] block: Add backing_blocker in BlockDriverState, Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 07/11] block: Parse "backing" option to reference existing BDS, Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 08/11] block: Support dropping active in bdrv_drop_intermediate, Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 09/11] stream: Use bdrv_drop_intermediate and drop close_unused_images, Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 10/11] qmp: Add command 'blockdev-backup', Fam Zheng, 2014/01/07
- [Qemu-devel] [PATCH v10 11/11] block: Allow backup on referenced named BlockDriverState, Fam Zheng, 2014/01/07
- Re: [Qemu-devel] [PATCH v10 00/11] Drop in_use from BlockDriverState and enable point-in-time snapshot exporting over NBD, Fam Zheng, 2014/01/08