From: Doug Ledford <dledford@xxxxxxxxxx> Date: Tue, 16 May 2017 11:57:04 -0400 > Regardless though, I'm rather purturbed about this entire thing. If > you are right that because this got into 4.11, it's now a done deal, > then the fact that this went through 4 review cycles on netdev@ that, > as I understand it, spanned roughly one years time, and not one single > person bothered to note that this was as much an RDMA driver as > anything else, and not one person bothered to note that linux-rdma@ was > not on the Cc: list, and not one person told the submitters that they > needed to include linux-rdma@ on the Cc: list of these submissions, and > you took it without any review comments from any RDMA people in the > course of a year, or an ack from me to show that the RDMA portion of > this had at least been given some sort of review, was a collosal fuckup > of cross tree maintainer cooperation. We rely on people from various areas of expertiece to contribute to patch review on netdev and give appropriate feedback. If you actually look through the history, I made many semantic reviews of the SMC changes, and kept pushing back. And in fact I did this several times, making them go through several revisions, in the hopes that someone would review more of the meat and substance of the patch set. Nobody do this for over a year. I can't push back on people with silly coding style and small semantic issues forever. And I think I made a serious effort to keep the patches getting posted over and over again to make sure they got more exposure. I think it's unsettling that there are no RDMA experts, or at least people remotely knowledgable about this "networking" technology, subscribed to netdev and taking a cursory look at pactches that might be relevant and effect that technology either directly or indirectly. So there is a lot of blame to go around. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
