diff options
| author | Yan Zheng <zheng.yan@oracle.com> | 2008-12-12 10:03:38 -0500 |
|---|---|---|
| committer | Chris Mason <chris.mason@oracle.com> | 2008-12-12 10:03:38 -0500 |
| commit | 17d217fe970d34720f4f1633dca73a6aa2f3d9d1 (patch) | |
| tree | 4e2e716400cc45a6697475629f4c046b96ff76e7 /fs/btrfs/file.c | |
| parent | e4404d6e8da678d852b7f767f665f8edf76c9e9f (diff) | |
| download | op-kernel-dev-17d217fe970d34720f4f1633dca73a6aa2f3d9d1.zip op-kernel-dev-17d217fe970d34720f4f1633dca73a6aa2f3d9d1.tar.gz | |
Btrfs: fix nodatasum handling in balancing code
Checksums on data can be disabled by mount option, so it's
possible some data extents don't have checksums or have
invalid checksums. This causes trouble for data relocation.
This patch contains following things to make data relocation
work.
1) make nodatasum/nodatacow mount option only affects new
files. Checksums and COW on data are only controlled by the
inode flags.
2) check the existence of checksum in the nodatacow checker.
If checksums exist, force COW the data extent. This ensure that
checksum for a given block is either valid or does not exist.
3) update data relocation code to properly handle the case
of checksum missing.
Signed-off-by: Yan Zheng <zheng.yan@oracle.com>
Diffstat (limited to 'fs/btrfs/file.c')
| -rw-r--r-- | fs/btrfs/file.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/fs/btrfs/file.c b/fs/btrfs/file.c index 71bfe3a..5070810 100644 --- a/fs/btrfs/file.c +++ b/fs/btrfs/file.c @@ -1060,14 +1060,6 @@ static ssize_t btrfs_file_write(struct file *file, const char __user *buf, last_index = (pos + count) >> PAGE_CACHE_SHIFT; /* - * if this is a nodatasum mount, force summing off for the inode - * all the time. That way a later mount with summing on won't - * get confused - */ - if (btrfs_test_opt(root, NODATASUM)) - btrfs_set_flag(inode, NODATASUM); - - /* * there are lots of better ways to do this, but this code * makes sure the first and last page in the file range are * up to date and ready for cow |
