From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:36060) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TNNfh-0004ek-SE for qemu-devel@nongnu.org; Sun, 14 Oct 2012 08:51:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TNNfg-0003XC-H3 for qemu-devel@nongnu.org; Sun, 14 Oct 2012 08:51:57 -0400 Received: from mail-pb0-f45.google.com ([209.85.160.45]:52257) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TNNfg-0003X4-BC for qemu-devel@nongnu.org; Sun, 14 Oct 2012 08:51:56 -0400 Received: by mail-pb0-f45.google.com with SMTP id rp2so4084857pbb.4 for ; Sun, 14 Oct 2012 05:51:55 -0700 (PDT) Sender: Dong Xu Wang From: Dong Xu Wang Date: Sun, 14 Oct 2012 20:51:25 +0800 Message-Id: <1350219095-3378-1-git-send-email-wdongxu@linux.vnet.ibm.com> Subject: [Qemu-devel] [PATCH V3 00/10] replace QEMUOptionParameter with QemuOpts parser List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, Dong Xu Wang 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 2 and 3 are 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. 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. Dong Xu Wang (10): qemu-option: opt_set(): split it up into more functions qemu-option: qemu_opts_validate(): fix duplicated code 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 | 5 +- block/cow.c | 51 +++--- block/qcow.c | 64 ++++--- block/qcow2.c | 177 +++++++++--------- block/qed.c | 90 +++++----- block/raw-posix.c | 68 ++++---- block/raw.c | 31 ++- block/sheepdog.c | 80 ++++---- block/vdi.c | 73 ++++---- block/vmdk.c | 79 ++++---- block/vpc.c | 56 ++++--- block/vvfat.c | 11 +- block_int.h | 6 +- blockdev.c | 2 +- qemu-config.c | 4 +- qemu-img.c | 63 +++---- qemu-option.c | 528 +++++++++++++++++++---------------------------------- qemu-option.h | 40 +---- qemu-sockets.c | 10 +- vl.c | 12 +- 21 files changed, 693 insertions(+), 848 deletions(-)