Thread (12 messages) 12 messages, 3 authors, 2011-01-11

Re: [PATCH] new UDPCP Communication Protocol

flat view

From: Stefani Seibold <stefani@seibold.net>
Date: 2011-01-11 21:41:02
Also in: lkml

Am Dienstag, den 11.01.2011, 13:19 -0800 schrieb David Miller:
From: Stefani Seibold <stefani@seibold.net>
Date: Tue, 11 Jan 2011 22:14:40 +0100
quoted
If nobody need it and no user in the near future out there, why should i
implement this? That is dogmatic only!
It's a hard requirement, sorry.

And I want you to do it especially because it shows clearly how poor
your implementation is, with all of it's code duplication.
First it is not so much code duplication. It it less than 20 percent of
the whole code. And most of this code was adapted to the need of the
protocol.

Second, the design is may in your opinion poor. I like it. What is
really poor is the kernel_...() socket functions, which are only simple
wrapper of the system calls without any performance improvement, skb
support and memory saving.

IPv6 would not very hard to implement and will be done if i get an go. 
You'll need yet another copy of all of this code to support ipv6.

Please implement this properly, and in doing so the ipv6 support will
be very simple if not trivial.
The implementation is clean and fast, it has absolut no side effect. It
is save to merge and all requirement was solved.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help