diff options
| author | Josef Bacik <jbacik@fusionio.com> | 2013-04-04 09:57:50 -0400 |
|---|---|---|
| committer | David Sterba <dsterba@suse.cz> | 2013-04-23 18:56:21 +0200 |
| commit | d93cad2677613b0a31315e428ead5339612a4b1c (patch) | |
| tree | bc78b1d7300f611929b4978920ac10b66fe49ebd /extent-tree.c | |
| parent | bee9403728ce017c0b9a8384a0bcb342b331688a (diff) | |
| download | btrfs-progs-d93cad2677613b0a31315e428ead5339612a4b1c.tar.gz | |
Btrfs-progs: add a free space cache checker to fsck V2
In trying to track down a weird tree log problem I wanted to make sure that the
free space cache was actually valid, which we currently have no way of doing.
So this patch adds a bunch of support for the free space cache code and then a
checker to fsck. Basically we go through and if we can actually load the free
space cache then we will walk the extent tree and verify that the free space
cache exactly matches what is in the extent tree. Hopefully this will always be
correct, the only time it wouldn't is if the extent tree is corrupt or we have
some sort of awful bug in the free space cache. Thanks,
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'extent-tree.c')
| -rw-r--r-- | extent-tree.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/extent-tree.c b/extent-tree.c index ed4756f..381572d 100644 --- a/extent-tree.c +++ b/extent-tree.c @@ -26,6 +26,7 @@ #include "transaction.h" #include "crc32c.h" #include "volumes.h" +#include "free-space-cache.h" #define BLOCK_GROUP_DATA EXTENT_WRITEBACK #define BLOCK_GROUP_METADATA EXTENT_UPTODATE @@ -3173,6 +3174,7 @@ out: int btrfs_free_block_groups(struct btrfs_fs_info *info) { struct btrfs_space_info *sinfo; + struct btrfs_block_group_cache *cache; u64 start; u64 end; u64 ptr; @@ -3184,8 +3186,14 @@ int btrfs_free_block_groups(struct btrfs_fs_info *info) if (ret) break; ret = get_state_private(&info->block_group_cache, start, &ptr); - if (!ret) - kfree((void *)(unsigned long)ptr); + if (!ret) { + cache = (struct btrfs_block_group_cache *)ptr; + if (cache->free_space_ctl) { + btrfs_remove_free_space_cache(cache); + kfree(cache->free_space_ctl); + } + kfree(cache); + } clear_extent_bits(&info->block_group_cache, start, end, (unsigned int)-1, GFP_NOFS); } |