Date: Wed, 29 Jun 2016 20:03:01 -0700 From: Cy Schubert <Cy.Schubert@komquats.com> To: "Bjoern A. Zeeb" <bz@FreeBSD.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r302289 - head/sys/contrib/ipfilter/netinet Message-ID: <201606300303.u5U331J9003137@slippy.cwsent.com> In-Reply-To: Message from "Bjoern A. Zeeb" <bz@FreeBSD.org> of "Thu, 30 Jun 2016 01:32:12 -0000." <201606300132.u5U1WCUa093377@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
In message <201606300132.u5U1WCUa093377@repo.freebsd.org>, "Bjoern A. Zeeb" wri tes: > Author: bz > Date: Thu Jun 30 01:32:12 2016 > New Revision: 302289 > URL: https://svnweb.freebsd.org/changeset/base/302289 > > Log: > Remove unused global variables as well as unused memory > allocations from ipfilter in preparation for VNET support. > > Suggested by: cy (see D7000) > Sponsored by: The FreeBSD Foundation > MFC after: 2 weeks > Approved by: re (gjb) > > Modified: > head/sys/contrib/ipfilter/netinet/ip_fil.h > head/sys/contrib/ipfilter/netinet/ip_fil_freebsd.c > head/sys/contrib/ipfilter/netinet/ip_nat.c > > Modified: head/sys/contrib/ipfilter/netinet/ip_fil.h > ============================================================================= > = > --- head/sys/contrib/ipfilter/netinet/ip_fil.h Wed Jun 29 23:56:24 201 > 6 (r302288) > +++ head/sys/contrib/ipfilter/netinet/ip_fil.h Thu Jun 30 01:32:12 201 > 6 (r302289) > @@ -1710,7 +1710,6 @@ typedef struct ipf_main_softc_s { > > #ifndef _KERNEL > extern int ipf_check __P((void *, struct ip *, int, void *, int, m > b_t **)); > -extern int (*ipf_checkp) __P((ip_t *, int, void *, int, mb_t **)); > extern struct ifnet *get_unit __P((char *, int)); > extern char *get_ifname __P((struct ifnet *)); > extern int ipfioctl __P((ipf_main_softc_t *, int, ioctlcmd_t, > > Modified: head/sys/contrib/ipfilter/netinet/ip_fil_freebsd.c > ============================================================================= > = > --- head/sys/contrib/ipfilter/netinet/ip_fil_freebsd.c Wed Jun 29 23:5 > 6:24 2016 (r302288) > +++ head/sys/contrib/ipfilter/netinet/ip_fil_freebsd.c Thu Jun 30 01:3 > 2:12 2016 (r302289) > @@ -99,10 +99,8 @@ MALLOC_DEFINE(M_IPFILTER, "ipfilter", "I > # endif > > > -static int (*ipf_savep) __P((void *, ip_t *, int, void *, int, str > uct mbuf **)); > static int ipf_send_ip __P((fr_info_t *, mb_t *)); > static void ipf_timer_func __P((void *arg)); > -int ipf_locks_done = 0; > > ipf_main_softc_t ipfmain; > > @@ -110,10 +108,6 @@ ipf_main_softc_t ipfmain; > # if defined(NETBSD_PF) > # include <net/pfil.h> > # endif /* NETBSD_PF */ > -/* > - * We provide the ipf_checkp name just to minimize changes later. > - */ > -int (*ipf_checkp) __P((void *, ip_t *ip, int hlen, void *ifp, int out, mb_t > **mp)); > > > static eventhandler_tag ipf_arrivetag, ipf_departtag, ipf_clonetag; > @@ -221,11 +215,6 @@ ipfattach(softc) > } > > > - if (ipf_checkp != ipf_check) { > - ipf_savep = ipf_checkp; > - ipf_checkp = ipf_check; > - } > - > bzero((char *)ipfmain.ipf_selwait, sizeof(ipfmain.ipf_selwait)); > softc->ipf_running = 1; > > @@ -268,12 +257,6 @@ ipfdetach(softc) > #endif > callout_drain(&softc->ipf_slow_ch); > > -#ifndef NETBSD_PF > - if (ipf_checkp != NULL) > - ipf_checkp = ipf_savep; > - ipf_savep = NULL; > -#endif > - > ipf_fini_all(softc); > > softc->ipf_running = -2; > > Modified: head/sys/contrib/ipfilter/netinet/ip_nat.c > ============================================================================= > = > --- head/sys/contrib/ipfilter/netinet/ip_nat.c Wed Jun 29 23:56:24 201 > 6 (r302288) > +++ head/sys/contrib/ipfilter/netinet/ip_nat.c Thu Jun 30 01:32:12 201 > 6 (r302289) > @@ -133,8 +133,6 @@ static const char rcsid[] = "@(#)$FreeBS > #define NBUMPSIDEDF(y,x)do { softn->ipf_nat_stats.ns_side[y].x++; \ > DT1(x, fr_info_t *, fin); } while (0) > > -frentry_t ipfnatblock; > - > static ipftuneable_t ipf_nat_tuneables[] = { > /* nat */ > { { (void *)offsetof(ipf_nat_softc_t, ipf_nat_lock) }, > @@ -275,9 +273,6 @@ static void ipf_nat_tabmove __P((ipf_nat > int > ipf_nat_main_load() > { > - bzero((char *)&ipfnatblock, sizeof(ipfnatblock)); > - ipfnatblock.fr_flags = FR_BLOCK|FR_QUICK; > - ipfnatblock.fr_ref = 1; > > return 0; > } > > Thank you Bjoern. -- Cheers, Cy Schubert <Cy.Schubert@cschubert.com> FreeBSD UNIX: <cy@FreeBSD.org> Web: http://www.FreeBSD.org The need of the many outweighs the greed of the few.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201606300303.u5U331J9003137>