> You are changing core kernel code without ACKs from relevant > maintainers. That's very unfortunate. Existing users certainly matter > but that doesn't mean you get to merge code without maintainers even > looking at it. > > So really, why don't you just use scripts/get_maintainer.pl and simply > ask the relevant people for their ACK? Actually I had done that before posting the patches and, doing it now again, I *do* have many of the relevant people on the ack list, and nearly all on the cc list of the patch postings. (I apologize that I see I missed you on my list.) I think every relevant maintainer has had the chance to review and acknowledge but some have, for whatever reason, chosen not to. > Looking at your patches, there's no trace that anyone outside your own > development team even looked at the patches. Hmmm... I have reviews/acks from IBM, Fujitsu, and Citrix (and a long list of documented Cc's) in the git comments, so I'm not sure what you are seeing. Ah, perhaps you are referring to the naming changes in the cleancache hooks? Akpm required me to rename various frontswap hooks to use "invalidate" in the function name instead of "flush". I took the opportunity to rename the cleancache hooks for consistency in this same patchset and this occurred in only in the most recent version of the patchset. It is true that I didn't ask for Ack's from those maintainers, though these changes would probably have gone through the trivial patch monkey later anyway. > Why do you feel that it's OK to ask Linus to pull them? Frontswap is essentially the second half of the cleancache patchset (or, more accurately, both are halves of the transcendent memory patchset). They are similar in that the hooks in core MM code are fairly trivial and the real value/functionality lies outside of the core kernel; as a result core MM maintainers don't have much interest I guess. Linus personally merged cleancache for 3.0 (quoting from his offlist email: "I've looked through it, and it seems simple enough, with a pretty minimal support burden"); I was assuming a similar path for frontswap. I repeat that I'm not trying to subvert any process. There just doesn't seem to be much of a process in place for this kind of a patchset, and I'm not letting silence or indifference or "don't like it much" get in the way. Thanks, Dan -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href