On Thu, Jun 28, 2018 at 12:36:12PM -0400, Brian Foster wrote: > I think these issues can be mostly resolved by the end goal of a > combined xfs_trans/xfs_defer_ops structure. Boilerplate code that > allocates a transaction, inits a dfops, finishes a dfops and commits the > transaction can then reduce to allocating and committing the transaction > (where the former and latter respectively init/finish an internal > dfops). There are other patterns and issues to consider, however, so > that will be a matter for a subsequent patch series. (I'm actually > thinking of cleaning up all of the firstblock stuff as a next step in > that direction.) Yes, I actually did a pass at firstblock a while ago when looking into the allocation reservation issues. It cleaned things up a lot, but I ended up dropping it as it didn't actually end up helping to solve the problem directly. -- 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