diff options
| author | Theodore Ts'o <tytso@mit.edu> | 2011-08-22 16:18:07 -0700 |
|---|---|---|
| committer | Ken Sumrall <ksumrall@android.com> | 2011-08-23 17:59:47 -0700 |
| commit | 5cb5fb6fef3fd81b55ddc48010e1b520e8473553 (patch) | |
| tree | db83388f8263f53460f54ecc11d9f7c458f8ce11 /scripts/gcc-wrapper.py | |
| parent | 76594495616ae4080423f77cbbe1572d954edb29 (diff) | |
ext4: fix nomblk_io_submit option so it correctly converts uninit blocks
Bug discovered by Jan Kara:
Finally, commit 1449032be17abb69116dbc393f67ceb8bd034f92 returned back
the old IO submission code but apparently it forgot to return the old
handling of uninitialized buffers so we unconditionnaly call
block_write_full_page() without specifying end_io function. So AFAICS
we never convert unwritten extents to written in some cases. For
example when I mount the fs as: mount -t ext4 -o
nomblk_io_submit,dioread_nolock /dev/ubdb /mnt and do
int fd = open(argv[1], O_RDWR | O_CREAT | O_TRUNC, 0600);
char buf[1024];
memset(buf, 'a', sizeof(buf));
fallocate(fd, 0, 0, 16384);
write(fd, buf, sizeof(buf));
I get a file full of zeros (after remounting the filesystem so that
pagecache is dropped) instead of seeing the first KB contain 'a's.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Change-Id: I349e4e807d2f8bc3573a5e1b2393212e00ca7650
Signed-off-by: Ken Sumrall <ksumrall@android.com>
Diffstat (limited to 'scripts/gcc-wrapper.py')
0 files changed, 0 insertions, 0 deletions
