Re: [PATCH] btrfs:Change BUG_ON to new error path in __clear_extent_bit

2016-04-06 Thread Bastien Philbert
On 2016-04-06 12:10 PM, Filipe Manana wrote: > On Wed, Apr 6, 2016 at 4:56 PM, Bastien Philbert > wrote: >> This remove the unnessary BUG_ON if the allocation with >> alloc_extent_state_atomic fails due to this function >> failure not being unrecoverable. Instead we

Re: [PATCH] btrfs:Change BUG_ON to new error path in __clear_extent_bit

2016-04-06 Thread Bastien Philbert
On 2016-04-06 12:10 PM, Filipe Manana wrote: > On Wed, Apr 6, 2016 at 4:56 PM, Bastien Philbert > wrote: >> This remove the unnessary BUG_ON if the allocation with >> alloc_extent_state_atomic fails due to this function >> failure not being unrecoverable. Instead we now change >> this BUG_ON

Re: [PATCH] btrfs:Change BUG_ON to new error path in __clear_extent_bit

2016-04-06 Thread Filipe Manana
On Wed, Apr 6, 2016 at 4:56 PM, Bastien Philbert wrote: > This remove the unnessary BUG_ON if the allocation with > alloc_extent_state_atomic fails due to this function > failure not being unrecoverable. Instead we now change > this BUG_ON into a new error path that

Re: [PATCH] btrfs:Change BUG_ON to new error path in __clear_extent_bit

2016-04-06 Thread Filipe Manana
On Wed, Apr 6, 2016 at 4:56 PM, Bastien Philbert wrote: > This remove the unnessary BUG_ON if the allocation with > alloc_extent_state_atomic fails due to this function > failure not being unrecoverable. Instead we now change > this BUG_ON into a new error path that jumps to the goto > label, out

[PATCH] btrfs:Change BUG_ON to new error path in __clear_extent_bit

2016-04-06 Thread Bastien Philbert
This remove the unnessary BUG_ON if the allocation with alloc_extent_state_atomic fails due to this function failure not being unrecoverable. Instead we now change this BUG_ON into a new error path that jumps to the goto label, out from freeing previously allocated resources before returning the

[PATCH] btrfs:Change BUG_ON to new error path in __clear_extent_bit

2016-04-06 Thread Bastien Philbert
This remove the unnessary BUG_ON if the allocation with alloc_extent_state_atomic fails due to this function failure not being unrecoverable. Instead we now change this BUG_ON into a new error path that jumps to the goto label, out from freeing previously allocated resources before returning the