On Tue, 14 May 2013 18:44:11 +0800, Liu Bo wrote: > On Tue, May 14, 2013 at 11:36:56AM +0200, Stefan Behrens wrote: >> @@ -396,7 +403,7 @@ static noinline int create_subvol(struct inode *dir, >> * of create_snapshot(). >> */ >> ret = btrfs_subvolume_reserve_metadata(root, &block_rsv, >> - 7, &qgroup_reserved); >> + 8, &qgroup_reserved); >> if (ret) >> return ret; > > This uuid_root will not use trans->block_rsv but empty_rsv since it does not > set > ref_cow, so you don't need to add one more to block_rsv, and same for > the below cases.
Hi Liu Bo, Thanks for your review comments! You are right, this won't work because the empty_block_rsv is used for the UUID tree as it is now. I need to avoid ENOSPC in the middle of the transaction. Can you please acknowledge or comment the following addition to get_block_rsv()? And the "plus 1" that I have added to the transaction reservations for subvolume/snapshot creation/destruction will work correctly afterwards. --- a/fs/btrfs/extent-tree.c +++ b/fs/btrfs/extent-tree.c @@ -4259,6 +4259,9 @@ static struct btrfs_block_rsv *get_block_rsv( if (root == root->fs_info->csum_root && trans->adding_csums) block_rsv = trans->block_rsv; + if (root == root->fs_info->uuid_root) + block_rsv = trans->block_rsv; + if (!block_rsv) block_rsv = root->block_rsv; >> @@ -567,9 +578,10 @@ static int create_snapshot(struct btrfs_root *root, >> struct inode *dir, >> * 1 - root item >> * 2 - root ref/backref >> * 1 - root of snapshot >> + * 1 - UUID item >> */ >> ret = btrfs_subvolume_reserve_metadata(BTRFS_I(dir)->root, >> - &pending_snapshot->block_rsv, 7, >> + &pending_snapshot->block_rsv, 8, >> &pending_snapshot->qgroup_reserved); >> if (ret) >> goto out; -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html