Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754060AbbDOJKB (ORCPT ); Wed, 15 Apr 2015 05:10:01 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:33016 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752201AbbDOJJw (ORCPT ); Wed, 15 Apr 2015 05:09:52 -0400 Date: Wed, 15 Apr 2015 11:09:48 +0200 From: Greg Kroah-Hartman To: Jiri Kosina Cc: Borislav Petkov , Al Viro , Andy Lutomirski , Linus Torvalds , Andrew Morton , Arnd Bergmann , "Eric W. Biederman" , One Thousand Gnomes , Tom Gundersen , "linux-kernel@vger.kernel.org" , Daniel Mack , David Herrmann , Djalal Harouni Subject: Re: [GIT PULL] kdbus for 4.1-rc1 Message-ID: <20150415090948.GA17347@kroah.com> References: <20150414175019.GA2874@kroah.com> <20150414192357.GA6107@kroah.com> <20150414192429.GC26075@pd.tnic> <20150414193229.GB6107@kroah.com> <20150414194004.GG889@ZenIV.linux.org.uk> <20150414194804.GB7540@kroah.com> <20150414195336.GG14069@pd.tnic> <20150415084440.GF16381@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1921 Lines: 46 On Wed, Apr 15, 2015 at 10:54:41AM +0200, Jiri Kosina wrote: > On Wed, 15 Apr 2015, Greg Kroah-Hartman wrote: > > > If you have a technical reason for why this code shouldn't be merged, > > great, please let me know and we can work to address that. Andy and Al > > have spent time reviewing and giving us comments, and that's wonderful > > and valuable and is why I treat their comments seriously. If you are > > interested in the code, please review it, otherwise I don't see what > > this adds to the conversation at all, do you? > > You've actually touched another issue I see here, and that is -- the code > is complex like crazy. > > I've spent big part of past two days trying to get my head around it, but > I am still far away from getting at least the 1000 miles overview of how > exactly the message passing is designed. > > I understand that the primary reason for this complexity is probably the > dbus protocol specification itself. Yes it is. > But the problem really is that I don't think you've received even a single > Reviewed-by: from someone who hasn't been directly involved in developing > the code, right? I've asked for it, but finding people to review code is hard, as you know. It's only 13k lines long, smaller than a serial port driver (my unit of code review), so it's not all that big. It's smaller than the USB3 host controller driver as well, and very few people ever reviewed that beast :) > For something that's potentially such a core mechanism as a completely > new, massively-adopted IPC, this does send a warning singal. If you know of a way to force others to review code, please let me know. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/