[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
- [Qemu-devel] [PATCH V17 0/9] replace QEMUOptionParameter with QemuOpts parser,
Dong Xu Wang <=
- [Qemu-devel] [PATCH V17 1/9] qemu-option: add def_value_str in QemuOptDesc struct and rewrite qemu_opts_print, Dong Xu Wang, 2013/07/17
- [Qemu-devel] [PATCH V17 3/9] qemu-option: create four QemuOptsList related functions, Dong Xu Wang, 2013/07/17
- [Qemu-devel] [PATCH V17 9/9] qemu-option: use qemu_opts_del without judging NULL, Dong Xu Wang, 2013/07/17
- [Qemu-devel] [PATCH V17 4/9] qemu-option: create some QemuOpts functons, Dong Xu Wang, 2013/07/17
- [Qemu-devel] [PATCH V17 7/9] qemu-option: remove QEMUOptionParameter related functions and struct, Dong Xu Wang, 2013/07/17
- [Qemu-devel] [PATCH V17 8/9] qemu-option: make qemu_opts_del accept opts being NULL, Dong Xu Wang, 2013/07/17
- [Qemu-devel] [PATCH V17 5/9] block: use QemuOpts support in block layer, Dong Xu Wang, 2013/07/17