LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [*v2 PATCH 06/22] IPVS: netns preparation for proto_tcp

To: Hans Schillstrom <hans@xxxxxxxxxxxxxxx>
Subject: Re: [*v2 PATCH 06/22] IPVS: netns preparation for proto_tcp
Cc: Simon Horman <horms@xxxxxxxxxxxx>, Hans Schillstrom <hans.schillstrom@xxxxxxxxxxxx>, daniel.lezcano@xxxxxxx, Wensong Zhang <wensong@xxxxxxxxxxxx>, lvs-devel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, netfilter-devel@xxxxxxxxxxxxxxx
From: Julian Anastasov <ja@xxxxxx>
Date: Wed, 15 Dec 2010 23:37:12 +0200 (EET)

        Hello,

On Tue, 14 Dec 2010, Hans Schillstrom wrote:

+/* Remove dot used
 static int
 tcp_set_state_timeout(struct ip_vs_protocol *pp, char *sname, int to)
 {
        return ip_vs_set_state_timeout(pp->timeout_table, IP_VS_TCP_S_LAST,
                                       tcp_state_name_table, sname, to);
-}
+} */

Can we just remove tcp_set_state_timeout ?

Sure, anyone that disagree ?

        OK. Long ago the idea was ipvsadm to change the
timeouts for all states but it was not finished. IIRC,
all sysctl vars to control these timeouts are disabled.
What is missing is some method (via netlink) to provide
protocol, state name and timeout to the tcp_set_state_timeout
function because some users want to reduce the memory
usage with lower timeouts. Example:

# ipvsadm --get-proto-timeouts TCP SYN_RECV,ESTABLISHED
TCP.SYN_RECV=60
TCP.ESTABLISHED=900

# ipvsadm --set-proto-timeouts TCP "SYN_RECV=10, ESTABLISHED=20"

Regards

--
Julian Anastasov <ja@xxxxxx>
--
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>