ndctl read-labels default output format

ndctl read-labels default output format

[Elliott, Robert (Persistent Memory)]

> -----Original Message-----
> From: Linux-nvdimm [mailto:linux-nvdimm-bounces@lists.01.org] On Behalf Of Dan Williams
> Sent: Saturday, April 9, 2016 2:40 PM
> To: linux-nvdimm@lists.01.org
> Subject: [ndctl PATCH 3/4] ndctl: new 'read-labels' utility
> 
> For debug, it is useful to be able to dump the label area of a dimm.
...
> +ndctl-read-labels - read out the label area on a dimm or set of dimms
> +
> +SYNOPSIS
> +--------
> +[verse]
> +'ndctl read-labels' <nmem0> [<nmem1>..<nmemN>] [<options>]
> +
> +include::labels-description.txt[]
> +This command dumps the raw binary data in a dimm's label area to stdout or a
> +file.  In the multi-dimm case the data is concatenated.
> +
> +OPTIONS
> +-------
> +include::labels-options.txt[]
> +-o::
> +	output file
> +-j::
> +--json::
> +	parse the label data into json assuming the 'NVDIMM Namespace
> +	Specification' format.
> +

Most ndctl commands output JSON.  This one is a bit unfriendly in dumping
binary data.  Could -j be the default mode (possibly also supporting the
-u/--human option)?

Also, it prints "read %d nmems" to stderr. That doesn't mess up piping
stdout to another tool like jq, but does mess up cutting-and-pasting
the console output and feeding it to another tool.  Could that line
be dropped?  Keep the similar prints for commands that request actions
rather than print information.

---
Robert Elliott, HPE Persistent Memory


_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm