diff options
| author | Andrew Morton <akpm@osdl.org> | 2006-03-24 03:18:11 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-24 07:33:26 -0800 |
| commit | 4741c9fd36b3bcadd37238321c469049da94a4b9 (patch) | |
| tree | 4b3031b04cdbf2d3c5a4fcc1965ed857f1cf0ce4 /include | |
| parent | fa5a734e406b53761fcc5ee22366006f71112c2d (diff) | |
| download | lwn-4741c9fd36b3bcadd37238321c469049da94a4b9.tar.gz lwn-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 'include')
| -rw-r--r-- | include/linux/fs.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/fs.h b/include/linux/fs.h index 0ad70c1e5e55..092cfaee0cd2 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h @@ -350,7 +350,7 @@ struct address_space_operations { /* Write back some dirty pages from this mapping. */ int (*writepages)(struct address_space *, struct writeback_control *); - /* Set a page dirty */ + /* Set a page dirty. Return true if this dirtied it */ int (*set_page_dirty)(struct page *page); int (*readpages)(struct file *filp, struct address_space *mapping, |