diff options
| author | Chris Mason | 2008-02-19 17:29:24 +0100 |
|---|---|---|
| committer | Chris Mason | 2008-09-25 17:04:00 +0200 |
| commit | 0762704b196d41941a9b439e9165efaf85c6609e (patch) | |
| tree | 2bbb1a222d7b0ccbd23da5cf614a8507338390a6 /fs/btrfs/file.c | |
| parent | Btrfs: Remove extent back refs in batches, and avoid duplicate searches (diff) | |
| download | kernel-qcow2-linux-0762704b196d41941a9b439e9165efaf85c6609e.tar.gz kernel-qcow2-linux-0762704b196d41941a9b439e9165efaf85c6609e.tar.xz kernel-qcow2-linux-0762704b196d41941a9b439e9165efaf85c6609e.zip | |
Btrfs: Properly clear dirty and delalloc extent bits while preparing the file for write
Yan Zheng noticed that we don't clear the extent state tree dirty and delalloc
bits when we clear the dirty bits on the page during file write.
This leads to csum errors later on.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/file.c')
| -rw-r--r-- | fs/btrfs/file.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/fs/btrfs/file.c b/fs/btrfs/file.c index b0352b5958a5..1a47251a9d4d 100644 --- a/fs/btrfs/file.c +++ b/fs/btrfs/file.c @@ -762,6 +762,13 @@ static int prepare_pages(struct btrfs_root *root, struct file *file, set_page_extent_mapped(pages[i]); WARN_ON(!PageLocked(pages[i])); } + if (start_pos < inode->i_size) { + u64 last_pos; + last_pos = (index + num_pages) << PAGE_CACHE_SHIFT; + clear_extent_bits(&BTRFS_I(inode)->io_tree, start_pos, + last_pos - 1, EXTENT_DIRTY | EXTENT_DELALLOC, + GFP_NOFS); + } return 0; } |