lkml.org 
[lkml]   [2015]   [Apr]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [GIT PULL] kdbus for 4.1-rc1
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.

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?

For something that's potentially such a core mechanism as a completely
new, massively-adopted IPC, this does send a warning singal.

Thanks,

--
Jiri Kosina
SUSE Labs


\
 
 \ /
  Last update: 2015-04-15 11:21    [W:0.233 / U:2.692 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site