Re: [PATCH 0/4] Improve swap page error handling

[Richard Purdie <richard@openedhand.com>]

On Thu, 2007-01-11 at 09:32 +1100, Nick Piggin wrote:
> Richard Purdie wrote: 
> > I think you were cc'd on some of it but you never commented. Anyhow,
> > I've reworked this patch series based on your comments. The hints were
> > appreciated, thanks. This was the way I'd originally hoped to be able to
> > work things, I just couldn't find the right way to do it.
> 
> IMO it seems a bit complex for so small a benefit. Last time I was
> working on this, I thought it would be almost as good to do something
> simple like stop trying to write out the page if PG_error is set (and
> clear that bit in delete_from_swap_cache or try_to_unusesomewhere).
> This way the admin could swapoff and scan the swap device at some
> point.

FWIW, the patches have got a lot less invasive and I was pleased with
the way the last set I posted worked out. 

1/4 is a lot of noise adding the page parameter to swap_free but doesn't
actually change much. 
2/4 is the guts of the solution in the form of two new functions. 
3/4 just hooks it in. 
4/4 is an optional cleanup.

I guess the key point for me is that the lack of proper handling of this
was bringing one of my systems to its knees due to IO loops before these
patches. Yes, there are ways to minimise the impact but why not fix it
properly? This proposal is certainly nowhere near as invasive as the
previous ones and since its got this far...

> > You can't proceed to do that until you're able to identify the bad pages
> > so this would be a necessary first step towards that, yes.
> 
> Agreed here, FWIW. I think that might be just as well done in
> userspace?

Maybe, I haven't made my mind up about that yet. I'd have to see how the
code looked I guess.

Cheers,

Richard