[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH V5 00/10] replace QEMUOptionParameter with QemuOpts
From: |
Dong Xu Wang |
Subject: |
[Qemu-devel] [PATCH V5 00/10] replace QEMUOptionParameter with QemuOpts parser |
Date: |
Thu, 1 Nov 2012 17:12:20 +0800 |
Patch 1-3 are from Luiz, added Markus's comments, discussion could be found
here:
http://lists.nongnu.org/archive/html/qemu-devel/2012-07/msg02716.html
Patch 3 was changed according Paolo's comments.
Patch 4-5: because qemu_opts_create can not fail while id is null, so create
function qemu_opts_create_nofail and use it.
Patch 6: create function qemu_opt_set_number, like qemu_opt_set_bool.
Patch 7: add def_value and use it in qemu_opts_print.
Patch 8: Create functions to pair with QEMUOptionParameter parser.
Patch 9: Use QemuOpts parser in Block.
Patch 10: Remove QEMUOptionParameter parser related code.
v4->v5:
1) Rewrite qemu_opts_create_nofail function based on Peter Maydell's comments.
2) Use g_strdup_printf in qemu_opt_set_number.
3) Rewrite qemu_opts_print.
4) .bdrv_create_options returns pointer directly. Fix a bug about "encryption".
5) Check qemu_opt_get_number in raw-posix.c.
v3->v4:
1) Rebased to the newest source tree.
2) Remove redundant "#include "block-cache.h"
3) Other small changes.
v2->v3:
1) rewrite qemu_opt_set_bool and qemu_opt_set_number according Paolo's coments.
2) split patches to make review easier.
v1->v2:
1) add Luiz's patches.
2) create qemu_opt_set_number() and qemu_opts_create_nofail() functions.
3) add QemuOptsList map to drivers.
4) use original opts parser, not creating new ones.
5) fix other bugs.
Luiz Capitulino (2):
qemu-option: opt_set(): split it up into more functions
qemu-option: qemu_opts_validate(): fix duplicated code
Dong Xu Wang (8):
qemu-option: qemu_opt_set_bool(): fix code duplication
introduce qemu_opts_create_nofail function
use qemu_opts_create_nofail
create new function: qemu_opt_set_number
add def_value and use it in qemu_opts_print.
Create four opts list related functions
Use QemuOpts support in block layer
remove QEMUOptionParameter related functions and struct
block.c | 91 +++++-----
block.h | 8 +-
block/cow.c | 46 +++---
block/qcow.c | 60 +++----
block/qcow2.c | 171 +++++++++---------
block/qed.c | 86 +++++-----
block/raw-posix.c | 65 +++----
block/raw.c | 22 ++-
block/sheepdog.c | 75 ++++----
block/vdi.c | 68 ++++----
block/vmdk.c | 74 ++++----
block/vpc.c | 51 +++---
block/vvfat.c | 11 +-
block_int.h | 6 +-
blockdev.c | 2 +-
hw/watchdog.c | 2 +-
qemu-config.c | 4 +-
qemu-img.c | 63 +++----
qemu-option.c | 512 ++++++++++++++++++-----------------------------------
qemu-option.h | 39 +----
qemu-sockets.c | 16 +-
vl.c | 12 +-
22 files changed, 637 insertions(+), 847 deletions(-)
- [Qemu-devel] [PATCH V5 00/10] replace QEMUOptionParameter with QemuOpts parser,
Dong Xu Wang <=
- [Qemu-devel] [PATCH V5 01/10] qemu-option: opt_set(): split it up into more functions, Dong Xu Wang, 2012/11/01
- [Qemu-devel] [PATCH V5 03/10] qemu-option: qemu_opt_set_bool(): fix code duplication, Dong Xu Wang, 2012/11/01
- [Qemu-devel] [PATCH V5 06/10] create new function: qemu_opt_set_number, Dong Xu Wang, 2012/11/01
- [Qemu-devel] [PATCH V5 04/10] introduce qemu_opts_create_nofail function, Dong Xu Wang, 2012/11/01
- [Qemu-devel] [PATCH V5 05/10] use qemu_opts_create_nofail, Dong Xu Wang, 2012/11/01
- [Qemu-devel] [PATCH V5 07/10] add def_print_str and use it in qemu_opts_print., Dong Xu Wang, 2012/11/01
- [Qemu-devel] [PATCH V5 02/10] qemu-option: qemu_opts_validate(): fix duplicated code, Dong Xu Wang, 2012/11/01
- [Qemu-devel] [PATCH V5 08/10] Create four opts list related functions, Dong Xu Wang, 2012/11/01