[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] qemu-img: Check for backing image if specifi
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2] qemu-img: Check for backing image if specified during create |
Date: |
Tue, 9 May 2017 13:19:23 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
On 05/09/2017 01:17 PM, Eric Blake wrote:
> On 05/09/2017 01:09 PM, John Snow wrote:
>> Or, rather, force the open of a backing image if one was specified
>> for creation. Using a similar -unsafe option as rebase, allow qemu-img
>> to ignore the backing file validation if possible.
>>
>> +++ b/qemu-img.c
>> @@ -145,9 +145,11 @@ static void QEMU_NORETURN help(void)
>> " 'snapshot_id_or_name' is deprecated, use 'snapshot_param'\n"
>> " instead\n"
>> " '-c' indicates that target image must be compressed (qcow
>> format only)\n"
>> - " '-u' enables unsafe rebasing. It is assumed that old and new
>> backing file\n"
>> - " match exactly. The image doesn't need a working backing
>> file before\n"
>> - " rebasing in this case (useful for renaming the backing
>> file)\n"
>> + " '-u' allows unsafe backing chains. For rebasing, it is
>> assumed that old and new\n"
>> + " backing file match exactly. The image doesn't need a
>> working backing file\n"
>> + " before rebasing in this case (useful for renaming the
>> backing file)\n"
Also, a '.' to end this sentence would be nice.
>> + " For image creation, allow creating without attempting to"
>
> Missing \n
>
> With that fixed,
> Reviewed-by: Eric Blake <address@hidden>
>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature