On Tue, May 19, 2020 at 08:40:17AM +0200, Christoph Hellwig wrote: > On Mon, May 18, 2020 at 06:23:37PM -0700, Darrick J. Wong wrote: > > On Mon, May 18, 2020 at 07:04:31PM +0200, Christoph Hellwig wrote: > > > Hi all, > > > > > > this series cleans up a bunch of lose ends in the inode iterator > > > functions. > > > > Funny, I was going to send a series next cycle that refactors a fair > > amount of the incore inode iterators and then collapses the > > free_cowblocks/free_eofblocks code into a single worker and radix tree > > tag that takes care of all the speculative preallocation extent gc > > stuff... > > > > incore walks: > > https://lore.kernel.org/linux-xfs/157784075463.1360343.1278255546758019580.stgit@magnolia/ > > Except for the last few those seem non-brainers tat would could merge > ASAP.. > > The rest also doesn't look bad, so I'll happily defer this series. > OTOH the first on really should go in, either your or my version :) Well, I'll send the set for a formal review... :) --D