LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [PATCH] Move IPVS from net/ipv4/ipvs to net/netfilter/ipvs in prepar

To: Julius Volz <juliusv@xxxxxxxxxx>
Subject: Re: [PATCH] Move IPVS from net/ipv4/ipvs to net/netfilter/ipvs in preparation for IPv6 support
Cc: netdev@xxxxxxxxxxxxxxx, lvs-devel@xxxxxxxxxxxxxxx, Horms <horms@xxxxxxxxxxxx>
From: Joseph Mack NA3T <jmack@xxxxxxxx>
Date: Fri, 9 May 2008 05:49:52 -0700 (PDT)
On Wed, 7 May 2008, Julius Volz wrote:

Hm, ok. This patch doesn't touch any functionality though. It only
moves files from one directory to another, so it's merely cosmetics,
no IPv6 knowledge required.

are you at a stage where it's all split up, after which you'll only be working on the ipv6 part and leaving the ipv4 untouched? Have you tested the ipv4 part to make sure it's still OK. ie the code is reorthogonalised but with no change in functionality?

If so then this might be a good point to submit to the kernel. Also Jason might be interested in the new version of the code (hopefully working in different parts of the code, you two won't be having many collisions).

One of the things I worry about (from personal experience) is people working on projects and getting stuck for time and later having to drop the work. If you can submit the reorthogonalised code, and have to stop sometime later, at least we've got that far down the road.

Horms,
Do you want ipvs() reorthogonalised into ipv4/ipv6 blocks, presumably with the ipv4 functionality untouched, and with ipv6 stubs, for submission to the kernel, or do you want to wait for the finished product?

Joe

--
Joseph Mack NA3T EME(B,D), FM05lw North Carolina
jmack (at) wm7d (dot) net - azimuthal equidistant map
generator at http://www.wm7d.net/azproj.shtml
Homepage http://www.austintek.com/ It's GNU/Linux!
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>