On Mon, November 04, 2013 at 18:42 (+0100), Josef Bacik wrote: > On Thu, Oct 24, 2013 at 03:22:06PM +0200, Jan Schmidt wrote: >> btrfs_dec_ref() queued a delayed ref for owner of a tree block. The qgroup >> tracking is based on delayed refs. The owner of a tree block is set when a >> tree block is allocated, it is never updated. >> >> When you allocate a tree block and then remove the subvolume that did the >> allocation, the qgroup accounting for that removal is correct. However, the >> removal was accounted again for each subvolume deletion that also referenced >> the tree block, because accounting was erroneously based on the owner. >> >> Instead of queueing delayed refs for the non-existent owner, we now >> queue delayed refs for the root being removed. This fixes the qgroup >> accounting. >> >> Signed-off-by: Jan Schmidt <list.btrfs@xxxxxxxxxxxxx> >> Tested-by: <dustymabe@xxxxxxxxx> > > This breaks btrfs/003, I'm kicking it out. Can you be a bit more specific? Works fine here. -Jan -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
