On Mon, Jul 01, 2019 at 10:01:59AM -0700, Darrick J. Wong wrote: > Hi all, > > This series breaks up fs/iomap.c by grouping the functions by major > functional area (swapfiles, fiemap, seek hole/data, directio, buffered > writes, buffered reads, page management, and page migration) in separate > source code files under fs/iomap/. No functional changes have been > made. > > Note that this is not the final format of the patches, because I intend > to pick a point towards the end of the merge window (after everyone > else's merges have landed), rebase this series atop that, and push it > back to Linus. The RFC is posted so that everyone can provide feedback > on the grouping strategy, not line-specific code movements. > > This has been lightly tested with fstests. Enjoy! > Comments and questions are, as always, welcome. Do you have a branch somewhere for the layout? To me it seems to be a little too fine grained and creates tons of tiny files, which make hacking the code painful.