From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:51543) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UCnpv-0004mC-LU for qemu-devel@nongnu.org; Tue, 05 Mar 2013 04:07:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UCnpt-0000Pg-3f for qemu-devel@nongnu.org; Tue, 05 Mar 2013 04:07:03 -0500 Received: from mx1.redhat.com ([209.132.183.28]:20028) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UCnps-0000PV-RA for qemu-devel@nongnu.org; Tue, 05 Mar 2013 04:07:01 -0500 Date: Tue, 5 Mar 2013 10:06:12 +0100 From: Stefan Hajnoczi Message-ID: <20130305090612.GC1017@stefanha-thinkpad.redhat.com> References: <1362043236-4635-1-git-send-email-wdongxu@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1362043236-4635-1-git-send-email-wdongxu@linux.vnet.ibm.com> Subject: Re: [Qemu-devel] [PATCH V12 0/4] replace QEMUOptionParameter with QemuOpts parser List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Dong Xu Wang Cc: kwolf@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com On Thu, Feb 28, 2013 at 05:20:32PM +0800, Dong Xu Wang wrote: > Patch 1 add def_value_str and use it in qemu_opts_print. > > Patch 2 Create functions to pair with QemuOpts parser. > > Patch 3 Use QemuOpts parser in Block. > > Patch 4 Remove QEMUOptionParameter parser related code. > > > > Dong Xu Wang (4): > add def_value_str and use it in qemu_opts_print > Create four QemuOptsList related functions > Use QemuOpts support in block layer > remove QEMUOptionParameter related code > > block.c | 91 ++++----- > block/cow.c | 46 +++-- > block/gluster.c | 37 ++-- > block/iscsi.c | 8 +- > block/qcow.c | 61 +++--- > block/qcow2.c | 173 ++++++++-------- > block/qed.c | 86 ++++---- > block/qed.h | 2 +- > block/raw-posix.c | 59 +++--- > block/raw-win32.c | 31 +-- > block/raw.c | 30 +-- > block/rbd.c | 62 +++--- > block/sheepdog.c | 75 ++++--- > block/vdi.c | 70 ++++--- > block/vmdk.c | 90 +++++---- > block/vpc.c | 57 +++--- > block/vvfat.c | 11 +- > include/block/block.h | 4 +- > include/block/block_int.h | 6 +- > include/qemu/option.h | 46 ++--- > qemu-img.c | 61 +++--- > util/qemu-option.c | 490 ++++++++++++++++++---------------------------- > 22 files changed, 723 insertions(+), 873 deletions(-) > > -- > 1.7.11.7 > Looks fine at the high level. Markus had specific comments in the last version, please wait for his review before applying. Reviewed-by: Stefan Hajnoczi