netdev - Re: [PATCH 00/26] IPVS: Add first IPv6 support to IPVS.

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]

Date:	Wed, 11 Jun 2008 20:23:45 +0200
From:	"Julius Volz" <juliusv@...gle.com>
To:	"Patrick McHardy" <kaber@...sh.net>
Cc:	lvs-devel@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH 00/26] IPVS: Add first IPv6 support to IPVS.

On Wed, Jun 11, 2008 at 7:23 PM, Patrick McHardy <kaber@...sh.net> wrote:
> I briefly looked over the patches I didn't comment on.
> I think there's too much duplication everywhere, a lot of
> them look like they could avoid almost all duplication by
> handling differences at a higher layer or simply sharing
> the code (like hashing).

Yes, the duplication is high unfortunately. I must admit that I didn't
feel secure enough to restructure all the existing code without
breaking it, so I copied lots of functions and modified them for IPv6.
My main goal was to keep all the old v4 stuff working first and then
remove the duplication later (or hope for smarter people).

So I obviously don't expect this to be ready for inclusion, but I will
have a lot of time to work on it (I'm doing it as an intern project)
and learn as long as I get good feedback like yours on what to
improve.

Another question I was unsure about: is the breaking of the
userspace-to-kernel interface even acceptable at all? I think the code
would get ugly (and have even more duplication) if you wanted to keep
the backwards compatibility. And you have to compile ipvsadm for your
kernel version anyways.

Julius

-- 
Google Switzerland GmbH
--
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