--- src/sys/netinet/if_ether.c 2006/05/20 06:32:40 1.32 +++ src/sys/netinet/if_ether.c 2006/06/25 11:02:40 1.33 @@ -92,7 +92,6 @@ */ #include "opt_inet.h" -#include "opt_bdg.h" #include #include @@ -113,10 +112,6 @@ #include #include #include -#ifdef BRIDGE -#include -#include -#endif #include #include @@ -599,12 +594,6 @@ SYSCTL_INT(_net_link_ether_inet, OID_AUT &log_arp_wrong_iface, 0, "log arp packets arriving on the wrong interface"); -#ifdef BRIDGE -#define BRIDGE_TEST (do_bridge) -#else -#define BRIDGE_TEST (0) /* cc will optimise the test away */ -#endif - static void arp_update_oncpu(struct mbuf *m, in_addr_t saddr, boolean_t create, boolean_t dologging) @@ -621,7 +610,7 @@ arp_update_oncpu(struct mbuf *m, in_addr struct in_addr isaddr = { saddr }; /* the following is not an error when doing bridging */ - if (!BRIDGE_TEST && rt->rt_ifp != ifp) { + if (rt->rt_ifp != ifp) { if (dologging && log_arp_wrong_iface && cpu == 0) { log(LOG_ERR, "arp: %s is on %s " @@ -784,17 +773,6 @@ in_arpinput(struct mbuf *m) * when we have clusters of interfaces). */ LIST_FOREACH(ia, INADDR_HASH(itaddr.s_addr), ia_hash) { - /* - * Old style bridging OBSOLETE - */ - if ((BRIDGE_TEST || (ia->ia_ifp == ifp)) && - itaddr.s_addr == ia->ia_addr.sin_addr.s_addr) { - goto match; - } - - /* - * New style bridging - */ if (ifp->if_bridge && ia->ia_ifp && ifp->if_bridge == ia->ia_ifp->if_bridge && itaddr.s_addr == ia->ia_addr.sin_addr.s_addr) { @@ -802,17 +780,6 @@ in_arpinput(struct mbuf *m) } } LIST_FOREACH(ia, INADDR_HASH(isaddr.s_addr), ia_hash) { - /* - * Old style bridging OBSOLETE - */ - if ((BRIDGE_TEST || (ia->ia_ifp == ifp)) && - isaddr.s_addr == ia->ia_addr.sin_addr.s_addr) { - goto match; - } - - /* - * New style bridging - */ if (ifp->if_bridge && ia->ia_ifp && ifp->if_bridge == ia->ia_ifp->if_bridge && isaddr.s_addr == ia->ia_addr.sin_addr.s_addr) { @@ -828,14 +795,7 @@ in_arpinput(struct mbuf *m) ia = ifatoia(ifa); goto match; } - /* - * If bridging, fall back to using any inet address. - * This is probably incorrect, the right way being try to match - * addresses for interfaces in the same cluster, so if we - * get here we should always drop the packet. - */ - if (!BRIDGE_TEST || - (ia = TAILQ_FIRST(&in_ifaddrhead)) == NULL) { + if ((ia = TAILQ_FIRST(&in_ifaddrhead)) == NULL) { m_freem(m); return; }