LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

[lvs-devel] Fixed: [PATCH 2.6.23.1] IPVS: fixes possible real server ove

Subject: [lvs-devel] Fixed: [PATCH 2.6.23.1] IPVS: fixes possible real server overcommit on director failover.
From: rumen at voicecho.com (Rumen Bogdanovski)
Date: Mon, 05 Nov 2007 17:25:16 +0200
I was thinking about some cosmetics by using currently unused connection flag 
IP_VS_CONN_F_SYNC
and introducing dest.syncconns (sync connection counter). This way we should 
introduce additional 
field in /proc/net/ip_vs and make it show dest.syncconns in addition to 
inactconns and activeconns. 
Also a new field in /proc/net/ip_vs_conn to designate whether it is a local or 
sync connection should 
be added. IMHO this way it should be much nicer and more informative to the 
user. However changing 
this will not be well accepted (I guess), so I have other idea:
to introduce /proc/net/ip_vs_sync and  /proc/net/ip_vs_conn_sync to show only 
syncconns the same way
/proc/net/ip_vs and /proc/net/ip_vs_conn does. 
I can work on this in my free time if one of the ideas is accepted.

Do you like the idea?
I am waiting for comments.

Rumen

On Mon, 2007-11-05 at 11:52 +0900, Simon Horman wrote:
> On Thu, Nov 01, 2007 at 03:28:48PM +0200, Rumen Bogdanovski wrote:
> > 
> > 
> > > I hacked up something, though I need to double check that
> > > I made all the substitutions correctly:
> > 
> > I think it is ok. It it is working for half an hour without a problem.
> > Adding/removing real servers, connecting/disconnecting, simulating
> > fail-overs no problems detected. And as I looked in the patch the
> > substitutions looked correct.
> 
> Ok Great.
> 
> Sorry for messing up the posting of your other two patches (twice).
> I will send them again. Once there is some response from netdev
> about them we can address this patch.




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