diff options
author | Andrew Morton <akpm@osdl.org> | 2006-03-24 06:18:11 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-24 10:33:26 -0500 |
commit | 4741c9fd36b3bcadd37238321c469049da94a4b9 (patch) | |
tree | 4b3031b04cdbf2d3c5a4fcc1965ed857f1cf0ce4 /arch | |
parent | fa5a734e406b53761fcc5ee22366006f71112c2d (diff) |
[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 'arch')
-rw-r--r-- | arch/cris/arch-v32/drivers/cryptocop.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/cris/arch-v32/drivers/cryptocop.c b/arch/cris/arch-v32/drivers/cryptocop.c index 501fa52d8d3a..c59ee28a35f4 100644 --- a/arch/cris/arch-v32/drivers/cryptocop.c +++ b/arch/cris/arch-v32/drivers/cryptocop.c | |||
@@ -2944,7 +2944,7 @@ static int cryptocop_ioctl_process(struct inode *inode, struct file *filp, unsig | |||
2944 | int spdl_err; | 2944 | int spdl_err; |
2945 | /* Mark output pages dirty. */ | 2945 | /* Mark output pages dirty. */ |
2946 | spdl_err = set_page_dirty_lock(outpages[i]); | 2946 | spdl_err = set_page_dirty_lock(outpages[i]); |
2947 | DEBUG(if (spdl_err)printk("cryptocop_ioctl_process: set_page_dirty_lock returned %d\n", spdl_err)); | 2947 | DEBUG(if (spdl_err < 0)printk("cryptocop_ioctl_process: set_page_dirty_lock returned %d\n", spdl_err)); |
2948 | } | 2948 | } |
2949 | for (i = 0; i < nooutpages; i++){ | 2949 | for (i = 0; i < nooutpages; i++){ |
2950 | put_page(outpages[i]); | 2950 | put_page(outpages[i]); |