Re: [PATCH 2/3] cxl/mbox: Add GET_POISON_LIST mailbox command support

From: Alison Schofield
Date: Thu Jun 16 2022 - 19:15:47 EST


On Thu, Jun 16, 2022 at 03:45:25PM -0700, Davidlohr Bueso wrote:
> On Thu, 16 Jun 2022, Alison Schofield wrote:
>
> >cxl list --media-errors -m mem1
> > lists media errors for requested memdev
> >
> >cxl list --media-errors -r region#
>
> A quick question on the tooling front: the above goes nicely with
> cxl-list, but what about the rest of the poisoning cmds? Do you have
> anything in mind? Do we want something specific for media and poison
> management instead? Ie:
>
> cxl media --list-errors <params>
Not clear how this one differs. Seems like we can get any piece of
the list w cxl list.

> cxl media --inject-errors <params>
> cxl media --clear-errors <params>
For inject/clear I'd probably start w what ndctl does today.
ndctl inject−error <namespace> [<options>]
where option -d --uninject performs the clear.

>
> Thanks,
> Davidlohr