Re: [Qemu-devel] [PATCH 08/17] blockdev: Move parsing of 'if' option to drive_init

["Benoît Canet" <benoit.canet@irqsave.net>]

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 <kwolf@redhat.com>
> ---
>  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
> 
>