[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v3 07/17] blockdev: Move parsing of 'media' option t
From: |
Kevin Wolf |
Subject: |
[Qemu-devel] [PATCH v3 07/17] blockdev: Move parsing of 'media' option to drive_init |
Date: |
Tue, 8 Oct 2013 14:16:24 +0200 |
This moves as much as possible of the processing of the 'media' option
to drive_init so that it can only be accessed using legacy functions,
but never with anything blockdev-add related.
Signed-off-by: Kevin Wolf <address@hidden>
Reviewed-by: Benoit Canet <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
---
blockdev.c | 73 ++++++++++++++++++++++++++++++++++++++++++--------------------
1 file changed, 50 insertions(+), 23 deletions(-)
diff --git a/blockdev.c b/blockdev.c
index cbe907b..0eaaffa 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -305,16 +305,18 @@ static bool check_throttle_config(ThrottleConfig *cfg,
Error **errp)
return true;
}
+typedef enum { MEDIA_DISK, MEDIA_CDROM } DriveMediaType;
+
/* Takes the ownership of bs_opts */
static DriveInfo *blockdev_init(QDict *bs_opts,
- BlockInterfaceType block_default_type)
+ BlockInterfaceType block_default_type,
+ DriveMediaType media)
{
const char *buf;
const char *file = NULL;
const char *serial;
const char *mediastr = "";
BlockInterfaceType type;
- enum { MEDIA_DISK, MEDIA_CDROM } media;
int bus_id, unit_id;
int cyls, heads, secs, translation;
int max_devs;
@@ -335,7 +337,6 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
BlockDriver *drv = NULL;
translation = BIOS_ATA_TRANSLATION_AUTO;
- media = MEDIA_DISK;
/* Check common options by copying from bs_opts to opts, all other options
* stay in bs_opts for processing by bdrv_open(). */
@@ -422,19 +423,11 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
}
}
- if ((buf = qemu_opt_get(opts, "media")) != NULL) {
- if (!strcmp(buf, "disk")) {
- media = MEDIA_DISK;
- } else if (!strcmp(buf, "cdrom")) {
- if (cyls || secs || heads) {
- error_report("CHS can't be set with media=%s", buf);
- return NULL;
- }
- media = MEDIA_CDROM;
- } else {
- error_report("'%s' invalid media", buf);
- return NULL;
- }
+ if (media == MEDIA_CDROM) {
+ if (cyls || secs || heads) {
+ error_report("CHS can't be set with media=cdrom");
+ return NULL;
+ }
}
if ((buf = qemu_opt_get(opts, "discard")) != NULL) {
@@ -755,11 +748,27 @@ static void qemu_opt_rename(QemuOpts *opts, const char
*from, const char *to)
}
}
+QemuOptsList qemu_legacy_drive_opts = {
+ .name = "drive",
+ .head = QTAILQ_HEAD_INITIALIZER(qemu_legacy_drive_opts.head),
+ .desc = {
+ {
+ .name = "media",
+ .type = QEMU_OPT_STRING,
+ .help = "media type (disk, cdrom)",
+ },
+ { /* end of list */ }
+ },
+};
+
DriveInfo *drive_init(QemuOpts *all_opts, BlockInterfaceType
block_default_type)
{
const char *value;
- DriveInfo *dinfo;
+ DriveInfo *dinfo = NULL;
QDict *bs_opts;
+ QemuOpts *legacy_opts;
+ DriveMediaType media = MEDIA_DISK;
+ Error *local_err = NULL;
/* Change legacy command line options into QMP ones */
qemu_opt_rename(all_opts, "iops", "throttling.iops-total");
@@ -812,8 +821,29 @@ DriveInfo *drive_init(QemuOpts *all_opts,
BlockInterfaceType block_default_type)
bs_opts = qdict_new();
qemu_opts_to_qdict(all_opts, bs_opts);
+ legacy_opts = qemu_opts_create_nofail(&qemu_legacy_drive_opts);
+ qemu_opts_absorb_qdict(legacy_opts, bs_opts, &local_err);
+ if (error_is_set(&local_err)) {
+ qerror_report_err(local_err);
+ error_free(local_err);
+ goto fail;
+ }
+
+ /* Media type */
+ value = qemu_opt_get(legacy_opts, "media");
+ if (value) {
+ if (!strcmp(value, "disk")) {
+ media = MEDIA_DISK;
+ } else if (!strcmp(value, "cdrom")) {
+ media = MEDIA_CDROM;
+ } else {
+ error_report("'%s' invalid media", value);
+ goto fail;
+ }
+ }
+
/* Actual block device init: Functionality shared with blockdev-add */
- dinfo = blockdev_init(bs_opts, block_default_type);
+ dinfo = blockdev_init(bs_opts, block_default_type, media);
if (dinfo == NULL) {
goto fail;
}
@@ -823,6 +853,7 @@ DriveInfo *drive_init(QemuOpts *all_opts,
BlockInterfaceType block_default_type)
dinfo->opts = all_opts;
fail:
+ qemu_opts_del(legacy_opts);
return dinfo;
}
@@ -2115,7 +2146,7 @@ void qmp_blockdev_add(BlockdevOptions *options, Error
**errp)
qdict_flatten(qdict);
- dinfo = blockdev_init(qdict, IF_NONE);
+ dinfo = blockdev_init(qdict, IF_NONE, MEDIA_DISK);
if (!dinfo) {
error_setg(errp, "Could not open image");
goto fail;
@@ -2184,10 +2215,6 @@ QemuOptsList qemu_common_drive_opts = {
.type = QEMU_OPT_STRING,
.help = "chs translation (auto, lba. none)",
},{
- .name = "media",
- .type = QEMU_OPT_STRING,
- .help = "media type (disk, cdrom)",
- },{
.name = "snapshot",
.type = QEMU_OPT_BOOL,
.help = "enable/disable snapshot mode",
--
1.8.1.4
- [Qemu-devel] [PATCH v3 00/17] blockdev-add QMP command, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 01/17] qapi-types/visit.py: Pass whole expr dict for structs, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 02/17] qapi-types/visit.py: Inheritance for structs, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 03/17] blockdev: Introduce DriveInfo.enable_auto_del, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 06/17] blockdev: Pass QDict to blockdev_init(), Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 04/17] blockdev: 'blockdev-add' QMP command, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 05/17] blockdev: Separate ID generation from DriveInfo creation, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 07/17] blockdev: Move parsing of 'media' option to drive_init,
Kevin Wolf <=
- [Qemu-devel] [PATCH v3 08/17] blockdev: Move parsing of 'if' option to drive_init, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 09/17] blockdev: Moving parsing of geometry options to drive_init, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 10/17] blockdev: Move parsing of 'boot' option to drive_init, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 11/17] blockdev: Move bus/unit/index processing to drive_init, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 12/17] blockdev: Move virtio-blk device creation to drive_init, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 13/17] blockdev: Remove IF_* check for read-only blockdev_init, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 14/17] qemu-iotests: Check autodel behaviour for device_del, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 15/17] blockdev: Remove 'media' parameter from blockdev_init(), Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 16/17] blockdev: Don't disable COR automatically with blockdev-add, Kevin Wolf, 2013/10/08
- [Qemu-devel] [PATCH v3 17/17] blockdev: blockdev_init() error conversion, Kevin Wolf, 2013/10/08