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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to