Btrfs: only set cache_generation if we setup the block group
authorJosef Bacik <josef@redhat.com>
Tue, 13 Dec 2011 21:04:54 +0000 (16:04 -0500)
committerJosef Bacik <josef@redhat.com>
Thu, 15 Dec 2011 16:04:24 +0000 (11:04 -0500)
A user reported a problem booting into a new kernel with the old format inodes.
He was panicing in cow_file_range while writing out the inode cache.  This is
because if the block group is not cached we'll just skip writing out the cache,
however if it gets dirtied again in the same transaction and it finished caching
we'd go ahead and write it out, but since we set cache_generation to the transid
we think we've already truncated it and will just carry on, running into
cow_file_range and blowing up.  We need to make sure we only set
cache_generation if we've done the truncate.  The user tested this patch and
verified that the panic no longer occured.  Thanks,

Reported-and-Tested-by: Klaus Bitto <klaus.bitto@gmail.com>
Signed-off-by: Josef Bacik <josef@redhat.com>
fs/btrfs/extent-tree.c

index 4eb4d2748bec27eaab3371f74d6a5fb6ef5f61b7..8603ee4e3dfd75fb96db37b5f12deb8deb043838 100644 (file)
@@ -2822,7 +2822,7 @@ out_free:
        btrfs_release_path(path);
 out:
        spin_lock(&block_group->lock);
-       if (!ret)
+       if (!ret && dcs == BTRFS_DC_SETUP)
                block_group->cache_generation = trans->transid;
        block_group->disk_cache_state = dcs;
        spin_unlock(&block_group->lock);