Re: [PATCH] Btrfs: make sure to use btrfs_header_owner when freeing tree block V2

[Josef Bacik <jbacik@fb.com>]

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 <jbacik@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