Re: [PATCH] btrfs: raid56: avoid double allocation

2013-07-25 Thread Miao Xie
On thu, 25 Jul 2013 16:18:41 +0800, Liu Bo wrote: > On Thu, Jul 25, 2013 at 02:09:40AM +0200, Andi Shyti wrote: >> alloc_rbio frees already bbio and raid_map, therefore they don't >> need to be freed in case it fails >> > > Looks good to me. > > Reviewed-by: Liu Bo Dan Carpenter has fixed

Re: [PATCH] btrfs: raid56: avoid double allocation

2013-07-25 Thread Liu Bo
On Thu, Jul 25, 2013 at 02:09:40AM +0200, Andi Shyti wrote: > alloc_rbio frees already bbio and raid_map, therefore they don't > need to be freed in case it fails > Looks good to me. Reviewed-by: Liu Bo > Signed-off-by: Andi Shyti > --- > fs/btrfs/raid56.c | 6 ++ > 1 file changed, 2 ins

[PATCH] btrfs: raid56: avoid double allocation

2013-07-24 Thread Andi Shyti
alloc_rbio frees already bbio and raid_map, therefore they don't need to be freed in case it fails Signed-off-by: Andi Shyti --- fs/btrfs/raid56.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/fs/btrfs/raid56.c b/fs/btrfs/raid56.c index 0525e13..35fe775 100644 --- a/f