diff options
author | Andrew Morton <akpm@osdl.org> | 2006-03-24 12:18:11 +0100 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-24 16:33:26 +0100 |
commit | 4741c9fd36b3bcadd37238321c469049da94a4b9 (patch) | |
tree | 4b3031b04cdbf2d3c5a4fcc1965ed857f1cf0ce4 /fs/buffer.c | |
parent | [PATCH] balance_dirty_pages_ratelimited: take nr_pages arg (diff) | |
download | linux-4741c9fd36b3bcadd37238321c469049da94a4b9.tar.xz linux-4741c9fd36b3bcadd37238321c469049da94a4b9.zip |
[PATCH] set_page_dirty() return value fixes
We need set_page_dirty() to return true if it actually transitioned the page
from a clean to dirty state. This wasn't right in a couple of places. Do a
kernel-wide audit, fix things up.
This leaves open the possibility of returning a negative errno from
set_page_dirty() sometime in the future. But we don't do that at present.
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/buffer.c')
-rw-r--r-- | fs/buffer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/buffer.c b/fs/buffer.c index 11ca6eb46a33..24262ea8cc50 100644 --- a/fs/buffer.c +++ b/fs/buffer.c @@ -865,8 +865,8 @@ int __set_page_dirty_buffers(struct page *page) } write_unlock_irq(&mapping->tree_lock); __mark_inode_dirty(mapping->host, I_DIRTY_PAGES); + return 1; } - return 0; } EXPORT_SYMBOL(__set_page_dirty_buffers); |