diff options
| author | Josef Bacik <josef@redhat.com> | 2011-08-04 14:25:02 (GMT) |
|---|---|---|
| committer | Josef Bacik <josef@redhat.com> | 2011-10-19 19:12:31 (GMT) |
| commit | 7709cde33f12db71efb377fae4ae7aab6c94ebc6 (patch) | |
| tree | f60b4ff7dafa0fcf64e0d9e9445d6984739cbfe6 /fs/btrfs/inode.c | |
| parent | 9e4871070b5f070cacf26525389d56e0345ba156 (diff) | |
| download | linux-fsl-qoriq-7709cde33f12db71efb377fae4ae7aab6c94ebc6.tar.xz | |
Btrfs: calculate checksum space correctly
We have not been reserving enough space for checksums. We were just reserving
bytes for the checksum items themselves, we were not taking into account having
to cow the tree and such. This patch adds a csum_bytes counter to the inode for
keeping track of the number of bytes outstanding we have for checksums. Then we
calculate how many leaves would be required for the checksums we are given and
use that to reserve space. This adds a significant amount of bytes to our
reservations, but we will handle this later. Thanks,
Signed-off-by: Josef Bacik <josef@redhat.com>
Diffstat (limited to 'fs/btrfs/inode.c')
| -rw-r--r-- | fs/btrfs/inode.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index 156c3a0..98b9fa2 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -6757,6 +6757,7 @@ struct inode *btrfs_alloc_inode(struct super_block *sb) ei->delalloc_bytes = 0; ei->disk_i_size = 0; ei->flags = 0; + ei->csum_bytes = 0; ei->index_cnt = (u64)-1; ei->last_unlink_trans = 0; @@ -6802,6 +6803,8 @@ void btrfs_destroy_inode(struct inode *inode) WARN_ON(inode->i_data.nrpages); WARN_ON(BTRFS_I(inode)->outstanding_extents); WARN_ON(BTRFS_I(inode)->reserved_extents); + WARN_ON(BTRFS_I(inode)->delalloc_bytes); + WARN_ON(BTRFS_I(inode)->csum_bytes); /* * This can happen where we create an inode, but somebody else also |