qemu-devel
[Top][All Lists]
Advanced

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

[Qemu-devel] [PATCH V17 0/9] replace QEMUOptionParameter with QemuOpts p


From: Dong Xu Wang
Subject: [Qemu-devel] [PATCH V17 0/9] replace QEMUOptionParameter with QemuOpts parser
Date: Wed, 17 Jul 2013 17:29:39 +0800

These patches will replace QEMUOptionParameter with QemuOpts. Change logs
please go to each patch's commit message.

Dong Xu Wang (9):
  qemu-option: add def_value_str in QemuOptDesc struct and rewrite
    qemu_opts_print
  qemu-option: avoid duplication of default value in QemuOpts
  qemu-option: create four QemuOptsList related functions
  qemu-option: create some QemuOpts functons
  block: use QemuOpts support in block layer
  qapi: query-command-line-options outputs def_value_str
  qemu-option: remove QEMUOptionParameter related functions and struct
  qemu-option: make qemu_opts_del accept opts being NULL
  qemu-option: use qemu_opts_del without judging NULL

 block.c                   |  96 ++++----
 block/cow.c               |  54 ++---
 block/gluster.c           |  37 ++-
 block/iscsi.c             |  31 ++-
 block/qcow.c              |  69 +++---
 block/qcow2.c             | 205 +++++++++--------
 block/qed.c               | 114 ++++-----
 block/qed.h               |   2 +-
 block/raw-posix.c         |  49 ++--
 block/raw-win32.c         |  31 +--
 block/raw.c               |  30 +--
 block/rbd.c               |  62 +++--
 block/sheepdog.c          |  81 ++++---
 block/ssh.c               |  29 ++-
 block/vdi.c               |  70 +++---
 block/vmdk.c              | 179 +++++++-------
 block/vpc.c               |  65 +++---
 block/vvfat.c             |  11 +-
 hw/core/qdev.c            |   4 +-
 include/block/block.h     |   5 +-
 include/block/block_int.h |   6 +-
 include/qemu/option.h     |  56 ++---
 qapi-schema.json          |   8 +-
 qemu-char.c               |   4 +-
 qemu-img.c                |  63 +++--
 qmp-commands.hx           |   2 +
 util/qemu-config.c        |   4 +
 util/qemu-option.c        | 576 +++++++++++++++++++++-------------------------
 28 files changed, 930 insertions(+), 1013 deletions(-)

-- 
V17 add qemu_opts_del's support when opts is NULL, and made changes based on
Eric's comments.

I tried to split PATCH 5 into small pieces, but I found it is very hard, bdrv_*
functions are all using QEMUOptionParameter or QemuOpts, if I split them, I have
to support two parsers in block.c and qemu-img.c, it will make code very hard to
read, so I leave them in one big patch, I am sorry it is realy hard to review.

Changelogs please goto each patch.

1.7.11.7




reply via email to

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