diff options
author | Bob Liu <lliubbo@gmail.com> | 2011-01-13 18:45:49 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-01-13 20:32:32 -0500 |
commit | c3f0da631539b3b8e17f6dda567af9958d49d14f (patch) | |
tree | b9bc0060ac0fa99d0dc9e6caadb0abf0172249ef /mm/page-writeback.c | |
parent | c691b9d983d7015d54057034f4cd9b6d8affd976 (diff) |
mm/page-writeback.c: fix __set_page_dirty_no_writeback() return value
__set_page_dirty_no_writeback() should return true if it actually
transitioned the page from a clean to dirty state although it seems nobody
uses its return value at present.
Signed-off-by: Bob Liu <lliubbo@gmail.com>
Acked-by: Wu Fengguang <fengguang.wu@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/page-writeback.c')
-rw-r--r-- | mm/page-writeback.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/page-writeback.c b/mm/page-writeback.c index b5d8a1f820a0..28763b8bdbdd 100644 --- a/mm/page-writeback.c +++ b/mm/page-writeback.c | |||
@@ -1103,7 +1103,7 @@ EXPORT_SYMBOL(write_one_page); | |||
1103 | int __set_page_dirty_no_writeback(struct page *page) | 1103 | int __set_page_dirty_no_writeback(struct page *page) |
1104 | { | 1104 | { |
1105 | if (!PageDirty(page)) | 1105 | if (!PageDirty(page)) |
1106 | SetPageDirty(page); | 1106 | return !TestSetPageDirty(page); |
1107 | return 0; | 1107 | return 0; |
1108 | } | 1108 | } |
1109 | 1109 | ||