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
