[PATCH 11/47] xfs: move deferred operations into a separate file

Christoph Hellwig hch at infradead.org
Mon Aug 1 03:02:23 CDT 2016


I looked over this again and I really don't see the use case of merging
it.  Yes, the freed extent, rmap and reflink code is fairly similar, but
there is all kinds of subtile differences that we need to paper over using
methods and flags.  I think we're better off not trying to share this
code and have a separate, but easily understandable implementation
for each btree.  At least for the traditional traditional freed extent
case the new code also is a lot less optimal than the previous version.



More information about the xfs mailing list