diff options
| author |   Todd Poynor <tpoynor@mvista.com> | 2005-10-19 00:29:38 +0100 |
|---|---|---|
| committer |   Thomas Gleixner <tglx@mtd.linutronix.de> | 2005-11-06 23:14:16 +0100 |
| commit | 4fc67fbe52d7c34dfd3e03a1a79f3e078904bba2 (patch) | |
| tree | 34c23c384bd19123afdad8495927afd400a2b5db | |
| parent | [MTD] Don't let gcc inline functions marked __xipram (diff) | |
| download | linux-dev-4fc67fbe52d7c34dfd3e03a1a79f3e078904bba2.tar.xz linux-dev-4fc67fbe52d7c34dfd3e03a1a79f3e078904bba2.zip | |
[JFFS2] Return 0, not number of bytes written, for success at commit_write
Some callers to block-layer commit_write function treat non-zero return as
error, notably the loopback mount driver sometimes used in conjunction with
JFFS2 on NAND flash for bad block avoidance, etc. Return zero for success
as do various other commit_write functions.
Signed-off-by: Todd Poynor <tpoynor@mvista.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
| -rw-r--r-- | fs/jffs2/file.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/jffs2/file.c b/fs/jffs2/file.c index 231404a74728..605ea6b0b473 100644 --- a/fs/jffs2/file.c +++ b/fs/jffs2/file.c @@ -7,7 +7,7 @@ * * For licensing information, see the file 'LICENCE' in this directory. * - * $Id: file.c,v 1.103 2005/09/07 08:34:54 havasi Exp $ + * $Id: file.c,v 1.104 2005/10/18 23:29:35 tpoynor Exp $ * */ @@ -279,6 +279,6 @@ static int jffs2_commit_write (struct file *filp, struct page *pg, ClearPageUptodate(pg); } - D1(printk(KERN_DEBUG "jffs2_commit_write() returning %d\n",writtenlen?writtenlen:ret)); - return writtenlen?writtenlen:ret; + D1(printk(KERN_DEBUG "jffs2_commit_write() returning %d\n",start+writtenlen==end?0:ret)); + return start+writtenlen==end?0:ret; } |