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-jiaqi...@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.willi...@intel.com>