diff options
| author | Josef Bacik <jbacik@fusionio.com> | 2013-05-03 21:00:52 +0000 |
|---|---|---|
| committer | Chris Mason <chris.mason@fusionio.com> | 2013-05-06 13:41:58 -0400 |
| commit | c6eb10a7a43390e975b3f63fdf32cea58af948ea (patch) | |
| tree | 426bf42d3f9523e8005786450ec4db4092d212aa /cmds-check.c | |
| parent | 9c821327408803229e93a788e032e8e9caf11686 (diff) | |
Btrfs-progs: init free space ctl with proper unit
btrfsck was blowing up when checking the free space cache when we ran xfstests
with -l 64k. That is because I was init'ing the free space ctl to whatever the
leafsize was, which isn't right for data block groups. With this patch btrfsck
no longer complains. This also fixes a tiny little typo in free-space-cache.c I
noticed while figuring this problem out. Thanks,
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'cmds-check.c')
| -rw-r--r-- | cmds-check.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/cmds-check.c b/cmds-check.c index 9fd53f42..71f59595 100644 --- a/cmds-check.c +++ b/cmds-check.c @@ -2799,8 +2799,15 @@ static int check_space_cache(struct btrfs_root *root) start = cache->key.objectid + cache->key.offset; if (!cache->free_space_ctl) { - if (btrfs_init_free_space_ctl(cache, - root->leafsize)) { + int sectorsize; + + if (cache->flags & (BTRFS_BLOCK_GROUP_METADATA | + BTRFS_BLOCK_GROUP_SYSTEM)) + sectorsize = root->leafsize; + else + sectorsize = root->sectorsize; + + if (btrfs_init_free_space_ctl(cache, sectorsize)) { ret = -ENOMEM; break; } |