[PATCH 32/42] btrfs: only free reserved extent if we didn't insert it

2018-10-12 Thread Josef Bacik
When we insert the file extent once the ordered extent completes we free the reserved extent reservation as it'll have been migrated to the bytes_used counter. However if we error out after this step we'll still clear the reserved extent reservation, resulting in a negative accounting of the

Re: [PATCH 32/42] btrfs: only free reserved extent if we didn't insert it

2018-10-11 Thread Filipe Manana
On Thu, Oct 11, 2018 at 8:58 PM Josef Bacik wrote: > > When we insert the file extent once the ordered extent completes we free > the reserved extent reservation as it'll have been migrated to the > bytes_used counter. However if we error out after this step we'll still > clear the reserved

[PATCH 32/42] btrfs: only free reserved extent if we didn't insert it

2018-10-11 Thread Josef Bacik
When we insert the file extent once the ordered extent completes we free the reserved extent reservation as it'll have been migrated to the bytes_used counter. However if we error out after this step we'll still clear the reserved extent reservation, resulting in a negative accounting of the

[PATCH 32/42] btrfs: only free reserved extent if we didn't insert it

2018-09-28 Thread Josef Bacik
When we insert the file extent once the ordered extent completes we free the reserved extent reservation as it'll have been migrated to the bytes_used counter. However if we error out after this step we'll still clear the reserved extent reservation, resulting in a negative accounting of the