Re: [PATCH] btrfs-progs: repair: Trickle down EEXISTS while freeing

2016-10-31 Thread David Sterba
On Mon, Oct 31, 2016 at 01:43:20PM -0500, Goldwyn Rodrigues wrote: > Hi David, > > On 10/28/2016 09:42 AM, David Sterba wrote: > > On Thu, Oct 27, 2016 at 01:05:08PM -0500, Goldwyn Rodrigues wrote: > >> While deleting pending extents, we insert extents to the > >> extent_tree. However, because

Re: [PATCH] btrfs-progs: repair: Trickle down EEXISTS while freeing

2016-10-31 Thread Goldwyn Rodrigues
Hi David, On 10/28/2016 09:42 AM, David Sterba wrote: > On Thu, Oct 27, 2016 at 01:05:08PM -0500, Goldwyn Rodrigues wrote: >> While deleting pending extents, we insert extents to the >> extent_tree. However, because of corruption, this might already be >> freed. We trickle the EEXISTS error and

Re: [PATCH] btrfs-progs: repair: Trickle down EEXISTS while freeing

2016-10-28 Thread David Sterba
On Fri, Oct 28, 2016 at 10:02:06AM -0500, Goldwyn Rodrigues wrote: > > > On 10/28/2016 09:42 AM, David Sterba wrote: > > On Thu, Oct 27, 2016 at 01:05:08PM -0500, Goldwyn Rodrigues wrote: > >> While deleting pending extents, we insert extents to the > >> extent_tree. However, because of

Re: [PATCH] btrfs-progs: repair: Trickle down EEXISTS while freeing

2016-10-28 Thread Goldwyn Rodrigues
On 10/28/2016 09:42 AM, David Sterba wrote: > On Thu, Oct 27, 2016 at 01:05:08PM -0500, Goldwyn Rodrigues wrote: >> While deleting pending extents, we insert extents to the >> extent_tree. However, because of corruption, this might already >> be freed. We trickle the EEXISTS error and let the

Re: [PATCH] btrfs-progs: repair: Trickle down EEXISTS while freeing

2016-10-28 Thread David Sterba
On Thu, Oct 27, 2016 at 01:05:08PM -0500, Goldwyn Rodrigues wrote: > While deleting pending extents, we insert extents to the > extent_tree. However, because of corruption, this might already > be freed. We trickle the EEXISTS error and let the caller > decide what needs to be done with it. In

[PATCH] btrfs-progs: repair: Trickle down EEXISTS while freeing

2016-10-27 Thread Goldwyn Rodrigues
While deleting pending extents, we insert extents to the extent_tree. However, because of corruption, this might already be freed. We trickle the EEXISTS error and let the caller decide what needs to be done with it. In this case, ignore, so the repair may proceed. The primary motivation of this