On 06/27/2014 05:05 PM, Josef Bacik wrote:
Mark noticed that his qgroup accounting for snapshot deletion wasn't working
properly on a particular file system. Turns out we pass the root->objectid of
the root we are deleting to btrfs_free_extent, and use that root always when we
call btrfs_free_tree_block. This isn't correct, the owner must match the
btrfs_header_owner() of the eb. So to fix this we need to use that when we call
btrfs_free_extent, and we also need to use btrfs_header_owner(eb) in
btrfs_free_tree_block as the root we pass in may not be the owner in the case of
snapshot delete (though it is for all the normal cases which is why it wasn't
noticed before.) With this patch on top of Mark's snapshot delete patch
everything is working a-ok. Thanks,
Signed-off-by: Josef Bacik <jba...@fb.com>
---
V1->V2: this one actually compiles.
Huh I may be completely full of crap here, let's just ignore all post 5pm
Friday patches from me for now. Thanks,
Josef
--
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
