commit 45d4f855046631d63dab8832ba8a8369ed8e04bd
author Dan Carpenter <dan.carpenter@oracle.com> Thu Apr 03 14:47:17 2014 -0700
committer Linus Torvalds <torvalds@linux-foundation.org> Thu Apr 03 16:20:57 2014 -0700
tree ff603cb87fa84875ac4c34a2bf0f277433257dcf
parent 2b665e276c15ba7d9fc8cdd16931883a51ed13e4

fs/direct-io.c: remove some left over checks

We know that "ret > 0" is true here.  These tests were left over from
commit 02afc27faec9 ('direct-io: Handle O_(D)SYNC AIO') and aren't
needed any more.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

fs/ext4/file.c

diff --git a/fs/ext4/file.c b/fs/ext4/file.c
index 1a50739..6db7f7d 100644
--- a/fs/ext4/file.c
+++ b/fs/ext4/file.c
@@ -153,7 +153,7 @@
 		ssize_t err;
 
 		err = generic_write_sync(file, iocb->ki_pos - ret, ret);
-		if (err < 0 && ret > 0)
+		if (err < 0)
 			ret = err;
 	}
 	blk_finish_plug(&plug);