qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 5/6] qemu-option: Remove qemu_opts_create_nof


From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH v2 5/6] qemu-option: Remove qemu_opts_create_nofail
Date: Thu, 05 Dec 2013 11:11:32 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux)

Peter Crosthwaite <address@hidden> writes:

> This is a boiler-plate _nofail variant of qemu_opts_create. Remove and
> use error_abort in call sites.
>
> null/0 arguments needs to be added for the id and fail_if_exists fields
> in affected callsites due to argument inconsistency between the normal and
> no_fail variants.
>
> Signed-off-by: Peter Crosthwaite <address@hidden>
> ---
> changed since v1:
> Wrap some long lines (Markus review)
> Flip fail_if_exists to 0 in call sites (Markus review)
> Remove spurious whitespace fix (Markus review)
> Mention fail_if_exists in commit msg (Markus review)
>
>  block/blkdebug.c       |  2 +-
>  block/blkverify.c      |  2 +-
>  block/curl.c           |  2 +-
>  block/gluster.c        |  2 +-
>  block/iscsi.c          |  2 +-
>  block/nbd.c            |  3 ++-
>  block/qcow2.c          |  2 +-
>  block/raw-posix.c      |  2 +-
>  block/raw-win32.c      |  5 +++--
>  block/rbd.c            |  2 +-
>  block/sheepdog.c       |  2 +-
>  block/vvfat.c          |  2 +-
>  blockdev.c             |  6 ++++--
>  hw/watchdog/watchdog.c |  3 ++-
>  include/qemu/option.h  |  1 -
>  qdev-monitor.c         |  2 +-
>  qemu-img.c             |  2 +-
>  util/qemu-config.c     |  2 +-
>  util/qemu-option.c     |  9 ---------
>  util/qemu-sockets.c    | 18 +++++++++---------
>  vl.c                   | 15 +++++++++------
>  21 files changed, 42 insertions(+), 44 deletions(-)
>
> diff --git a/block/blkdebug.c b/block/blkdebug.c
> index 16d2b91..7f915ea 100644
> --- a/block/blkdebug.c
> +++ b/block/blkdebug.c
> @@ -359,7 +359,7 @@ static int blkdebug_open(BlockDriverState *bs, QDict 
> *options, int flags,
>      const char *filename, *config;
>      int ret;
>  
> -    opts = qemu_opts_create_nofail(&runtime_opts);
> +    opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          error_propagate(errp, local_err);
> diff --git a/block/blkverify.c b/block/blkverify.c
> index 3c63528..1c1637f 100644
> --- a/block/blkverify.c
> +++ b/block/blkverify.c
> @@ -125,7 +125,7 @@ static int blkverify_open(BlockDriverState *bs, QDict 
> *options, int flags,
>      const char *filename, *raw;
>      int ret;
>  
> -    opts = qemu_opts_create_nofail(&runtime_opts);
> +    opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          error_propagate(errp, local_err);
> diff --git a/block/curl.c b/block/curl.c
> index 5a46f97..a603936 100644
> --- a/block/curl.c
> +++ b/block/curl.c
> @@ -413,7 +413,7 @@ static int curl_open(BlockDriverState *bs, QDict 
> *options, int flags,
>          return -EROFS;
>      }
>  
> -    opts = qemu_opts_create_nofail(&runtime_opts);
> +    opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          qerror_report_err(local_err);
> diff --git a/block/gluster.c b/block/gluster.c
> index 877686a..563d497 100644
> --- a/block/gluster.c
> +++ b/block/gluster.c
> @@ -298,7 +298,7 @@ static int qemu_gluster_open(BlockDriverState *bs,  QDict 
> *options,
>      Error *local_err = NULL;
>      const char *filename;
>  
> -    opts = qemu_opts_create_nofail(&runtime_opts);
> +    opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          qerror_report_err(local_err);
> diff --git a/block/iscsi.c b/block/iscsi.c
> index a2d578c..2cec43a 100644
> --- a/block/iscsi.c
> +++ b/block/iscsi.c
> @@ -1241,7 +1241,7 @@ static int iscsi_open(BlockDriverState *bs, QDict 
> *options, int flags,
>          return -EINVAL;
>      }
>  
> -    opts = qemu_opts_create_nofail(&runtime_opts);
> +    opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          qerror_report_err(local_err);
> diff --git a/block/nbd.c b/block/nbd.c
> index c8deeee..5bd9359 100644
> --- a/block/nbd.c
> +++ b/block/nbd.c
> @@ -234,7 +234,8 @@ static int nbd_config(BDRVNBDState *s, QDict *options)
>          return -EINVAL;
>      }
>  
> -    s->socket_opts = qemu_opts_create_nofail(&socket_optslist);
> +    s->socket_opts = qemu_opts_create(&socket_optslist, NULL, 0,
> +                                      &error_abort);
>  
>      qemu_opts_absorb_qdict(s->socket_opts, options, &local_err);
>      if (error_is_set(&local_err)) {
> diff --git a/block/qcow2.c b/block/qcow2.c
> index 6e5d98d..cf28377 100644
> --- a/block/qcow2.c
> +++ b/block/qcow2.c
> @@ -669,7 +669,7 @@ static int qcow2_open(BlockDriverState *bs, QDict 
> *options, int flags,
>      }
>  
>      /* Enable lazy_refcounts according to image and command line options */
> -    opts = qemu_opts_create_nofail(&qcow2_runtime_opts);
> +    opts = qemu_opts_create(&qcow2_runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          error_propagate(errp, local_err);
> diff --git a/block/raw-posix.c b/block/raw-posix.c
> index f836c8e..dce7c27 100644
> --- a/block/raw-posix.c
> +++ b/block/raw-posix.c
> @@ -284,7 +284,7 @@ static int raw_open_common(BlockDriverState *bs, QDict 
> *options,
>      const char *filename;
>      int fd, ret;
>  
> -    opts = qemu_opts_create_nofail(&raw_runtime_opts);
> +    opts = qemu_opts_create(&raw_runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          error_propagate(errp, local_err);
> diff --git a/block/raw-win32.c b/block/raw-win32.c
> index 2bad5a3..ce314fd 100644
> --- a/block/raw-win32.c
> +++ b/block/raw-win32.c
> @@ -248,7 +248,7 @@ static int raw_open(BlockDriverState *bs, QDict *options, 
> int flags,
>  
>      s->type = FTYPE_FILE;
>  
> -    opts = qemu_opts_create_nofail(&raw_runtime_opts);
> +    opts = qemu_opts_create(&raw_runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          error_propagate(errp, local_err);
> @@ -550,7 +550,8 @@ static int hdev_open(BlockDriverState *bs, QDict 
> *options, int flags,
>      Error *local_err = NULL;
>      const char *filename;
>  
> -    QemuOpts *opts = qemu_opts_create_nofail(&raw_runtime_opts);
> +    QemuOpts *opts = qemu_opts_create(&raw_runtime_opts, NULL, 0,
> +                                      &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          error_propagate(errp, local_err);
> diff --git a/block/rbd.c b/block/rbd.c
> index 4a1ea5b..f453f04 100644
> --- a/block/rbd.c
> +++ b/block/rbd.c
> @@ -461,7 +461,7 @@ static int qemu_rbd_open(BlockDriverState *bs, QDict 
> *options, int flags,
>      const char *filename;
>      int r;
>  
> -    opts = qemu_opts_create_nofail(&runtime_opts);
> +    opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          qerror_report_err(local_err);
> diff --git a/block/sheepdog.c b/block/sheepdog.c
> index ef387de..59d92a9 100644
> --- a/block/sheepdog.c
> +++ b/block/sheepdog.c
> @@ -1375,7 +1375,7 @@ static int sd_open(BlockDriverState *bs, QDict 
> *options, int flags,
>  
>      s->bs = bs;
>  
> -    opts = qemu_opts_create_nofail(&runtime_opts);
> +    opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          qerror_report_err(local_err);
> diff --git a/block/vvfat.c b/block/vvfat.c
> index 3ddaa0b..f67d8ae 100644
> --- a/block/vvfat.c
> +++ b/block/vvfat.c
> @@ -1085,7 +1085,7 @@ DLOG(if (stderr == NULL) {
>      setbuf(stderr, NULL);
>  })
>  
> -    opts = qemu_opts_create_nofail(&runtime_opts);
> +    opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
>      qemu_opts_absorb_qdict(opts, options, &local_err);
>      if (error_is_set(&local_err)) {
>          qerror_report_err(local_err);
> diff --git a/blockdev.c b/blockdev.c
> index 330aa4a..78e8455 100644
> --- a/blockdev.c
> +++ b/blockdev.c
> @@ -682,7 +682,8 @@ 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);
> +    legacy_opts = qemu_opts_create(&qemu_legacy_drive_opts, NULL, 1,
> +                                   &error_abort);
>      qemu_opts_absorb_qdict(legacy_opts, bs_opts, &local_err);
>      if (error_is_set(&local_err)) {
>          qerror_report_err(local_err);

I'm afraid this one still changed fail_if_exists.

> @@ -853,7 +854,8 @@ DriveInfo *drive_init(QemuOpts *all_opts, 
> BlockInterfaceType block_default_type)
>  
>      if (type == IF_VIRTIO) {
>          QemuOpts *devopts;
> -        devopts = qemu_opts_create_nofail(qemu_find_opts("device"));
> +        devopts = qemu_opts_create(qemu_find_opts("device"), NULL, 1,
> +                                   &error_abort);
>          if (arch_type == QEMU_ARCH_S390X) {
>              qemu_opt_set(devopts, "driver", "virtio-blk-s390");
>          } else {

Again.

> diff --git a/hw/watchdog/watchdog.c b/hw/watchdog/watchdog.c
> index 387962e..0ab2a0e 100644
> --- a/hw/watchdog/watchdog.c
> +++ b/hw/watchdog/watchdog.c
> @@ -66,7 +66,8 @@ int select_watchdog(const char *p)
>      QLIST_FOREACH(model, &watchdog_list, entry) {
>          if (strcasecmp(model->wdt_name, p) == 0) {
>              /* add the device */
> -            opts = qemu_opts_create_nofail(qemu_find_opts("device"));
> +            opts = qemu_opts_create(qemu_find_opts("device"), NULL, 1,
> +                                    &error_abort);
>              qemu_opt_set(opts, "driver", p);
>              return 0;
>          }

Again.

> diff --git a/include/qemu/option.h b/include/qemu/option.h
> index 5c0c6dd..3ea871a 100644
> --- a/include/qemu/option.h
> +++ b/include/qemu/option.h
> @@ -136,7 +136,6 @@ int qemu_opt_foreach(QemuOpts *opts, qemu_opt_loopfunc 
> func, void *opaque,
>  QemuOpts *qemu_opts_find(QemuOptsList *list, const char *id);
>  QemuOpts *qemu_opts_create(QemuOptsList *list, const char *id,
>                             int fail_if_exists, Error **errp);
> -QemuOpts *qemu_opts_create_nofail(QemuOptsList *list);
>  void qemu_opts_reset(QemuOptsList *list);
>  void qemu_opts_loc_restore(QemuOpts *opts);
>  int qemu_opts_set(QemuOptsList *list, const char *id,
> diff --git a/qdev-monitor.c b/qdev-monitor.c
> index dc37a43..6280771 100644
> --- a/qdev-monitor.c
> +++ b/qdev-monitor.c
> @@ -730,7 +730,7 @@ int qemu_global_option(const char *str)
>          return -1;
>      }
>  
> -    opts = qemu_opts_create_nofail(&qemu_global_opts);
> +    opts = qemu_opts_create(&qemu_global_opts, NULL, 0, &error_abort);
>      qemu_opt_set(opts, "driver", driver);
>      qemu_opt_set(opts, "property", property);
>      qemu_opt_set(opts, "value", str+offset+1);
> diff --git a/qemu-img.c b/qemu-img.c
> index b6b5644..f414164 100644
> --- a/qemu-img.c
> +++ b/qemu-img.c
> @@ -2477,7 +2477,7 @@ static int img_resize(int argc, char **argv)
>      }
>  
>      /* Parse size */
> -    param = qemu_opts_create_nofail(&resize_options);
> +    param = qemu_opts_create(&resize_options, NULL, 0, &error_abort);
>      if (qemu_opt_set(param, BLOCK_OPT_SIZE, size)) {
>          /* Error message already printed when size parsing fails */
>          ret = -1;
> diff --git a/util/qemu-config.c b/util/qemu-config.c
> index 04da942..7973659 100644
> --- a/util/qemu-config.c
> +++ b/util/qemu-config.c
> @@ -311,7 +311,7 @@ int qemu_config_parse(FILE *fp, QemuOptsList **lists, 
> const char *fname)
>                  error_free(local_err);
>                  goto out;
>              }
> -            opts = qemu_opts_create_nofail(list);
> +            opts = qemu_opts_create(list, NULL, 0, &error_abort);
>              continue;
>          }
>          if (sscanf(line, " %63s = \"%1023[^\"]\"", arg, value) == 2) {
> diff --git a/util/qemu-option.c b/util/qemu-option.c
> index efcb5dc..668e5d9 100644
> --- a/util/qemu-option.c
> +++ b/util/qemu-option.c
> @@ -791,15 +791,6 @@ QemuOpts *qemu_opts_create(QemuOptsList *list, const 
> char *id,
>      return opts;
>  }
>  
> -QemuOpts *qemu_opts_create_nofail(QemuOptsList *list)
> -{
> -    QemuOpts *opts;
> -    Error *errp = NULL;
> -    opts = qemu_opts_create(list, NULL, 0, &errp);
> -    assert_no_error(errp);
> -    return opts;
> -}
> -
>  void qemu_opts_reset(QemuOptsList *list)
>  {
>      QemuOpts *opts, *next_opts;
> diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
> index 6b97dc1..8818d7c 100644
> --- a/util/qemu-sockets.c
> +++ b/util/qemu-sockets.c
> @@ -578,7 +578,7 @@ int inet_listen(const char *str, char *ostr, int olen,
>  
>      addr = inet_parse(str, errp);
>      if (addr != NULL) {
> -        opts = qemu_opts_create_nofail(&socket_optslist);
> +        opts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
>          inet_addr_to_opts(opts, addr);
>          qapi_free_InetSocketAddress(addr);
>          sock = inet_listen_opts(opts, port_offset, errp);
> @@ -617,7 +617,7 @@ int inet_connect(const char *str, Error **errp)
>  
>      addr = inet_parse(str, errp);
>      if (addr != NULL) {
> -        opts = qemu_opts_create_nofail(&socket_optslist);
> +        opts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
>          inet_addr_to_opts(opts, addr);
>          qapi_free_InetSocketAddress(addr);
>          sock = inet_connect_opts(opts, errp, NULL, NULL);
> @@ -651,7 +651,7 @@ int inet_nonblocking_connect(const char *str,
>  
>      addr = inet_parse(str, errp);
>      if (addr != NULL) {
> -        opts = qemu_opts_create_nofail(&socket_optslist);
> +        opts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
>          inet_addr_to_opts(opts, addr);
>          qapi_free_InetSocketAddress(addr);
>          sock = inet_connect_opts(opts, errp, callback, opaque);
> @@ -794,7 +794,7 @@ int unix_listen(const char *str, char *ostr, int olen, 
> Error **errp)
>      char *path, *optstr;
>      int sock, len;
>  
> -    opts = qemu_opts_create_nofail(&socket_optslist);
> +    opts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
>  
>      optstr = strchr(str, ',');
>      if (optstr) {
> @@ -822,7 +822,7 @@ int unix_connect(const char *path, Error **errp)
>      QemuOpts *opts;
>      int sock;
>  
> -    opts = qemu_opts_create_nofail(&socket_optslist);
> +    opts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
>      qemu_opt_set(opts, "path", path);
>      sock = unix_connect_opts(opts, errp, NULL, NULL);
>      qemu_opts_del(opts);
> @@ -839,7 +839,7 @@ int unix_nonblocking_connect(const char *path,
>  
>      g_assert(callback != NULL);
>  
> -    opts = qemu_opts_create_nofail(&socket_optslist);
> +    opts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
>      qemu_opt_set(opts, "path", path);
>      sock = unix_connect_opts(opts, errp, callback, opaque);
>      qemu_opts_del(opts);
> @@ -889,7 +889,7 @@ int socket_connect(SocketAddress *addr, Error **errp,
>      QemuOpts *opts;
>      int fd;
>  
> -    opts = qemu_opts_create_nofail(&socket_optslist);
> +    opts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
>      switch (addr->kind) {
>      case SOCKET_ADDRESS_KIND_INET:
>          inet_addr_to_opts(opts, addr->inet);
> @@ -921,7 +921,7 @@ int socket_listen(SocketAddress *addr, Error **errp)
>      QemuOpts *opts;
>      int fd;
>  
> -    opts = qemu_opts_create_nofail(&socket_optslist);
> +    opts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
>      switch (addr->kind) {
>      case SOCKET_ADDRESS_KIND_INET:
>          inet_addr_to_opts(opts, addr->inet);
> @@ -949,7 +949,7 @@ int socket_dgram(SocketAddress *remote, SocketAddress 
> *local, Error **errp)
>      QemuOpts *opts;
>      int fd;
>  
> -    opts = qemu_opts_create_nofail(&socket_optslist);
> +    opts = qemu_opts_create(&socket_optslist, NULL, 0, &error_abort);
>      switch (remote->kind) {
>      case SOCKET_ADDRESS_KIND_INET:
>          qemu_opt_set(opts, "host", remote->inet->host);
> diff --git a/vl.c b/vl.c
> index 8d5d874..a5ee155 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -543,7 +543,7 @@ QemuOpts *qemu_get_machine_opts(void)
>      assert(list);
>      opts = qemu_opts_find(list, NULL);
>      if (!opts) {
> -        opts = qemu_opts_create_nofail(list);
> +        opts = qemu_opts_create(list, NULL, 0, &error_abort);
>      }
>      return opts;
>  }
> @@ -2252,7 +2252,8 @@ static int balloon_parse(const char *arg)
>                  return  -1;
>          } else {
>              /* create empty opts */
> -            opts = qemu_opts_create_nofail(qemu_find_opts("device"));
> +            opts = qemu_opts_create(qemu_find_opts("device"), NULL, 1,
> +                                    &error_abort);
>          }
>          qemu_opt_set(opts, "driver", "virtio-balloon");
>          return 0;

Again.

> @@ -2512,14 +2513,14 @@ static int virtcon_parse(const char *devname)
>          exit(1);
>      }
>  
> -    bus_opts = qemu_opts_create_nofail(device);
> +    bus_opts = qemu_opts_create(device, NULL, 0, &error_abort);
>      if (arch_type == QEMU_ARCH_S390X) {
>          qemu_opt_set(bus_opts, "driver", "virtio-serial-s390");
>      } else {
>          qemu_opt_set(bus_opts, "driver", "virtio-serial-pci");
>      }
>  
> -    dev_opts = qemu_opts_create_nofail(device);
> +    dev_opts = qemu_opts_create(device, NULL, 0, &error_abort);
>      qemu_opt_set(dev_opts, "driver", "virtconsole");
>  
>      snprintf(label, sizeof(label), "virtcon%d", index);
> @@ -3377,7 +3378,8 @@ int main(int argc, char **argv, char **envp)
>  
>                  qemu_opt_set_bool(fsdev, "readonly",
>                                  qemu_opt_get_bool(opts, "readonly", 0));
> -                device = qemu_opts_create_nofail(qemu_find_opts("device"));
> +                device = qemu_opts_create(qemu_find_opts("device"), NULL, 1,
> +                                          &error_abort);
>                  qemu_opt_set(device, "driver", "virtio-9p-pci");
>                  qemu_opt_set(device, "fsdev",
>                               qemu_opt_get(opts, "mount_tag"));

Again.

> @@ -3397,7 +3399,8 @@ int main(int argc, char **argv, char **envp)
>                  }
>                  qemu_opt_set(fsdev, "fsdriver", "synth");
>  
> -                device = qemu_opts_create_nofail(qemu_find_opts("device"));
> +                device = qemu_opts_create(qemu_find_opts("device"), NULL, 1,
> +                                          &error_abort);
>                  qemu_opt_set(device, "driver", "virtio-9p-pci");
>                  qemu_opt_set(device, "fsdev", "v_synth");
>                  qemu_opt_set(device, "mount_tag", "v_synth");

Again.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]