netdev - Re: [PATCH] new UDPCP Communication Protocol

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1294782117.17531.18.camel@wall-e>
Date:	Tue, 11 Jan 2011 22:41:57 +0100
From:	Stefani Seibold <stefani@...bold.net>
To:	David Miller <davem@...emloft.net>
Cc:	eric.dumazet@...il.com, linux-kernel@...r.kernel.org,
	akpm@...ux-foundation.org, netdev@...r.kernel.org,
	shemminger@...tta.com, jj@...osbits.net, daniel.baluta@...il.com,
	jochen@...hen.org, hagen@...u.net, torvalds@...ux-foundation.org,
	pavel@....cz
Subject: Re: [PATCH] new UDPCP Communication Protocol

Am Dienstag, den 11.01.2011, 13:19 -0800 schrieb David Miller:
> From: Stefani Seibold <stefani@...bold.net>
> Date: Tue, 11 Jan 2011 22:14:40 +0100
> 
> > 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.

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html