ext4: fix ext4_ext_direct_IO()'s return value after converting uninit extents
authorMingming <cmm@us.ibm.com>
Tue, 10 Nov 2009 15:48:08 +0000 (10:48 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Mon, 14 Dec 2009 16:07:16 +0000 (08:07 -0800)
commit3e270fa58d81a994d1e4371be444f2e5bb072866
tree1759262863e8607cedcae5642db954e327722eb7
parentba593a59c912b3ae1f18b74d8ad66594ba3a47b6
ext4: fix ext4_ext_direct_IO()'s return value after converting uninit extents

(cherry picked from commit 109f55651954def97fa41ee71c464d268c512ab0)

After a direct I/O request covering an uninitalized extent (i.e.,
created using the fallocate system call) or a hole in a file, ext4
will convert the uninitialized extent so it is marked as initialized
by calling ext4_convert_unwritten_extents().  This function returns
zero on success.

This return value was getting returned by ext4_direct_IO(); however
the file system's direct_IO function is supposed to return the number
of bytes read or written on a success.  By returning zero, it confused
the direct I/O code into falling back to buffered I/O unnecessarily.

Signed-off-by: Mingming Cao <cmm@us.ibm.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
fs/ext4/extents.c
fs/ext4/inode.c