ext4: don't set PageUptodate in ext4_end_bio()
authorCurt Wohlgemuth <curtw@google.com>
Sat, 30 Apr 2011 17:26:26 +0000 (13:26 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Sat, 30 Apr 2011 17:26:26 +0000 (13:26 -0400)
commit39db00f1c45e770856264bdb3ceca27980b01965
treea06fcf532d9bbb06395016a653840efa7436c6bf
parent2035e776050aea57fb5255557216473e82793f2c
ext4: don't set PageUptodate in ext4_end_bio()

In the bio completion routine, we should not be setting
PageUptodate at all -- it's set at sys_write() time, and is
unaffected by success/failure of the write to disk.

This can cause a page corruption bug when the file system's
block size is less than the architecture's VM page size.

if we have only written a single block -- we might end up
setting the page's PageUptodate flag, indicating that page
is completely read into memory, which may not be true.
This could cause subsequent reads to get bad data.

This commit also takes the opportunity to clean up error
handling in ext4_end_bio(), and remove some extraneous code:

   - fixes ext4_end_bio() to set AS_EIO in the
     page->mapping->flags on error, which was left out by
     mistake.  This is needed so that fsync() will
     return an error if there was an I/O error.
   - remove the clear_buffer_dirty() call on unmapped
     buffers for each page.
   - consolidate page/buffer error handling in a single
     section.

Signed-off-by: Curt Wohlgemuth <curtw@google.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Reported-by: Jim Meyering <jim@meyering.net>
Reported-by: Hugh Dickins <hughd@google.com>
Cc: Mingming Cao <cmm@us.ibm.com>
fs/ext4/page-io.c