[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH 2/2] qemu-nbd: Minor texi updates
From: |
Eric Blake |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH 2/2] qemu-nbd: Minor texi updates |
Date: |
Mon, 4 Jan 2016 11:16:03 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 |
On 12/30/2015 12:57 PM, Sitsofe Wheeler wrote:
> - Change some spacing.
> - Remove duplicate entry for --format.
> - Reword --discard documentation.
> - Add --detect-zeroes documentation.
>
> Signed-off-by: Sitsofe Wheeler <address@hidden>
> ---
> qemu-nbd.texi | 22 ++++++++++++++--------
> 1 file changed, 14 insertions(+), 8 deletions(-)
>
> @@ -22,8 +22,9 @@ offset into the image
> interface to bind to (default @samp{0.0.0.0})
> @item -k, address@hidden
> Use a unix socket with path @var{path}
> address@hidden -f, address@hidden
> -Set image format as @var{format}
> address@hidden -f, address@hidden
> +force the use of the block driver for format @var{fmt} instead of
> +auto-detecting
Why are you abbreviating 'format' to 'fmt'? Oh, because that's how the
other duplicate one was worded.
Should probably start with 'Force', to match the capitalization used
earlier....
> @item -r, --read-only
> export read-only
> @item -P, address@hidden
> @@ -44,17 +45,22 @@ the emulator's @code{-drive cache=...} option for allowed
> values.
> choose asynchronous I/O mode between @samp{threads} (the default)
> and @samp{native} (Linux only).
> @item address@hidden
> -toggles whether @dfn{discard} (also known as @dfn{trim} or @dfn{unmap})
...then again, later lines are not starting with upper case. Maybe
that's worth its own patch to make it uniformly consistent?
> -requests are ignored or passed to the filesystem. The default is no
> -(@samp{--discard=ignore}).
> +controls whether @dfn{discard} (also known as @dfn{trim} or @dfn{unmap})
> +requests are ignored or passed to the filesystem. @var{discard} is one of
> address@hidden (or @samp{off}), @samp{unmap} (or @samp{on}). The default is
> address@hidden
> address@hidden address@hidden
> +enables the automatic conversion of plain zero writes by the OS to driver
> +specific optimized zero write commands. @var{detect-zeroes} is one of
I'd probably write this one as 'driver-specific'
> address@hidden, @samp{on} or @samp{unmap}. @samp{unmap}
> +converts a zero write to an unmap operation and can only be used if
> address@hidden is set to @samp{unmap}. The default is @samp{off}.
> @item -c, address@hidden
> connect @var{filename} to NBD device @var{dev}
> @item -d, --disconnect
> disconnect the specified device
> @item -e, address@hidden
> device can be shared by @var{num} clients (default @samp{1})
> address@hidden -f, address@hidden
> -force block driver for format @var{fmt} instead of auto-detecting
> @item -t, --persistent
> don't exit on the last connection
> @item -v, --verbose
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-trivial] [Qemu-devel] [PATCH 2/2] qemu-nbd: Minor texi updates,
Eric Blake <=