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