[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v28 00/33] replace QEMUOptionParameter with Qemu
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v28 00/33] replace QEMUOptionParameter with QemuOpts |
Date: |
Wed, 11 Jun 2014 20:00:59 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Thu, Jun 05, 2014 at 05:20:39PM +0800, Chunyan Liu wrote:
> This patch series is to replace QEMUOptionParameter with QemuOpts, so that
> only
> one Qemu Option structure is kept in QEMU code.
>
> ---
> Changes to v27:
> * rebase to git master (code has been updated in this part since v27 sent
> so rebase to make the patch series work.)
>
> All patches are also available from:
> https://github.com/chunyanliu/qemu/commits/QemuOpts
>
>
> Chunyan Liu (33):
> QemuOpts: move find_desc_by_name ahead for later calling
> QemuOpts: repurpose qemu_opts_print to replace
> print_option_parameters
> QemuOpts: add def_value_str to QemuOptDesc
> qapi: output def_value_str when query command line options
> QemuOpts: change opt->name|str from (const char *) to (char *)
> QemuOpts: move qemu_opt_del ahead for later calling
> QemuOpts: add qemu_opt_get_*_del functions for replace work
> QemuOpts: add qemu_opts_print_help to replace print_option_help
> QemuOpts: add conversion between QEMUOptionParameter to QemuOpts
> QemuOpts: add qemu_opts_append to replace append_option_parameters
> QemuOpts: check NULL input for qemu_opts_del
> change block layer to support both QemuOpts and QEMUOptionParamter
> vvfat.c: handle cross_driver's create_options and create_opts
> cow.c: replace QEMUOptionParameter with QemuOpts
> gluster.c: replace QEMUOptionParameter with QemuOpts
> iscsi.c: replace QEMUOptionParameter with QemuOpts
> nfs.c: replace QEMUOptionParameter with QemuOpts
> qcow.c: replace QEMUOptionParameter with QemuOpts
> QemuOpts: export qemu_opt_find
> qcow2.c: replace QEMUOptionParameter with QemuOpts
> qed.c: replace QEMUOptionParameter with QemuOpts
> raw-posix.c: replace QEMUOptionParameter with QemuOpts
> raw-win32.c: replace QEMUOptionParameter with QemuOpts
> raw_bsd.c: replace QEMUOptionParameter with QemuOpts
> rbd.c: replace QEMUOptionParameter with QemuOpts
> sheepdog.c: replace QEMUOptionParameter with QemuOpts
> ssh.c: replace QEMUOptionParameter with QemuOpts
> vdi.c: replace QEMUOptionParameter with QemuOpts
> vhdx.c: replace QEMUOptionParameter with QemuOpts
> vmdk.c: replace QEMUOptionParameter with QemuOpts
> vpc.c: replace QEMUOptionParameter with QemuOpts
> cleanup QEMUOptionParameter
> QemuOpts: cleanup tmp 'allocated' member from QemuOptsList
>
> block.c | 99 ++++----
> block/cow.c | 52 ++--
> block/gluster.c | 73 +++---
> block/iscsi.c | 32 ++-
> block/nfs.c | 10 +-
> block/qcow.c | 72 +++---
> block/qcow2.c | 259 ++++++++++----------
> block/qed.c | 112 +++++----
> block/qed.h | 3 +-
> block/raw-posix.c | 55 ++---
> block/raw-win32.c | 38 +--
> block/raw_bsd.c | 25 +-
> block/rbd.c | 61 +++--
> block/sheepdog.c | 105 ++++----
> block/ssh.c | 30 ++-
> block/vdi.c | 71 +++---
> block/vhdx.c | 97 ++++----
> block/vhdx.h | 1 +
> block/vmdk.c | 121 +++++-----
> block/vpc.c | 60 ++---
> block/vvfat.c | 14 +-
> include/block/block.h | 7 +-
> include/block/block_int.h | 9 +-
> include/qemu/option.h | 53 +---
> include/qemu/option_int.h | 4 +-
> qapi-schema.json | 5 +-
> qapi/opts-visitor.c | 10 +-
> qemu-img.c | 91 ++++---
> qmp-commands.hx | 2 +
> tests/qemu-iotests/049.out | 2 +-
> tests/qemu-iotests/061.out | 2 +-
> util/qemu-config.c | 4 +
> util/qemu-option.c | 590
> ++++++++++++++++++++-------------------------
> 33 files changed, 1037 insertions(+), 1132 deletions(-)
>
> --
> 1.7.12.4
>
Thanks, applied to my block tree:
https://github.com/stefanha/qemu/commits/block
Stefan
pgp2BFD9JP50a.pgp
Description: PGP signature
- [Qemu-devel] [PATCH v28 29/33] vhdx.c: replace QEMUOptionParameter with QemuOpts, (continued)
- [Qemu-devel] [PATCH v28 29/33] vhdx.c: replace QEMUOptionParameter with QemuOpts, Chunyan Liu, 2014/06/05
- [Qemu-devel] [PATCH v28 31/33] vpc.c: replace QEMUOptionParameter with QemuOpts, Chunyan Liu, 2014/06/05
- [Qemu-devel] [PATCH v28 33/33] QemuOpts: cleanup tmp 'allocated' member from QemuOptsList, Chunyan Liu, 2014/06/05
- [Qemu-devel] [PATCH v28 32/33] cleanup QEMUOptionParameter, Chunyan Liu, 2014/06/05
- [Qemu-devel] [PATCH v28 30/33] vmdk.c: replace QEMUOptionParameter with QemuOpts, Chunyan Liu, 2014/06/05
- Re: [Qemu-devel] [PATCH v28 00/33] replace QEMUOptionParameter with QemuOpts, Stefan Hajnoczi, 2014/06/05
- Re: [Qemu-devel] [PATCH v28 00/33] replace QEMUOptionParameter with QemuOpts, Stefan Hajnoczi, 2014/06/09
- Re: [Qemu-devel] [PATCH v28 00/33] replace QEMUOptionParameter with QemuOpts,
Stefan Hajnoczi <=
- Re: [Qemu-devel] [PATCH v28 00/33] replace QEMUOptionParameter with QemuOpts, Peter Lieven, 2014/06/23