From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steffen Klassert Subject: Re: [PATCH] ipv6: fix two typos in a comment in xfrm6_init() Date: Thu, 8 Nov 2012 15:19:36 +0100 Message-ID: <20121108141936.GC22290@secunet.com> References: <1352363888-14068-1-git-send-email-roy.qing.li@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, davem@davemloft.net To: roy.qing.li@gmail.com Return-path: Received: from a.mx.secunet.com ([195.81.216.161]:47929 "EHLO a.mx.secunet.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755975Ab2KHOTq (ORCPT ); Thu, 8 Nov 2012 09:19:46 -0500 Content-Disposition: inline In-Reply-To: <1352363888-14068-1-git-send-email-roy.qing.li@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: On Thu, Nov 08, 2012 at 04:38:08PM +0800, roy.qing.li@gmail.com wrote: > From: Li RongQing > > Signed-off-by: Li RongQing > --- > net/ipv6/xfrm6_policy.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/net/ipv6/xfrm6_policy.c b/net/ipv6/xfrm6_policy.c > index f3ed8ca..93832e8 100644 > --- a/net/ipv6/xfrm6_policy.c > +++ b/net/ipv6/xfrm6_policy.c > @@ -332,10 +332,10 @@ int __init xfrm6_init(void) > /* > * We need a good default value for the xfrm6 gc threshold. > * In ipv4 we set it to the route hash table size * 8, which > - * is half the size of the maximaum route cache for ipv4. It > + * is half the size of the maximum route cache for ipv4. The routing cache is removed, so this comment is obsolete. But it reminds me that we set the gc threshold to ip_rt_max_size/2 in ipv4. With the routing cache removal patch, ip_rt_max_size was set to INT_MAX. So the gc starts to remove entries when a threshold of INT_MAX/2 is reached. cat /proc/sys/net/ipv4/xfrm4_gc_thresh 1073741823 I guess this was not intentional.