Module Name: src Committed By: ozaki-r Date: Fri Apr 1 09:00:27 UTC 2016
Modified Files: src/sys/net: route.c Log Message: Remove unnecessary RTTIMER_CALLOUT macro rttimer#rtt_func never be NULL. To generate a diff of this commit: cvs rdiff -u -r1.155 -r1.156 src/sys/net/route.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/route.c diff -u src/sys/net/route.c:1.155 src/sys/net/route.c:1.156 --- src/sys/net/route.c:1.155 Fri Apr 1 02:00:14 2016 +++ src/sys/net/route.c Fri Apr 1 09:00:27 2016 @@ -1,4 +1,4 @@ -/* $NetBSD: route.c,v 1.155 2016/04/01 02:00:14 ozaki-r Exp $ */ +/* $NetBSD: route.c,v 1.156 2016/04/01 09:00:27 ozaki-r Exp $ */ /*- * Copyright (c) 1998, 2008 The NetBSD Foundation, Inc. @@ -96,7 +96,7 @@ #endif #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: route.c,v 1.155 2016/04/01 02:00:14 ozaki-r Exp $"); +__KERNEL_RCSID(0, "$NetBSD: route.c,v 1.156 2016/04/01 09:00:27 ozaki-r Exp $"); #include <sys/param.h> #ifdef RTFLUSH_DEBUG @@ -1203,16 +1203,6 @@ rt_ifa_remlocal(struct ifaddr *ifa, stru LIST_HEAD(, rttimer_queue) rttimer_queue_head; static int rt_init_done = 0; -#define RTTIMER_CALLOUT(r) do { \ - if (r->rtt_func != NULL) { \ - (*r->rtt_func)(r->rtt_rt, r); \ - } else { \ - rtrequest((int) RTM_DELETE, \ - rt_getkey(r->rtt_rt), \ - 0, 0, 0, 0); \ - } \ - } while (/*CONSTCOND*/0) - /* * Some subtle order problems with domain initialization mean that * we cannot count on this being run from rt_init before various @@ -1267,7 +1257,7 @@ rt_timer_queue_remove_all(struct rttimer LIST_REMOVE(r, rtt_link); TAILQ_REMOVE(&rtq->rtq_head, r, rtt_next); if (destroy) - RTTIMER_CALLOUT(r); + (*r->rtt_func)(r->rtt_rt, r); rtfree(r->rtt_rt); /* we are already at splsoftnet */ pool_put(&rttimer_pool, r); @@ -1307,7 +1297,7 @@ rt_timer_remove_all(struct rtentry *rt, LIST_REMOVE(r, rtt_link); TAILQ_REMOVE(&r->rtt_queue->rtq_head, r, rtt_next); if (destroy) - RTTIMER_CALLOUT(r); + (*r->rtt_func)(r->rtt_rt, r); if (r->rtt_queue->rtq_count > 0) r->rtt_queue->rtq_count--; else @@ -1326,6 +1316,7 @@ rt_timer_add(struct rtentry *rt, struct rttimer *r; int s; + KASSERT(func != NULL); /* * If there's already a timer with this action, destroy it before * we add a new one. @@ -1378,7 +1369,7 @@ rt_timer_timer(void *arg) (r->rtt_time + rtq->rtq_timeout) < time_uptime) { LIST_REMOVE(r, rtt_link); TAILQ_REMOVE(&rtq->rtq_head, r, rtt_next); - RTTIMER_CALLOUT(r); + (*r->rtt_func)(r->rtt_rt, r); rtfree(r->rtt_rt); pool_put(&rttimer_pool, r); if (rtq->rtq_count > 0)