On Fri, Apr 28, 2017 at 12:40:32PM -0700, Darrick J. Wong wrote: > So either we have to keep the AGF buffer locked and attached across > all the transaction rolls until we around to processing the refcount > decrement deferred op, or figure out something else clever...? We can't. We need to unlock the AGF buffer over rolls because we must not lock two AGF buffers unless we can ensure they are in increasing agno order. For the alloc path we go great length to ensure this, but we will also need to do that for the free path, which currently is a problem due to the two extents we free per transaction. Maybe we need to move away from the defer everything in one defer_ops scheme to target defers instead so that we can keep the AGF locked where needed and release it when we can't keep it locked. -- 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