On Thu, Sep 16, 2021 at 01:11:21PM -0400, James Bottomley wrote: > > Actually, I don't see who should ack being an unknown. The MAINTAINERS > file covers most of the kernel and a set of scripts will tell you based > on your code who the maintainers are ... that would seem to be the > definitive ack list. It's *really* not that simple. It is *not* the case that if a change touches a single line of fs/ext4 (as well as 60+ other filesystems), for example: - ei = kmem_cache_alloc(ext4_inode_cachep, GFP_NOFS); + ei = alloc_inode_sb(sb, ext4_inode_cachep, GFP_NOFS); that the submitter *must* get a ACK from me --- or that I am entitled to NACK the entire 79 patch series for any reason I feel like, or to withhold my ACK as hostage until the submitter does some development work that I want. What typically happens is if someone were to try to play games like this inside, say, the Networking subsystem, past a certain point, David Miller will just take the patch series, ignoring people who have NACK's down if they can't be justified. The difference is that even though Andrew Morton (the titular maintainer for all of Memory Management, per the MAINTAINERS file), Andrew seems to have a much lighter touch on how the mm subsystem is run. > I think the problem is the ack list for features covering large areas > is large and the problems come when the acker's don't agree ... some > like it, some don't. The only deadlock breaking mechanism we have for > this is either Linus yelling at everyone or something happening to get > everyone into alignment (like an MM summit meeting). Our current model > seems to be every acker has a foot on the brake, which means a single > nack can derail the process. It gets even worse if you get a couple of > nacks each requesting mutually conflicting things. > > We also have this other problem of subsystems not being entirely > collaborative. If one subsystem really likes it and another doesn't, > there's a fear in the maintainers of simply being overridden by the > pull request going through the liking subsystem's tree. This could be > seen as a deadlock breaking mechanism, but fear of this happening > drives overreactions. > > We could definitely do a clear definition of who is allowed to nack and > when can that be overridden. Well, yes. And this is why I think there is a process issue here that *is* within the MAINTAINERS SUMMIT purview, and if we need to technical BOF to settle the specific question of what needs to happen, whether it happens at LPC, or it needs to happen after LPC, then let's have it happen. I'd be really disappointed if we have to wait until December 2022 for the next LSF/MM, and if we don't get consensus there, ala DAX, that we then have to wait until late 2023, etc. As others have said, this is holding up some work that file system developers would really like to see. - Ted
