From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Subject: Re: [PATCH] ipvs: fix synchronization on connection close Date: Fri, 11 Dec 2009 14:35:42 +1100 Message-ID: <20091211033542.GA28136@verge.net.au> References: <1260434502-14166-1-git-send-email-dfeng@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, lvs-devel@vger.kernel.org, linux-kernel@vger.kernel.org, Wensong Zhang , Julian Anastasov , "David S. Miller" , Patrick McHardy To: Xiaotian Feng Return-path: Content-Disposition: inline In-Reply-To: <1260434502-14166-1-git-send-email-dfeng@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Thu, Dec 10, 2009 at 04:41:42PM +0800, Xiaotian Feng wrote: > commit 9d3a0de makes slaves expire as they would do on the master > with much shorter timeouts. But it introduces another problem: > When we close a connection, on master server the connection became > CLOSE_WAIT/TIME_WAIT, it was synced to slaves, but if master is > finished within it's timeouts (CLOSE), it will not be synced to > slaves. Then slaves will be kept on CLOSE_WAIT/TIME_WAIT until > timeout reaches. Thus we should also sync with CLOSE. > > Signed-off-by: Xiaotian Feng > Cc: Wensong Zhang > Cc: Simon Horman > Cc: Julian Anastasov > Cc: David S. Miller This seems reasonable to me. Acked-by: Simon Horman > --- > net/netfilter/ipvs/ip_vs_core.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/net/netfilter/ipvs/ip_vs_core.c b/net/netfilter/ipvs/ip_vs_core.c > index b95699f..847ffca 100644 > --- a/net/netfilter/ipvs/ip_vs_core.c > +++ b/net/netfilter/ipvs/ip_vs_core.c > @@ -1366,6 +1366,7 @@ ip_vs_in(unsigned int hooknum, struct sk_buff *skb, > == sysctl_ip_vs_sync_threshold[0])) || > ((cp->protocol == IPPROTO_TCP) && (cp->old_state != cp->state) && > ((cp->state == IP_VS_TCP_S_FIN_WAIT) || > + (cp->state == IP_VS_TCP_S_CLOSE) || > (cp->state == IP_VS_TCP_S_CLOSE_WAIT) || > (cp->state == IP_VS_TCP_S_TIME_WAIT))))) > ip_vs_sync_conn(cp); > -- > 1.6.5.2 > > -- > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html