[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qemu-img: check bs_n when use old style option
From: |
858585 jemmy |
Subject: |
Re: [Qemu-devel] [PATCH] qemu-img: check bs_n when use old style option |
Date: |
Thu, 20 Apr 2017 15:59:09 +0800 |
On Thu, Apr 20, 2017 at 3:51 PM, Fam Zheng <address@hidden> wrote:
> On Thu, 04/20 12:04, address@hidden wrote:
>> From: Lidong Chen <address@hidden>
>>
>> When use old style option like -o backing_file, img_convert
>> continue run when bs_n > 1, this patch fix this bug.
>>
>> Signed-off-by: Lidong Chen <address@hidden>
>> ---
>> qemu-img.c | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>> diff --git a/qemu-img.c b/qemu-img.c
>> index b220cf7..c673aef 100644
>> --- a/qemu-img.c
>> +++ b/qemu-img.c
>> @@ -2225,6 +2225,13 @@ static int img_convert(int argc, char **argv)
>> out_baseimg = out_baseimg_param;
>> }
>>
>> + if (bs_n > 1 && out_baseimg) {
>> + error_report("-B makes no sense when concatenating multiple input "
>> + "images");
>> + ret = -1;
>> + goto out;
>> + }
>> +
>> /* Check if compression is supported */
>> if (compress) {
>> bool encryption =
>> --
>> 1.8.3.1
>>
>>
>
> Is this essentially the same as the check a few lines above:
>
> ...
> if (bs_n < 1) {
> error_exit("Must specify image file name");
> }
>
>
> if (bs_n > 1 && out_baseimg) {
> error_report("-B makes no sense when concatenating multiple input "
> "images");
> ret = -1;
> goto out;
> }
>
> src_flags = 0;
> ret = bdrv_parse_cache_mode(src_cache, &src_flags, &src_writethrough);
> if (ret < 0) {
> error_report("Invalid source cache option: %s", src_cache);
> goto out;
> }
> ...
>
> How about moving that down?
moving that down is ok.
but will exit later if use -B option.
which way do you think better?
>
> Fam