LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [v2 PATCH 0/4] IPVS: Backup Adding Ipv6 and Persistence support

To: Hans Schillstrom <hans@xxxxxxxxxxxxxxx>
Subject: Re: [v2 PATCH 0/4] IPVS: Backup Adding Ipv6 and Persistence support
Cc: Hans Schillstrom <hans.schillstrom@xxxxxxxxxxxx>, Julian Anastasov <ja@xxxxxx>, LVS-Devel <lvs-devel@xxxxxxxxxxxxxxx>, "wensong@xxxxxxxxxxxx" <wensong@xxxxxxxxxxxx>, "daniel.lezcano@xxxxxxx" <daniel.lezcano@xxxxxxx>
From: Simon Horman <horms@xxxxxxxxxxxx>
Date: Tue, 9 Nov 2010 09:39:11 +0900
On Mon, Nov 08, 2010 at 11:23:25PM +0100, Hans Schillstrom wrote:
> 
> On Monday, November 08, 2010 23:00:29 Simon Horman wrote:
> > On Mon, Nov 08, 2010 at 04:15:10PM +0100, Hans Schillstrom wrote:
> > > On Monday 08 November 2010 07:21:23 Simon Horman wrote:
> > > [ snip ]
> > > 
> > > > How about this, which applies on top of
> > > > "IPVS: Add persistence engine to connection entry".
> > > >
> > > > From: Simon Horman <horms@xxxxxxxxxxxx>
> > > > Subject: IPVS: Only match pe_data created by the same pe
> > > >
> > > > Only match persistence engine data if it was
> > > > created by the same persistence engine.
> > > >
> > > > Reported-by: Julian Anastasov <ja@xxxxxx>
> > > > Signed-off-by: Simon Horman <horms@xxxxxxxxxxxx>
> > > 
> > > Thanks I'll will use that one,
> > > I think it's time to cook a new backup patch,
> > > or do any one have more patches in the pipe ?
> > 
> > I have added this and the other patches that I am completely happy with
> > to the staging branch of lvs-test-2.6.
> > 
> > Specifically the patches in there are:
> > 
> > * IPVS: Only match pe_data created by the same pe
> > * IPVS: Add persistence engine to connection entry
> > * IPVS: Backup, Adding structs for new sync format
> > * IPVS: Prepare for transferring firewall marks (fwmark) to the backup 
> > daemon.
> > * IPVS: ip_vs_pe.c, use strncmp to be safe.
> > 
> > If you are unhappy with any of those changes let me know,
> > I'm happy to revert and rebase the staging branch
> > (its mainly for your benefit at this stage).
> 
> Thanks a lot,
> I will start with the cooking tomorrow, with lvs-test-2.6 as my base

Be sure to use the staging branch :-)

> > I would also like to add the following clean-up changes,
> > could I get an Ack or Nack from you on each of them?
> > 
> > * IPVS: Make the cp argument to ip_vs_sync_conn() static
> >   http://www.spinics.net/lists/lvs-devel/msg01375.html
> ACK,
> I'll do the same in the last patch for _v0/_v1 sending
> 
> > * IPVS: Remove useless { } block from ip_vs_process_message()
> >   http://www.spinics.net/lists/lvs-devel/msg01369.html
> ACK,
> 
> > * IPVS: buffer argument to ip_vs_process_message() should not be const
> >   http://www.spinics.net/lists/lvs-devel/msg01377.html
> > 
> ACK

Thanks.

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