diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2006-10-11 04:21:58 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-11 14:14:22 -0400 |
commit | 887ed2f3aecde2ff24e06666932dc5f144745044 (patch) | |
tree | 870280190c3825122dc8e30e7c2be66465b9f9f2 /mm/truncate.c | |
parent | cd0810410beb86c570aeb3dcd3dc1fc5ab9e6077 (diff) |
[PATCH] VM: Fix the gfp_mask in invalidate_complete_page2
If try_to_release_page() is called with a zero gfp mask, then the
filesystem is effectively denied the possibility of sleeping while
attempting to release the page. There doesn't appear to be any valid
reason why this should be banned, given that we're not calling this from a
memory allocation context.
For this reason, change the gfp_mask argument of the call to GFP_KERNEL.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: Steve Dickson <SteveD@redhat.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'mm/truncate.c')
-rw-r--r-- | mm/truncate.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/truncate.c b/mm/truncate.c index fca28839c46e..11ca480701dd 100644 --- a/mm/truncate.c +++ b/mm/truncate.c | |||
@@ -302,7 +302,7 @@ invalidate_complete_page2(struct address_space *mapping, struct page *page) | |||
302 | if (page->mapping != mapping) | 302 | if (page->mapping != mapping) |
303 | return 0; | 303 | return 0; |
304 | 304 | ||
305 | if (PagePrivate(page) && !try_to_release_page(page, 0)) | 305 | if (PagePrivate(page) && !try_to_release_page(page, GFP_KERNEL)) |
306 | return 0; | 306 | return 0; |
307 | 307 | ||
308 | write_lock_irq(&mapping->tree_lock); | 308 | write_lock_irq(&mapping->tree_lock); |