[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V26 12/32] change block layer to support both Qe
From: |
Chun Yan Liu |
Subject: |
Re: [Qemu-devel] [PATCH V26 12/32] change block layer to support both QemuOpts and QEMUOptionParamter |
Date: |
Sat, 03 May 2014 21:51:59 -0600 |
>>> On 4/30/2014 at 09:57 AM, in message <address@hidden>, Eric Blake
<address@hidden> wrote:
> On 04/29/2014 03:10 AM, Chunyan Liu wrote:
> > Change block layer to support both QemuOpts and QEMUOptionParameter.
> > After this patch, it will change backend drivers one by one. At the end,
> > QEMUOptionParameter will be removed and only QemuOpts is kept.
> >
> > Signed-off-by: Dong Xu Wang <address@hidden>
> > Signed-off-by: Chunyan Liu <address@hidden>
> > ---
> > Changes to V25:
> > * fix Eric's comments:
> > * update bdrv_create_co_entry and bdrv_amend_options code, to let it
> > more readable.
> > * add assertion in bdrv_register.
> > * improve comments to create_opts in header file.
> >
> > block.c | 158
> ++++++++++++++++++++++++++++++++--------------
> > block/cow.c | 2 +-
> > block/qcow.c | 2 +-
> > block/qcow2.c | 2 +-
> > block/qed.c | 2 +-
> > block/raw_bsd.c | 2 +-
> > block/vhdx.c | 2 +-
> > block/vmdk.c | 4 +-
> > block/vvfat.c | 2 +-
> > include/block/block.h | 7 +-
> > include/block/block_int.h | 13 +++-
> > qemu-img.c | 94 +++++++++++++--------------
> > 12 files changed, 180 insertions(+), 110 deletions(-)
> >
>
> >
> > - if (options) {
> > - param = parse_option_parameters(options, create_options, param);
> > - if (param == NULL) {
> > - error_report("Invalid options for file format '%s'.",
> out_fmt);
> > - ret = -1;
> > - goto out;
> > - }
> > - } else {
> > - param = parse_option_parameters("", create_options, param);
> > + opts = qemu_opts_create(create_opts, NULL, 0, &error_abort);
> > + if (options && qemu_opts_do_parse(opts, options, NULL)) {
> > + error_report("Invalid options for file format '%s'.", out_fmt);
>
> Pre-existing, but error messages shouldn't end in '.' Trivial enough to
> not stop the review.
Didn't modify it since to avoid print info change, let qemu-iotests PASS.
With a simple grep, there are many places in current code where error
messages are ended in '.', could change all in a time maybe.
>
> Yay! I like it! It's looking a lot better with all the fixes you've
> folded in.
>
> Reviewed-by: Eric Blake <address@hidden>
>
> --
> Eric Blake eblake redhat com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>
>