Re: [PATCH] [13/16] POISON: The high level memory error handler in the VM II

Andi Kleen
2009-04-09T14:56:08Z

> Even though try_to_releasepage only checks page_writeback() the lower
> filesystems all bail on dirty pages or dirty buffers (see the checks
> done by try_to_free_buffers).
>
> It looks like the only way we have to clean a page and all the buffers
> in it is the invalidatepage call. But that doesn't return success or
> failure, so maybe invalidatepage followed by releasepage?

Ok. I'll poke at it more.

>
> I'll have to read harder next week, the FS invalidatepage may expect
> truncate to be the only caller.

I have to be careful with locks; another lock would deadlock. Ok
I could drop the page lock temporarily, but that would be somewhat
risky of someone else coming in unexpectedly.

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