diff options
author | Mingming <cmm@us.ibm.com> | 2009-11-06 04:01:23 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2009-11-06 04:01:23 -0500 |
commit | ba230c3f6dc88ec008806adb27b12088486d508e (patch) | |
tree | 4cd81bc7425161f1103408ce210be9172a2ee6c3 /fs/timerfd.c | |
parent | 4b70df181611012a3556f017b57dfcef7e1d279f (diff) | |
download | op-kernel-dev-ba230c3f6dc88ec008806adb27b12088486d508e.zip op-kernel-dev-ba230c3f6dc88ec008806adb27b12088486d508e.tar.gz |
ext4: Fix return value of ext4_split_unwritten_extents() to fix direct I/O
To prepare for a direct I/O write, we need to split the unwritten
extents before submitting the I/O. When no extents needed to be
split, ext4_split_unwritten_extents() was incorrectly returning 0
instead of the size of uninitialized extents. This bug caused the
wrong return value sent back to VFS code when it gets called from
async IO path, leading to an unnecessary fall back to buffered IO.
This bug also hid the fact that the check to see whether or not a
split would be necessary was incorrect; we can only skip splitting the
extent if the write completely covers the uninitialized extent.
Signed-off-by: Mingming Cao <cmm@us.ibm.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/timerfd.c')
0 files changed, 0 insertions, 0 deletions