Module Name: src
Committed By: rmind
Date: Fri Jun 6 01:02:47 UTC 2014
Modified Files:
src/sys/net: if.c if_faith.c if_mpls.c route.c route.h
src/sys/netinet: ip_carp.c
src/sys/netinet6: icmp6.c in6.c nd6.c
Log Message:
- Eliminate RTFREE() macro in favour of rtfree() function.
- Make rtcache() function static.
To generate a diff of this commit:
cvs rdiff -u -r1.276 -r1.277 src/sys/net/if.c
cvs rdiff -u -r1.48 -r1.49 src/sys/net/if_faith.c
cvs rdiff -u -r1.13 -r1.14 src/sys/net/if_mpls.c
cvs rdiff -u -r1.130 -r1.131 src/sys/net/route.c
cvs rdiff -u -r1.83 -r1.84 src/sys/net/route.h
cvs rdiff -u -r1.56 -r1.57 src/sys/netinet/ip_carp.c
cvs rdiff -u -r1.168 -r1.169 src/sys/netinet6/icmp6.c
cvs rdiff -u -r1.170 -r1.171 src/sys/netinet6/in6.c
cvs rdiff -u -r1.151 -r1.152 src/sys/netinet6/nd6.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/sys/net/if.c
diff -u src/sys/net/if.c:1.276 src/sys/net/if.c:1.277
--- src/sys/net/if.c:1.276 Thu Jun 5 23:48:16 2014
+++ src/sys/net/if.c Fri Jun 6 01:02:47 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: if.c,v 1.276 2014/06/05 23:48:16 rmind Exp $ */
+/* $NetBSD: if.c,v 1.277 2014/06/06 01:02:47 rmind Exp $ */
/*-
* Copyright (c) 1999, 2000, 2001, 2008 The NetBSD Foundation, Inc.
@@ -90,7 +90,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if.c,v 1.276 2014/06/05 23:48:16 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if.c,v 1.277 2014/06/06 01:02:47 rmind Exp $");
#include "opt_inet.h"
@@ -927,7 +927,7 @@ if_rt_walktree(struct rtentry *rt, void
rt_mask(rt), rt->rt_flags, NULL);
KASSERT((rt->rt_flags & RTF_UP) == 0);
rt->rt_ifp = NULL;
- RTFREE(rt);
+ rtfree(rt);
if (error != 0)
printf("%s: warning: unable to delete rtentry @ %p, "
"error = %d\n", ifp->if_xname, rt, error);
Index: src/sys/net/if_faith.c
diff -u src/sys/net/if_faith.c:1.48 src/sys/net/if_faith.c:1.49
--- src/sys/net/if_faith.c:1.48 Thu Jun 5 23:48:16 2014
+++ src/sys/net/if_faith.c Fri Jun 6 01:02:47 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: if_faith.c,v 1.48 2014/06/05 23:48:16 rmind Exp $ */
+/* $NetBSD: if_faith.c,v 1.49 2014/06/06 01:02:47 rmind Exp $ */
/* $KAME: if_faith.c,v 1.21 2001/02/20 07:59:26 itojun Exp $ */
/*
@@ -40,7 +40,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_faith.c,v 1.48 2014/06/05 23:48:16 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_faith.c,v 1.49 2014/06/06 01:02:47 rmind Exp $");
#include "opt_inet.h"
@@ -290,7 +290,7 @@ faithprefix(struct in6_addr *in6)
else
ret = 0;
if (rt)
- RTFREE(rt);
+ rtfree(rt);
return ret;
}
#endif
Index: src/sys/net/if_mpls.c
diff -u src/sys/net/if_mpls.c:1.13 src/sys/net/if_mpls.c:1.14
--- src/sys/net/if_mpls.c:1.13 Thu Jun 5 23:48:16 2014
+++ src/sys/net/if_mpls.c Fri Jun 6 01:02:47 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: if_mpls.c,v 1.13 2014/06/05 23:48:16 rmind Exp $ */
+/* $NetBSD: if_mpls.c,v 1.14 2014/06/06 01:02:47 rmind Exp $ */
/*
* Copyright (c) 2010 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_mpls.c,v 1.13 2014/06/05 23:48:16 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_mpls.c,v 1.14 2014/06/06 01:02:47 rmind Exp $");
#include "opt_inet.h"
#include "opt_mpls.h"
@@ -268,7 +268,7 @@ mpls_output(struct ifnet *ifp, struct mb
}
err = mpls_send_frame(m, rt1->rt_ifp, rt);
- RTFREE(rt1);
+ rtfree(rt1);
return err;
}
@@ -439,7 +439,7 @@ done:
if (error != 0 && m != NULL)
m_freem(m);
if (rt != NULL)
- RTFREE(rt);
+ rtfree(rt);
return error;
}
Index: src/sys/net/route.c
diff -u src/sys/net/route.c:1.130 src/sys/net/route.c:1.131
--- src/sys/net/route.c:1.130 Sat Apr 26 11:10:10 2014
+++ src/sys/net/route.c Fri Jun 6 01:02:47 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: route.c,v 1.130 2014/04/26 11:10:10 pooka Exp $ */
+/* $NetBSD: route.c,v 1.131 2014/06/06 01:02:47 rmind Exp $ */
/*-
* Copyright (c) 1998, 2008 The NetBSD Foundation, Inc.
@@ -93,7 +93,7 @@
#include "opt_route.h"
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: route.c,v 1.130 2014/04/26 11:10:10 pooka Exp $");
+__KERNEL_RCSID(0, "$NetBSD: route.c,v 1.131 2014/06/06 01:02:47 rmind Exp $");
#include <sys/param.h>
#include <sys/kmem.h>
@@ -301,7 +301,7 @@ rtflushall(int family)
rtcache_invalidate(&dom->dom_rtcache);
}
-void
+static void
rtcache(struct route *ro)
{
struct domain *dom;
@@ -703,7 +703,7 @@ rtrequest1(int req, struct rt_addrinfo *
if ((rt = rt_deladdr(rtbl, dst, netmask)) == NULL)
senderr(ESRCH);
if (rt->rt_gwroute) {
- RTFREE(rt->rt_gwroute);
+ rtfree(rt->rt_gwroute);
rt->rt_gwroute = NULL;
}
if (rt->rt_parent) {
@@ -795,7 +795,7 @@ rtrequest1(int req, struct rt_addrinfo *
rtdeletemsg(crt);
rc = rt_addaddr(rtbl, rt, netmask);
}
- RTFREE(crt);
+ rtfree(crt);
RT_DPRINTF("rt->_rt_key = %p\n", (void *)rt->_rt_key);
}
RT_DPRINTF("rt->_rt_key = %p\n", (void *)rt->_rt_key);
@@ -851,7 +851,7 @@ rt_setgate(struct rtentry *rt, const str
RT_DPRINTF("rt->_rt_key = %p\n", (void *)rt->_rt_key);
if (rt->rt_gwroute) {
- RTFREE(rt->rt_gwroute);
+ rtfree(rt->rt_gwroute);
rt->rt_gwroute = NULL;
}
KASSERT(rt->_rt_key != NULL);
@@ -1284,7 +1284,7 @@ rtcache_clear(struct route *ro)
LIST_REMOVE(ro, ro_rtcache_next);
- RTFREE(ro->_ro_rt);
+ rtfree(ro->_ro_rt);
ro->_ro_rt = NULL;
ro->ro_invalid = false;
rtcache_invariants(ro);
Index: src/sys/net/route.h
diff -u src/sys/net/route.h:1.83 src/sys/net/route.h:1.84
--- src/sys/net/route.h:1.83 Sat Apr 26 11:10:10 2014
+++ src/sys/net/route.h Fri Jun 6 01:02:47 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: route.h,v 1.83 2014/04/26 11:10:10 pooka Exp $ */
+/* $NetBSD: route.h,v 1.84 2014/06/06 01:02:47 rmind Exp $ */
/*
* Copyright (c) 1980, 1986, 1993
@@ -382,7 +382,6 @@ void rt_timer_queue_destroy(struct rtti
void rt_timer_remove_all(struct rtentry *, int);
unsigned long rt_timer_count(struct rttimer_queue *);
void rt_timer_timer(void *);
-void rtcache(struct route *);
void rtflushall(int);
struct rtentry *
rtalloc1(const struct sockaddr *, int);
@@ -495,15 +494,6 @@ rtcache_validate(const struct route *ro)
}
-static inline void
-RTFREE(struct rtentry *rt)
-{
- if (rt->rt_refcnt <= 1)
- rtfree(rt);
- else
- rt->rt_refcnt--;
-}
-
int rt_walktree(sa_family_t, int (*)(struct rtentry *, void *), void *);
void route_enqueue(struct mbuf *, int);
int rt_inithead(rtbl_t **, int);
Index: src/sys/netinet/ip_carp.c
diff -u src/sys/netinet/ip_carp.c:1.56 src/sys/netinet/ip_carp.c:1.57
--- src/sys/netinet/ip_carp.c:1.56 Thu May 29 23:02:48 2014
+++ src/sys/netinet/ip_carp.c Fri Jun 6 01:02:47 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: ip_carp.c,v 1.56 2014/05/29 23:02:48 rmind Exp $ */
+/* $NetBSD: ip_carp.c,v 1.57 2014/06/06 01:02:47 rmind Exp $ */
/* $OpenBSD: ip_carp.c,v 1.113 2005/11/04 08:11:54 mcbride Exp $ */
/*
@@ -31,7 +31,7 @@
#include "opt_mbuftrace.h"
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ip_carp.c,v 1.56 2014/05/29 23:02:48 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ip_carp.c,v 1.57 2014/06/06 01:02:47 rmind Exp $");
/*
* TODO:
@@ -390,7 +390,7 @@ carp_setroute(struct carp_softc *sc, int
hr_otherif = (rt && rt->rt_ifp != &sc->sc_if &&
rt->rt_flags & (RTF_CLONING|RTF_CLONED));
if (rt != NULL) {
- RTFREE(rt);
+ rtfree(rt);
rt = NULL;
}
@@ -434,7 +434,7 @@ carp_setroute(struct carp_softc *sc, int
break;
}
if (rt != NULL) {
- RTFREE(rt);
+ rtfree(rt);
rt = NULL;
}
break;
Index: src/sys/netinet6/icmp6.c
diff -u src/sys/netinet6/icmp6.c:1.168 src/sys/netinet6/icmp6.c:1.169
--- src/sys/netinet6/icmp6.c:1.168 Fri May 30 01:39:03 2014
+++ src/sys/netinet6/icmp6.c Fri Jun 6 01:02:47 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: icmp6.c,v 1.168 2014/05/30 01:39:03 christos Exp $ */
+/* $NetBSD: icmp6.c,v 1.169 2014/06/06 01:02:47 rmind Exp $ */
/* $KAME: icmp6.c,v 1.217 2001/06/20 15:03:29 jinmei Exp $ */
/*
@@ -62,7 +62,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: icmp6.c,v 1.168 2014/05/30 01:39:03 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: icmp6.c,v 1.169 2014/06/06 01:02:47 rmind Exp $");
#include "opt_inet.h"
#include "opt_ipsec.h"
@@ -1132,8 +1132,8 @@ icmp6_mtudisc_update(struct ip6ctlparam
rt->rt_rmx.rmx_mtu = mtu;
}
}
- if (rt) { /* XXX: need braces to avoid conflict with else in RTFREE. */
- RTFREE(rt);
+ if (rt) {
+ rtfree(rt);
}
/*
@@ -2188,7 +2188,7 @@ icmp6_redirect_input(struct mbuf *m, int
"ICMP6 redirect rejected; no route "
"with inet6 gateway found for redirect dst: %s\n",
icmp6_redirect_diag(&src6, &reddst6, &redtgt6)));
- RTFREE(rt);
+ rtfree(rt);
goto bad;
}
@@ -2200,7 +2200,7 @@ icmp6_redirect_input(struct mbuf *m, int
"%s\n",
ip6_sprintf(gw6),
icmp6_redirect_diag(&src6, &reddst6, &redtgt6)));
- RTFREE(rt);
+ rtfree(rt);
goto bad;
}
} else {
@@ -2210,7 +2210,7 @@ icmp6_redirect_input(struct mbuf *m, int
icmp6_redirect_diag(&src6, &reddst6, &redtgt6)));
goto bad;
}
- RTFREE(rt);
+ rtfree(rt);
rt = NULL;
}
if (IN6_IS_ADDR_MULTICAST(&reddst6)) {
Index: src/sys/netinet6/in6.c
diff -u src/sys/netinet6/in6.c:1.170 src/sys/netinet6/in6.c:1.171
--- src/sys/netinet6/in6.c:1.170 Sat May 17 20:44:24 2014
+++ src/sys/netinet6/in6.c Fri Jun 6 01:02:47 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: in6.c,v 1.170 2014/05/17 20:44:24 rmind Exp $ */
+/* $NetBSD: in6.c,v 1.171 2014/06/06 01:02:47 rmind Exp $ */
/* $KAME: in6.c,v 1.198 2001/07/18 09:12:38 itojun Exp $ */
/*
@@ -62,7 +62,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: in6.c,v 1.170 2014/05/17 20:44:24 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: in6.c,v 1.171 2014/06/06 01:02:47 rmind Exp $");
#include "opt_inet.h"
#include "opt_compat_netbsd.h"
@@ -1168,7 +1168,7 @@ in6_update_ifa1(struct ifnet *ifp, struc
if (memcmp(&mltaddr.sin6_addr,
&satocsin6(rt_getkey(rt))->sin6_addr,
MLTMASK_LEN)) {
- RTFREE(rt);
+ rtfree(rt);
rt = NULL;
} else if (rt->rt_ifp != ifp) {
IN6_DPRINTF("%s: rt_ifp %p -> %p (%s) "
@@ -1199,7 +1199,7 @@ in6_update_ifa1(struct ifnet *ifp, struc
if (error)
goto cleanup;
} else {
- RTFREE(rt);
+ rtfree(rt);
}
imm = in6_joingroup(ifp, &mltaddr.sin6_addr, &error, 0);
if (!imm) {
@@ -1253,7 +1253,7 @@ in6_update_ifa1(struct ifnet *ifp, struc
if (memcmp(&mltaddr.sin6_addr,
&satocsin6(rt_getkey(rt))->sin6_addr,
32 / NBBY)) {
- RTFREE(rt);
+ rtfree(rt);
rt = NULL;
} else if (rt->rt_ifp != ifp) {
IN6_DPRINTF("%s: rt_ifp %p -> %p (%s) "
@@ -1284,7 +1284,7 @@ in6_update_ifa1(struct ifnet *ifp, struc
goto cleanup;
#undef MLTMASK_LEN
} else {
- RTFREE(rt);
+ rtfree(rt);
}
imm = in6_joingroup(ifp, &mltaddr.sin6_addr, &error, 0);
if (!imm) {
Index: src/sys/netinet6/nd6.c
diff -u src/sys/netinet6/nd6.c:1.151 src/sys/netinet6/nd6.c:1.152
--- src/sys/netinet6/nd6.c:1.151 Thu Jun 5 16:06:49 2014
+++ src/sys/netinet6/nd6.c Fri Jun 6 01:02:47 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: nd6.c,v 1.151 2014/06/05 16:06:49 roy Exp $ */
+/* $NetBSD: nd6.c,v 1.152 2014/06/06 01:02:47 rmind Exp $ */
/* $KAME: nd6.c,v 1.279 2002/06/08 11:16:51 itojun Exp $ */
/*
@@ -31,7 +31,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: nd6.c,v 1.151 2014/06/05 16:06:49 roy Exp $");
+__KERNEL_RCSID(0, "$NetBSD: nd6.c,v 1.152 2014/06/06 01:02:47 rmind Exp $");
#include "opt_ipsec.h"
@@ -847,7 +847,7 @@ nd6_lookup1(const struct in6_addr *addr6
* interface route.
*/
if (create) {
- RTFREE(rt);
+ rtfree(rt);
rt = NULL;
}
}