diff options
author | Darrick J. Wong <djwong@us.ibm.com> | 2011-05-27 15:23:34 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2011-05-28 01:03:21 -0400 |
commit | 3d08bcc887a1c8d12be8d81f747ffa2e8a44b67b (patch) | |
tree | 258f4e3d384e451c921c5e0c503fd368f236a20a /mm/filemap.c | |
parent | 98702467f829177b3993f17da9fe5c202d160e5e (diff) |
mm: Wait for writeback when grabbing pages to begin a write
When grabbing a page for a buffered IO write, the mm should wait for writeback
on the page to complete so that the page does not become writable during the IO
operation. This change is needed to provide page stability during writes for
all filesystems.
Signed-off-by: Darrick J. Wong <djwong@us.ibm.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'mm/filemap.c')
-rw-r--r-- | mm/filemap.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/mm/filemap.c b/mm/filemap.c index bcdc393b6580..dac95a24deac 100644 --- a/mm/filemap.c +++ b/mm/filemap.c | |||
@@ -2327,7 +2327,7 @@ struct page *grab_cache_page_write_begin(struct address_space *mapping, | |||
2327 | repeat: | 2327 | repeat: |
2328 | page = find_lock_page(mapping, index); | 2328 | page = find_lock_page(mapping, index); |
2329 | if (page) | 2329 | if (page) |
2330 | return page; | 2330 | goto found; |
2331 | 2331 | ||
2332 | page = __page_cache_alloc(mapping_gfp_mask(mapping) & ~gfp_notmask); | 2332 | page = __page_cache_alloc(mapping_gfp_mask(mapping) & ~gfp_notmask); |
2333 | if (!page) | 2333 | if (!page) |
@@ -2340,6 +2340,8 @@ repeat: | |||
2340 | goto repeat; | 2340 | goto repeat; |
2341 | return NULL; | 2341 | return NULL; |
2342 | } | 2342 | } |
2343 | found: | ||
2344 | wait_on_page_writeback(page); | ||
2343 | return page; | 2345 | return page; |
2344 | } | 2346 | } |
2345 | EXPORT_SYMBOL(grab_cache_page_write_begin); | 2347 | EXPORT_SYMBOL(grab_cache_page_write_begin); |