[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/17] blockdev: Move parsing of 'if' option to
From: |
Benoît Canet |
Subject: |
Re: [Qemu-devel] [PATCH 08/17] blockdev: Move parsing of 'if' option to drive_init |
Date: |
Fri, 20 Sep 2013 16:50:25 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Le Friday 20 Sep 2013 à 13:54:21 (+0200), Kevin Wolf a écrit :
> It's always IF_NONE for blockdev-add.
>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> blockdev.c | 40 ++++++++++++++++++++++------------------
> 1 file changed, 22 insertions(+), 18 deletions(-)
>
> diff --git a/blockdev.c b/blockdev.c
> index 5b25d7f..dc3f01a 100644
> --- a/blockdev.c
> +++ b/blockdev.c
> @@ -306,14 +306,13 @@ 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 type,
> DriveMediaType media)
> {
> const char *buf;
> const char *file = NULL;
> const char *serial;
> const char *mediastr = "";
> - BlockInterfaceType type;
> int bus_id, unit_id;
> int cyls, heads, secs, translation;
> int max_devs;
> @@ -374,17 +373,6 @@ static DriveInfo *blockdev_init(QDict *bs_opts,
> file = qemu_opt_get(opts, "file");
> serial = qemu_opt_get(opts, "serial");
>
> - if ((buf = qemu_opt_get(opts, "if")) != NULL) {
> - for (type = 0; type < IF_COUNT && strcmp(buf, if_name[type]); type++)
> - ;
> - if (type == IF_COUNT) {
> - error_report("unsupported bus type '%s'", buf);
> - return NULL;
> - }
> - } else {
> - type = block_default_type;
> - }
> -
> max_devs = if_max_devs[type];
>
> if (cyls || heads || secs) {
> @@ -753,6 +741,10 @@ QemuOptsList qemu_legacy_drive_opts = {
> .name = "media",
> .type = QEMU_OPT_STRING,
> .help = "media type (disk, cdrom)",
> + },{
> + .name = "if",
> + .type = QEMU_OPT_STRING,
> + .help = "interface (ide, scsi, sd, mtd, floppy, pflash, virtio)",
> },
> { /* end of list */ }
> },
> @@ -765,6 +757,7 @@ DriveInfo *drive_init(QemuOpts *all_opts,
> BlockInterfaceType block_default_type)
> QDict *bs_opts;
> QemuOpts *legacy_opts;
> DriveMediaType media = MEDIA_DISK;
> + BlockInterfaceType type;
> Error *local_err = NULL;
>
> /* Change legacy command line options into QMP ones */
> @@ -839,8 +832,23 @@ DriveInfo *drive_init(QemuOpts *all_opts,
> BlockInterfaceType block_default_type)
> }
> }
>
> + /* Controller type */
> + value = qemu_opt_get(legacy_opts, "if");
> + if (value) {
> + for (type = 0;
> + type < IF_COUNT && strcmp(value, if_name[type]);
> + type++) {
> + }
> + if (type == IF_COUNT) {
> + error_report("unsupported bus type '%s'", value);
> + return NULL;
goto fail; here ?
> + }
> + } else {
> + type = block_default_type;
> + }
> +
> /* Actual block device init: Functionality shared with blockdev-add */
> - dinfo = blockdev_init(bs_opts, block_default_type, media);
> + dinfo = blockdev_init(bs_opts, type, media);
> if (dinfo == NULL) {
> goto fail;
> }
> @@ -2188,10 +2196,6 @@ QemuOptsList qemu_common_drive_opts = {
> .type = QEMU_OPT_NUMBER,
> .help = "unit number (i.e. lun for scsi)",
> },{
> - .name = "if",
> - .type = QEMU_OPT_STRING,
> - .help = "interface (ide, scsi, sd, mtd, floppy, pflash, virtio)",
> - },{
> .name = "index",
> .type = QEMU_OPT_NUMBER,
> .help = "index number",
> --
> 1.8.1.4
>
>
- Re: [Qemu-devel] [PATCH 04/17] blockdev: 'blockdev-add' QMP command, (continued)
- [Qemu-devel] [PATCH 07/17] blockdev: Move parsing of 'media' option to drive_init, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 08/17] blockdev: Move parsing of 'if' option to drive_init, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 09/17] blockdev: Moving parsing of geometry options to drive_init, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 10/17] blockdev: Move parsing of 'boot' option to drive_init, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 11/17] blockdev: Move bus/unit/index processing to drive_init, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 12/17] blockdev: Move virtio-blk device creation to drive_init, Kevin Wolf, 2013/09/20
- [Qemu-devel] [PATCH 13/17] blockdev: Remove IF_* check for read-only blockdev_init, Kevin Wolf, 2013/09/20