Re: [PATCH] Move IPVS from net/ipv4/ipvs to net/netfilter/ipvs in preparation for IPv6 support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Friday 09 May 2008 21:49:52 Joseph Mack NA3T wrote:
> 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?

I haven't applied the patch and done a diff to individual files to be certain, but I'm pretty sure that the patch is mainly just a directory change. IPVS itself is already fairly well abstracted protocol-wise. Just another guess, but it sounds like Julius is submitting the patch as it would make more meaty patches smaller.

> 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).

Call me crazy, but I plan to put my patch into production in two weeks time without the connect-to-vip-from-director parts. If everything goes well, I'll post patches again with adequate comments (and adjustments to comments). And to make this relevant: My patches don't do anything protocol specific other than to change when things are done and so would be fairly easy to update for IPV6 if/when it is integrated.

-- 
Jason Stubbs <j.stubbs@xxxxxxxxxxxxxxx>
LINKTHINK INC.
東京都渋谷区桜ヶ丘町22-14 N.E.S S棟 3F
TEL 03-5728-4772  FAX 03-5728-4773
--
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

[Index of Archives]     [Linux Filesystem Devel]     [Linux NFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]     [X.Org]

  Powered by Linux