Re: [PATCH 0/3] cachefiles, afs: mm wait fixes

From: Matthew Wilcox
Date: Tue Mar 23 2021 - 08:11:44 EST


On Tue, Mar 23, 2021 at 11:53:22AM +0000, David Howells wrote:
>
> Here are some patches to fix page waiting-related issues in cachefiles and
> afs[1]:
>
> (1) In cachefiles, remove the use of the wait_bit_key struct to access
> something that's actually in wait_page_key format. The proper struct
> is now available in the header, so that should be used instead.
>
> (2) Add a proper wait function for waiting killably on the page writeback
> flag. This includes a recent bugfix here (presumably commit
> c2407cf7d22d0c0d94cf20342b3b8f06f1d904e7).
>
> (3) In afs, use the function added in (2) rather than using
> wait_on_page_bit_killable() which doesn't have the aforementioned
> bugfix.
>
> Note that I modified this to work with the upstream code where the
> page pointer isn't cached in a local variable.

Thanks, the minor modifications to the patches (changelogs, fixes to apply
to upstream) all look good to me.