From owner-svn-src-all@FreeBSD.ORG Tue Mar 13 13:20:46 2012 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx2.freebsd.org (mx2.freebsd.org [69.147.83.53]) by hub.freebsd.org (Postfix) with ESMTP id CCF9E10656D5; Tue, 13 Mar 2012 13:20:46 +0000 (UTC) (envelope-from melifaro@FreeBSD.org) Received: from dhcp170-36-red.yandex.net (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx2.freebsd.org (Postfix) with ESMTP id 1E942151906; Tue, 13 Mar 2012 13:20:15 +0000 (UTC) Message-ID: <4F5F48F0.2090905@FreeBSD.org> Date: Tue, 13 Mar 2012 17:17:36 +0400 From: "Alexander V. Chernikov" User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:8.0) Gecko/20111117 Thunderbird/8.0 MIME-Version: 1.0 To: "Bjoern A. Zeeb" References: <201203131108.q2DB8e5d040995@svn.freebsd.org> <1FC92093-FAEA-4C84-8A27-7726C4930CD6@FreeBSD.org> In-Reply-To: <1FC92093-FAEA-4C84-8A27-7726C4930CD6@FreeBSD.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r232921 - in head/sys: modules/netgraph/netflow netgraph/netflow X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 13 Mar 2012 13:20:47 -0000 On 13.03.2012 16:51, Bjoern A. Zeeb wrote: > On 13. Mar 2012, at 11:08 , Alexander V. Chernikov wrote: > >> Author: melifaro >> Date: Tue Mar 13 11:08:40 2012 >> New Revision: 232921 >> URL: http://svn.freebsd.org/changeset/base/232921 >> >> Log: >> Use rt_numfibs variable instead of compile-time RT_NUMFIBS. > > Thanks a lot for doing this! Sorry for doing this for too long. > > This will allow us to go ahead and get rid of the mandatory kernel option now. I'll try to find my patch from last year and post it for your review. > > Are you intending to MFC this? netflow v9 stuff is still not MFCed to 8/7 (And I forgot 9 contains it). So it should be 2w for 9 > > > Requested by: bz (a while ago with low prio) >> Reviewed by: glebius (previous version) >> Approved by: kib(mentor), ae(mentor) >> >> Modified: >> head/sys/modules/netgraph/netflow/Makefile >> head/sys/netgraph/netflow/netflow.c >> head/sys/netgraph/netflow/ng_netflow.c >> head/sys/netgraph/netflow/ng_netflow.h >> >> Modified: head/sys/modules/netgraph/netflow/Makefile >> ============================================================================== >> --- head/sys/modules/netgraph/netflow/Makefile Tue Mar 13 10:54:14 2012 (r232920) >> +++ head/sys/modules/netgraph/netflow/Makefile Tue Mar 13 11:08:40 2012 (r232921) >> @@ -17,8 +17,6 @@ opt_inet6.h: >> echo "#define INET6 1"> ${.TARGET} >> .endif >> >> -opt_route.h: >> - echo "#define ROUTETABLES RT_MAXFIBS"> ${.TARGET} >> .endif >> >> .include >> >> Modified: head/sys/netgraph/netflow/netflow.c >> ============================================================================== >> --- head/sys/netgraph/netflow/netflow.c Tue Mar 13 10:54:14 2012 (r232920) >> +++ head/sys/netgraph/netflow/netflow.c Tue Mar 13 11:08:40 2012 (r232921) >> @@ -630,7 +630,7 @@ ng_netflow_cache_flush(priv_p priv) >> free(priv->hash6, M_NETFLOW_HASH); >> #endif >> >> - for (i = 0; i< RT_NUMFIBS; i++) { >> + for (i = 0; i< priv->maxfibs; i++) { >> if ((fe = priv_to_fib(priv, i)) == NULL) >> continue; >> >> >> Modified: head/sys/netgraph/netflow/ng_netflow.c >> ============================================================================== >> --- head/sys/netgraph/netflow/ng_netflow.c Tue Mar 13 10:54:14 2012 (r232920) >> +++ head/sys/netgraph/netflow/ng_netflow.c Tue Mar 13 11:08:40 2012 (r232921) >> @@ -225,6 +225,11 @@ ng_netflow_constructor(node_p node) >> /* Initialize private data */ >> priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); >> >> + /* Initialize fib data */ >> + priv->maxfibs = rt_numfibs; >> + priv->fib_data = malloc(sizeof(fib_export_p) * priv->maxfibs, >> + M_NETGRAPH, M_WAITOK | M_ZERO); >> + >> /* Make node and its data point at each other */ >> NG_NODE_SET_PRIVATE(node, priv); >> priv->node = node; >> @@ -901,8 +906,10 @@ loopend: >> >> /* Check packet FIB */ >> fib = M_GETFIB(m); >> - if (fib>= RT_NUMFIBS) { >> - CTR2(KTR_NET, "ng_netflow_rcvdata(): packet fib %d is out of range of available fibs: 0 .. %d", fib, RT_NUMFIBS); >> + if (fib>= priv->maxfibs) { >> + CTR2(KTR_NET, "ng_netflow_rcvdata(): packet fib %d is out of " >> + "range of available fibs: 0 .. %d", >> + fib, priv->maxfibs); >> goto bypass; >> } >> >> @@ -973,6 +980,7 @@ ng_netflow_rmnode(node_p node) >> NG_NODE_SET_PRIVATE(node, NULL); >> NG_NODE_UNREF(priv->node); >> >> + free(priv->fib_data, M_NETGRAPH); >> free(priv, M_NETGRAPH); >> >> return (0); >> >> Modified: head/sys/netgraph/netflow/ng_netflow.h >> ============================================================================== >> --- head/sys/netgraph/netflow/ng_netflow.h Tue Mar 13 10:54:14 2012 (r232920) >> +++ head/sys/netgraph/netflow/ng_netflow.h Tue Mar 13 11:08:40 2012 (r232921) >> @@ -413,7 +413,8 @@ struct netflow { >> struct flow_hash_entry *hash6; >> #endif >> /* Multiple FIB support */ >> - fib_export_p fib_data[RT_NUMFIBS]; /* array of pointers to fib-specific data */ >> + fib_export_p *fib_data; /* array of pointers to per-fib data */ >> + uint16_t maxfibs; /* number of allocated fibs */ >> >> /* >> * RFC 3954 clause 7.3 > -- WBR, Alexander