[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v5 15/27] qemu-img: Update documentation of "-L"
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH v5 15/27] qemu-img: Update documentation of "-L" option |
Date: |
Tue, 24 May 2016 20:09:16 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.0 |
On 17.05.2016 09:35, Fam Zheng wrote:
> Signed-off-by: Fam Zheng <address@hidden>
> ---
> qemu-img-cmds.hx | 44 ++++++++++++++++++++++----------------------
> qemu-img.c | 1 +
> qemu-img.texi | 3 +++
> 3 files changed, 26 insertions(+), 22 deletions(-)
>
[...]
> diff --git a/qemu-img.texi b/qemu-img.texi
> index afaebdd..b91f2e1 100644
> --- a/qemu-img.texi
> +++ b/qemu-img.texi
> @@ -77,6 +77,9 @@ progress is reported when the process receives a
> @code{SIGUSR1} signal.
> @item -q
> Quiet mode - do not print any output (except errors). There's no progress bar
> in case both @var{-q} and @var{-p} options are used.
> address@hidden -L
> +disables image locking. The image will not be locked, other processes can
> +access and lock this image while we are using it.
I think "while it is in use by qemu-img" sounds more program-y.
Other than that, the patch is OK unless you decide that indeed two
distinct flags for source and target make sense for convert and rebase.
Max
> @item -S @var{size}
> indicates the consecutive number of bytes that must contain only zeros
> for qemu-img to create a sparse image during conversion. This value is
> rounded
>
signature.asc
Description: OpenPGP digital signature
- [Qemu-block] [PATCH v5 11/27] raw-posix: Implement .bdrv_lockf, (continued)
- [Qemu-block] [PATCH v5 11/27] raw-posix: Implement .bdrv_lockf, Fam Zheng, 2016/05/17
- [Qemu-block] [PATCH v5 09/27] osdep: Introduce qemu_dup, Fam Zheng, 2016/05/17
- [Qemu-block] [PATCH v5 10/27] raw-posix: Use qemu_dup, Fam Zheng, 2016/05/17
- [Qemu-block] [PATCH v5 12/27] gluster: Implement .bdrv_lockf, Fam Zheng, 2016/05/17
- [Qemu-block] [PATCH v5 13/27] qemu-io: Add "-L" option for BDRV_O_NO_LOCK, Fam Zheng, 2016/05/17
- [Qemu-block] [PATCH v5 15/27] qemu-img: Update documentation of "-L" option, Fam Zheng, 2016/05/17
- Re: [Qemu-block] [PATCH v5 15/27] qemu-img: Update documentation of "-L" option,
Max Reitz <=
- [Qemu-block] [PATCH v5 16/27] qemu-nbd: Add "--no-lock/-L" option, Fam Zheng, 2016/05/17
- [Qemu-block] [PATCH v5 14/27] qemu-img: Add "-L" option to sub commands, Fam Zheng, 2016/05/17
- [Qemu-block] [PATCH v5 18/27] mirror: Disable image locking on target backing chain, Fam Zheng, 2016/05/17
- [Qemu-block] [PATCH v5 17/27] block: Don't lock drive-backup target image in none mode, Fam Zheng, 2016/05/17
- [Qemu-block] [PATCH v5 19/27] qemu-iotests: 140: Disable image lock for qemu-io access, Fam Zheng, 2016/05/17