Re: xfs: outstanding patches for 2.6.39

Andi Kleen
2011-03-23T16:02:23Z

> We can't do that right now, anyway.

It seemed to work on XFS when we tested it. Undoutedly after your
patch it won't work anymore.

> If the page is not in use, we don't care about it after this patch
> set is applied - the page is either active in a buffer or it has been
> freed. If it is in use, then we'll shut the filesystem down if we
> detect the memory corruption just like we currently do. Hence I
> don't see any regression here.

I think you're confusing the memory_failure() HWPoison path with some XFS
internal checking. I don't think XFS has any HWPoison checking on its own.

> As it is, there is no way for the filesytem to be notified about
> such failures on active pages in buffers, so in reality we can't
> reliably detect them so there is little point in trying to recover
> from such errors.

Well it works today if the page is in pagecache. memory-failure will
just remove it transparently.

In principle you can check for HWPoison manually yourself, but I'm not sure
that is a good way to do it.

-Andi
--
ak@linux.intel.com -- Speaking for myself only.