diff options
-rw-r--r-- | fs/exofs/dir.c | 2 | ||||
-rw-r--r-- | fs/ext2/dir.c | 2 | ||||
-rw-r--r-- | fs/jfs/jfs_metapage.c | 4 | ||||
-rw-r--r-- | fs/minix/dir.c | 2 | ||||
-rw-r--r-- | fs/sysv/dir.c | 2 | ||||
-rw-r--r-- | fs/ufs/dir.c | 2 | ||||
-rw-r--r-- | include/linux/mm.h | 2 | ||||
-rw-r--r-- | mm/page-writeback.c | 14 |
8 files changed, 15 insertions, 15 deletions
diff --git a/fs/exofs/dir.c b/fs/exofs/dir.c index 8eeb694332fe..98233a97b7b8 100644 --- a/fs/exofs/dir.c +++ b/fs/exofs/dir.c | |||
@@ -72,7 +72,7 @@ static int exofs_commit_chunk(struct page *page, loff_t pos, unsigned len) | |||
72 | set_page_dirty(page); | 72 | set_page_dirty(page); |
73 | 73 | ||
74 | if (IS_DIRSYNC(dir)) | 74 | if (IS_DIRSYNC(dir)) |
75 | err = write_one_page(page, 1); | 75 | err = write_one_page(page); |
76 | else | 76 | else |
77 | unlock_page(page); | 77 | unlock_page(page); |
78 | 78 | ||
diff --git a/fs/ext2/dir.c b/fs/ext2/dir.c index d9650c9508e4..e2709695b177 100644 --- a/fs/ext2/dir.c +++ b/fs/ext2/dir.c | |||
@@ -100,7 +100,7 @@ static int ext2_commit_chunk(struct page *page, loff_t pos, unsigned len) | |||
100 | } | 100 | } |
101 | 101 | ||
102 | if (IS_DIRSYNC(dir)) { | 102 | if (IS_DIRSYNC(dir)) { |
103 | err = write_one_page(page, 1); | 103 | err = write_one_page(page); |
104 | if (!err) | 104 | if (!err) |
105 | err = sync_inode_metadata(dir, 1); | 105 | err = sync_inode_metadata(dir, 1); |
106 | } else { | 106 | } else { |
diff --git a/fs/jfs/jfs_metapage.c b/fs/jfs/jfs_metapage.c index 489aaa1403e5..744fa3c079e6 100644 --- a/fs/jfs/jfs_metapage.c +++ b/fs/jfs/jfs_metapage.c | |||
@@ -711,7 +711,7 @@ void force_metapage(struct metapage *mp) | |||
711 | get_page(page); | 711 | get_page(page); |
712 | lock_page(page); | 712 | lock_page(page); |
713 | set_page_dirty(page); | 713 | set_page_dirty(page); |
714 | write_one_page(page, 1); | 714 | write_one_page(page); |
715 | clear_bit(META_forcewrite, &mp->flag); | 715 | clear_bit(META_forcewrite, &mp->flag); |
716 | put_page(page); | 716 | put_page(page); |
717 | } | 717 | } |
@@ -756,7 +756,7 @@ void release_metapage(struct metapage * mp) | |||
756 | set_page_dirty(page); | 756 | set_page_dirty(page); |
757 | if (test_bit(META_sync, &mp->flag)) { | 757 | if (test_bit(META_sync, &mp->flag)) { |
758 | clear_bit(META_sync, &mp->flag); | 758 | clear_bit(META_sync, &mp->flag); |
759 | write_one_page(page, 1); | 759 | write_one_page(page); |
760 | lock_page(page); /* write_one_page unlocks the page */ | 760 | lock_page(page); /* write_one_page unlocks the page */ |
761 | } | 761 | } |
762 | } else if (mp->lsn) /* discard_metapage doesn't remove it */ | 762 | } else if (mp->lsn) /* discard_metapage doesn't remove it */ |
diff --git a/fs/minix/dir.c b/fs/minix/dir.c index 7edc9b395700..baa9721f1299 100644 --- a/fs/minix/dir.c +++ b/fs/minix/dir.c | |||
@@ -57,7 +57,7 @@ static int dir_commit_chunk(struct page *page, loff_t pos, unsigned len) | |||
57 | mark_inode_dirty(dir); | 57 | mark_inode_dirty(dir); |
58 | } | 58 | } |
59 | if (IS_DIRSYNC(dir)) | 59 | if (IS_DIRSYNC(dir)) |
60 | err = write_one_page(page, 1); | 60 | err = write_one_page(page); |
61 | else | 61 | else |
62 | unlock_page(page); | 62 | unlock_page(page); |
63 | return err; | 63 | return err; |
diff --git a/fs/sysv/dir.c b/fs/sysv/dir.c index 5bdae85ceef7..f5191cb2c947 100644 --- a/fs/sysv/dir.c +++ b/fs/sysv/dir.c | |||
@@ -45,7 +45,7 @@ static int dir_commit_chunk(struct page *page, loff_t pos, unsigned len) | |||
45 | mark_inode_dirty(dir); | 45 | mark_inode_dirty(dir); |
46 | } | 46 | } |
47 | if (IS_DIRSYNC(dir)) | 47 | if (IS_DIRSYNC(dir)) |
48 | err = write_one_page(page, 1); | 48 | err = write_one_page(page); |
49 | else | 49 | else |
50 | unlock_page(page); | 50 | unlock_page(page); |
51 | return err; | 51 | return err; |
diff --git a/fs/ufs/dir.c b/fs/ufs/dir.c index de01b8f2aa78..48609f1d9580 100644 --- a/fs/ufs/dir.c +++ b/fs/ufs/dir.c | |||
@@ -53,7 +53,7 @@ static int ufs_commit_chunk(struct page *page, loff_t pos, unsigned len) | |||
53 | mark_inode_dirty(dir); | 53 | mark_inode_dirty(dir); |
54 | } | 54 | } |
55 | if (IS_DIRSYNC(dir)) | 55 | if (IS_DIRSYNC(dir)) |
56 | err = write_one_page(page, 1); | 56 | err = write_one_page(page); |
57 | else | 57 | else |
58 | unlock_page(page); | 58 | unlock_page(page); |
59 | return err; | 59 | return err; |
diff --git a/include/linux/mm.h b/include/linux/mm.h index 7cb17c6b97de..ca9c8b27cecb 100644 --- a/include/linux/mm.h +++ b/include/linux/mm.h | |||
@@ -2199,7 +2199,7 @@ extern void filemap_map_pages(struct vm_fault *vmf, | |||
2199 | extern int filemap_page_mkwrite(struct vm_fault *vmf); | 2199 | extern int filemap_page_mkwrite(struct vm_fault *vmf); |
2200 | 2200 | ||
2201 | /* mm/page-writeback.c */ | 2201 | /* mm/page-writeback.c */ |
2202 | int write_one_page(struct page *page, int wait); | 2202 | int __must_check write_one_page(struct page *page); |
2203 | void task_dirty_inc(struct task_struct *tsk); | 2203 | void task_dirty_inc(struct task_struct *tsk); |
2204 | 2204 | ||
2205 | /* readahead.c */ | 2205 | /* readahead.c */ |
diff --git a/mm/page-writeback.c b/mm/page-writeback.c index 143c1c25d680..b901fe52b153 100644 --- a/mm/page-writeback.c +++ b/mm/page-writeback.c | |||
@@ -2366,15 +2366,16 @@ int do_writepages(struct address_space *mapping, struct writeback_control *wbc) | |||
2366 | } | 2366 | } |
2367 | 2367 | ||
2368 | /** | 2368 | /** |
2369 | * write_one_page - write out a single page and optionally wait on I/O | 2369 | * write_one_page - write out a single page and wait on I/O |
2370 | * @page: the page to write | 2370 | * @page: the page to write |
2371 | * @wait: if true, wait on writeout | ||
2372 | * | 2371 | * |
2373 | * The page must be locked by the caller and will be unlocked upon return. | 2372 | * The page must be locked by the caller and will be unlocked upon return. |
2374 | * | 2373 | * |
2375 | * write_one_page() returns a negative error code if I/O failed. | 2374 | * write_one_page() returns a negative error code if I/O failed. Note that |
2375 | * the address_space is not marked for error. The caller must do this if | ||
2376 | * needed. | ||
2376 | */ | 2377 | */ |
2377 | int write_one_page(struct page *page, int wait) | 2378 | int write_one_page(struct page *page) |
2378 | { | 2379 | { |
2379 | struct address_space *mapping = page->mapping; | 2380 | struct address_space *mapping = page->mapping; |
2380 | int ret = 0; | 2381 | int ret = 0; |
@@ -2385,13 +2386,12 @@ int write_one_page(struct page *page, int wait) | |||
2385 | 2386 | ||
2386 | BUG_ON(!PageLocked(page)); | 2387 | BUG_ON(!PageLocked(page)); |
2387 | 2388 | ||
2388 | if (wait) | 2389 | wait_on_page_writeback(page); |
2389 | wait_on_page_writeback(page); | ||
2390 | 2390 | ||
2391 | if (clear_page_dirty_for_io(page)) { | 2391 | if (clear_page_dirty_for_io(page)) { |
2392 | get_page(page); | 2392 | get_page(page); |
2393 | ret = mapping->a_ops->writepage(page, &wbc); | 2393 | ret = mapping->a_ops->writepage(page, &wbc); |
2394 | if (ret == 0 && wait) { | 2394 | if (ret == 0) { |
2395 | wait_on_page_writeback(page); | 2395 | wait_on_page_writeback(page); |
2396 | if (PageError(page)) | 2396 | if (PageError(page)) |
2397 | ret = -EIO; | 2397 | ret = -EIO; |