From: Alison Schofield <alison.schofield@intel.com>
To: "Xingtao Yao (Fujitsu)" <yaoxt.fnst@fujitsu.com>
Cc: "nvdimm@lists.linux.dev" <nvdimm@lists.linux.dev>,
"linux-cxl@vger.kernel.org" <linux-cxl@vger.kernel.org>,
Dave Jiang <dave.jiang@intel.com>
Subject: Re: [ndctl PATCH v13 7/8] cxl/list: add --media-errors option to cxl list
Date: Mon, 8 Jul 2024 13:56:55 -0700 [thread overview]
Message-ID: <ZoxSl2tBU5ARmLuC@aschofie-mobl2> (raw)
In-Reply-To: <OSZPR01MB6453378E193594694EFD71BD8DDA2@OSZPR01MB6453.jpnprd01.prod.outlook.com>
On Mon, Jul 08, 2024 at 02:26:11AM +0000, Xingtao Yao (Fujitsu) wrote:
> > -v::
> > --verbose::
> > Increase verbosity of the output. This can be specified
> > @@ -431,7 +485,7 @@ OPTIONS
> > devices with --idle.
> > - *-vvv*
> > Everything *-vv* provides, plus enable
> > - --health and --partition.
> > + --health, --partition, --media-errors.
> insert an “and” before "--media-errors" may be better.
Well...I'm not a fan of the "and', but since it was there before,
and it is still used in the single -v case, I'll put it back to
be the same.
Eagle eye! Thanks for testing it out!
--Alison
>
> Tested-by: Xingtao Yao <yaoxt.fnst@fujitsu.com>
next prev parent reply other threads:[~2024-07-08 20:56 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-06 6:24 [ndctl PATCH v13 0/8] Support poison list retrieval alison.schofield
2024-07-06 6:24 ` [ndctl PATCH v13 1/8] util/trace: move trace helpers from ndctl/cxl/ to ndctl/util/ alison.schofield
2024-07-24 20:50 ` Dave Jiang
2024-07-06 6:24 ` [ndctl PATCH v13 2/8] util/trace: add an optional pid check to event parsing alison.schofield
2024-07-06 6:24 ` [ndctl PATCH v13 3/8] util/trace: pass an event_ctx to its own parse_event method alison.schofield
2024-07-24 20:55 ` Dave Jiang
2024-07-06 6:24 ` [ndctl PATCH v13 4/8] util/trace: add helpers to retrieve tep fields by type alison.schofield
2024-07-06 6:24 ` [ndctl PATCH v13 5/8] libcxl: add interfaces for GET_POISON_LIST mailbox commands alison.schofield
2024-07-06 6:24 ` [ndctl PATCH v13 6/8] cxl/list: collect and parse media_error records alison.schofield
2024-07-08 2:07 ` Xingtao Yao (Fujitsu)
2024-07-08 20:54 ` Alison Schofield
2024-07-06 6:24 ` [ndctl PATCH v13 7/8] cxl/list: add --media-errors option to cxl list alison.schofield
2024-07-08 2:26 ` Xingtao Yao (Fujitsu)
2024-07-08 20:56 ` Alison Schofield [this message]
2024-07-06 6:24 ` [ndctl PATCH v13 8/8] cxl/test: add cxl-poison.sh unit test alison.schofield
2024-07-08 2:10 ` Xingtao Yao (Fujitsu)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZoxSl2tBU5ARmLuC@aschofie-mobl2 \
--to=alison.schofield@intel.com \
--cc=dave.jiang@intel.com \
--cc=linux-cxl@vger.kernel.org \
--cc=nvdimm@lists.linux.dev \
--cc=yaoxt.fnst@fujitsu.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.