On Thu, May 04, 2017 at 04:59:43AM -0700, Christoph Hellwig wrote: > On Tue, May 02, 2017 at 08:05:25AM -0400, Brian Foster wrote: > > But doesn't that mean this wouldn't be a new problem introduced by such > > a change? > > It would make the existing problem at lot easier to hit I think. > I don't really see how. As it is, we currently do something like unmap extent 1, unmap extent 2, roll transaction, free extent 1, free extent 2 (where the extent frees lock the agf). As I understand it, the prospective change would move the agf locking up into the unmap transaction and hold it across the roll. I suppose you could make the case that the AGFs are held locked for a longer period of time, but it's hard to characterize whether that would make a deadlock more likely or not. Even if it did, couldn't we either just fix the unmap count to one extent (unconditionally or in just the reflink case) or until we hit an extent that is in decreasing ag order from one that has already been unmapped in the current tp..? > > transaction. That being said, I'm not following your thought wrt to this > > particular situation. Are you suggesting that we not defer the reflink > > adjustment in particular unmap cases, or that we just limit the number > > of extent unmaps per-tp based on crossing an AG boundary, or something > > else entirely? > > To me it seems like we should try to do the extent count adjustments > in the current transaction for a given extent if we can, but give me > a little more time to think how to best do that. I'm travelling at the > moment and don't have much quiet time to actually engage my brain. Sure, that sounds potentially interesting. Brian > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
