diff options
| author |   mpi <mpi@openbsd.org> | 2015-01-13 12:16:18 +0000 |
|---|---|---|
| committer | mpi <mpi@openbsd.org> | 2015-01-13 12:16:18 +0000 |
| commit | c97481e527693f6399e288020ba9fbae554b53b4 (patch) | |
| tree | 3dfaa07635f0f7eaac825fbf1a62bb020a588018 | |
| parent | Restore sending RTM_ADD and RTM_DELETE messages to userland for every (diff) | |
| download | wireguard-openbsd-c97481e527693f6399e288020ba9fbae554b53b4.tar.xz wireguard-openbsd-c97481e527693f6399e288020ba9fbae554b53b4.zip | |
Only notify userland about resolved ARP entries if the Ethernet address
changed or if we asked for it.
Should reduce the RTM_RESOLVE storm seeing by claudio@
With inputs from and ok mikeb@
| -rw-r--r-- | sys/netinet/if_ether.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/sys/netinet/if_ether.c b/sys/netinet/if_ether.c index 4f70ce698d0..01458833ac9 100644 --- a/sys/netinet/if_ether.c +++ b/sys/netinet/if_ether.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if_ether.c,v 1.140 2015/01/08 14:29:18 mpi Exp $ */ +/* $OpenBSD: if_ether.c,v 1.141 2015/01/13 12:16:18 mpi Exp $ */ /* $NetBSD: if_ether.c,v 1.31 1996/05/11 12:59:58 mycroft Exp $ */ /* @@ -560,7 +560,7 @@ in_arpinput(struct mbuf *m) u_int8_t *ether_shost = NULL; #endif char addr[INET_ADDRSTRLEN]; - int op; + int op, changed = 0; ea = mtod(m, struct ether_arp *); op = ntohs(ea->arp_op); @@ -684,6 +684,7 @@ in_arpinput(struct mbuf *m) ac->ac_if.if_xname); rt->rt_expire = 1; /* no longer static */ } + changed = 1; } } else if (rt->rt_ifp != &ac->ac_if && #if NBRIDGE > 0 @@ -711,8 +712,10 @@ in_arpinput(struct mbuf *m) if (rt->rt_expire) rt->rt_expire = time_second + arpt_keep; rt->rt_flags &= ~RTF_REJECT; + /* Notify userland that an ARP resolution has been done. */ + if (la->la_asked || changed) + rt_sendmsg(rt, RTM_RESOLVE, rt->rt_ifp->if_rdomain); la->la_asked = 0; - rt_sendmsg(rt, RTM_RESOLVE, rt->rt_ifp->if_rdomain); while ((mh = la->la_hold_head) != NULL) { if ((la->la_hold_head = mh->m_nextpkt) == NULL) la->la_hold_tail = NULL; |