[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 1/2] Clarify virsh freecell manpage entry



On 03/22/2012 01:59 PM, Dave Allan wrote:
> ---
>  tools/virsh.pod |   23 ++++++++++++++++++-----
>  1 files changed, 18 insertions(+), 5 deletions(-)
> 
> diff --git a/tools/virsh.pod b/tools/virsh.pod
> index f0df4fd..b4deae8 100644
> --- a/tools/virsh.pod
> +++ b/tools/virsh.pod
> @@ -401,11 +401,24 @@ B<virsh> list --title
>    0 Domain-0             running    Mailserver 1
>    2 fedora               paused
> 
> -=item B<freecell> [B<cellno> | I<--all>]
> -
> -Prints the available amount of memory on the machine or within a
> -NUMA cell if I<cellno> is provided.  If I<--all> is provided instead
> -of I<--cellno>, then show the information on all NUMA cells.
> +=item B<freecell> [{ [I<--cellno>] B<cellno> | I<--all> }]
> +
> +Prints the available amount of memory on the machine or within a NUMA
> +cell.  The freecell command can provide one of three different
> +displays of available memory on the machine depending on the options
> +specified.  With no options, it displays the total free memory on the
> +machine.  With the --all option, it displays the free memory on each
> +node and the total free memory on the machine.  Finally, with a
> +numeric argument or with --cellno it will display the free memory for
> +that node only.

I like this part.

> +
> + For example:
> +
> +  freecell            => total free memory on the machine
> +  freecell --all      => free memory on each node, and the total
> +  freecell --cellno 0 => free memory on node 0 only
> +  freecell 0          => free memory on node 0 only
> +  freecell --cellno=0 => free memory on node 0 only

This may be a bit verbose; we aren't giving this many examples elsewhere
in the man page.  I'm not entirely opposed to verbosity, if it helps
silence confusion, but if others agree to keep this many examples, can
we at least add some variety, as in:

freecell 1  => free memory on node 1 only

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]