[PATCH] Btrfs: do not BUG if we fail to remove the orphan item for dead snapshots UPDATED

2010-12-10 Thread Josef Bacik
Not being able to delete an orphan item isn't a horrible thing. The worst that happens is the next time around we try and do the orphan cleanup and we can't find the referenced object and just delete the item and move on. Thanks, Signed-off-by: Josef Bacik --- V1->V2: Added a comment to explain

Re: [PATCH] Btrfs: do not BUG if we fail to remove the orphan item for dead snapshots

2010-12-10 Thread Chris Mason
Excerpts from Josef Bacik's message of 2010-12-10 08:30:36 -0500: > On Fri, Dec 10, 2010 at 03:16:15PM +0800, Li Zefan wrote: > > 01:25, Josef Bacik wrote: > > > Not being able to delete an orphan item isn't a horrible thing. The > > > worst that > > > happens is the next time around we try and d

Re: [PATCH] Btrfs: do not BUG if we fail to remove the orphan item for dead snapshots

2010-12-10 Thread Josef Bacik
On Fri, Dec 10, 2010 at 03:16:15PM +0800, Li Zefan wrote: > 01:25, Josef Bacik wrote: > > Not being able to delete an orphan item isn't a horrible thing. The worst > > that > > happens is the next time around we try and do the orphan cleanup and we > > can't > > find the referenced object and ju

Re: [PATCH] Btrfs: do not BUG if we fail to remove the orphan item for dead snapshots

2010-12-09 Thread Li Zefan
01:25, Josef Bacik wrote: > Not being able to delete an orphan item isn't a horrible thing. The worst > that > happens is the next time around we try and do the orphan cleanup and we can't > find the referenced object and just delete the item and move on. Thanks, > Would be better to add code

[PATCH] Btrfs: do not BUG if we fail to remove the orphan item for dead snapshots

2010-12-08 Thread Josef Bacik
Not being able to delete an orphan item isn't a horrible thing. The worst that happens is the next time around we try and do the orphan cleanup and we can't find the referenced object and just delete the item and move on. Thanks, Signed-off-by: Josef Bacik --- fs/btrfs/extent-tree.c |1 -