Re: [ndctl PATCH] daxctl: Skip over memory failure node status

[Dan Williams <dan.j.williams@intel.com>]

Adam Manzanares wrote:
> On Mon, Feb 13, 2023 at 03:26:58PM -0800, Dan Williams wrote:
> > Adam Manzanares wrote:
> > > When trying to match a dax device to a memblock physical address
> > > memblock_in_dev will fail if the the phys_index sysfs file does
> > > not exist in the memblock. Currently the memory failure directory
> > > associated with a node is currently interpreted as a memblock.
> > > Skip over the memory_failure directory within the node directory.
> > 
> > Oh, interesting, I did not know memory_failure() added entries to sysfs.
> > My grep-fu is failing me though... I only found node_init_cache_dev()
> > that creates a file named "memory_side_cache" under a node. This fix
> > will work for that as well, but I am still curious where the memory
> > failure file originates.
> 
> I found the issue on next-20230119, I have a suspicion your grep-fu will
> work fine there. 

Yup, thanks. For those following along at home, it's these patches:

https://lore.kernel.org/all/20230120034622.2698268-1-jiaqiyan@google.com

...which has some implications for interoperating with CXL Scan Media
which is distinct from a hardware patrol scrubber, but that's a
discussion for a different patch set.

For this patch though:

Reviewed-by: Dan Williams <dan.j.williams@intel.com>