Nuke code dealing with empty address list or trying to set the link-layer
authorJoerg Sonnenberger <joerg@dragonflybsd.org>
Tue, 14 Jun 2005 11:41:37 +0000 (11:41 +0000)
committerJoerg Sonnenberger <joerg@dragonflybsd.org>
Tue, 14 Jun 2005 11:41:37 +0000 (11:41 +0000)
address by hand.

sys/dev/netif/cs/if_cs.c
sys/dev/netif/ed/if_ed.c
sys/dev/netif/el/if_el.c
sys/dev/netif/ex/if_ex.c
sys/dev/netif/fe/if_fe.c
sys/dev/netif/lnc/if_lnc.c
sys/dev/netif/rdp/if_rdp.c
sys/dev/netif/sbni/if_sbni.c
sys/dev/netif/sn/if_sn.c
sys/dev/netif/wl/if_wl.c
sys/dev/netif/xe/if_xe.c

index 94bb2c4..b6360d7 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/cs/if_cs.c,v 1.19.2.1 2001/01/25 20:13:48 imp Exp $
- * $DragonFly: src/sys/dev/netif/cs/if_cs.c,v 1.17 2005/06/13 21:44:40 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/cs/if_cs.c,v 1.18 2005/06/14 11:41:37 joerg Exp $
  */
 
 /*
@@ -684,12 +684,6 @@ cs_init(void *xsc)
 
        crit_enter();
 
-       /* address not known */
-       if (TAILQ_EMPTY(&ifp->if_addrhead)) { /* unlikely? XXX */
-               crit_exit();
-               return;
-       }
-
        /*
         * reset whatchdog timer
         */
index 9833d3f..177617e 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/ed/if_ed.c,v 1.224 2003/12/08 07:54:12 obrien Exp $
- * $DragonFly: src/sys/dev/netif/ed/if_ed.c,v 1.23 2005/06/14 11:08:40 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/ed/if_ed.c,v 1.24 2005/06/14 11:41:37 joerg Exp $
  */
 
 /*
@@ -1897,12 +1897,6 @@ ed_init(xsc)
                return;
        }
 
-       /* address not known */
-       if (TAILQ_EMPTY(&ifp->if_addrhead)) { /* unlikely? XXX */
-               crit_exit();
-               return;
-       }
-
        /*
         * Initialize the NIC in the exact order outlined in the NS manual.
         * This init procedure is "mandatory"...don't change what or when
index 4f06b91..9d1ed48 100644 (file)
@@ -7,7 +7,7 @@
  * Questions, comments, bug reports and fixes to kimmel@cs.umass.edu.
  *
  * $FreeBSD: src/sys/i386/isa/if_el.c,v 1.47.2.2 2000/07/17 21:24:30 archie Exp $
- * $DragonFly: src/sys/dev/netif/el/if_el.c,v 1.14 2005/05/27 15:36:09 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/el/if_el.c,v 1.15 2005/06/14 11:41:37 joerg Exp $
  */
 /* Except of course for the portions of code lifted from other FreeBSD
  * drivers (mainly elread, elget and el_ioctl)
@@ -241,10 +241,6 @@ el_init(xsc)
        ifp = &sc->arpcom.ac_if;
        base = sc->el_base;
 
-       /* If address not known, do nothing. */
-       if(TAILQ_EMPTY(&ifp->if_addrhead)) /* XXX unlikely */
-               return;
-
        s = splimp();
 
        /* First, reset the board. */
index 841a65f..2aa20f8 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/ex/if_ex.c,v 1.26.2.3 2001/03/05 05:33:20 imp Exp $
- * $DragonFly: src/sys/dev/netif/ex/if_ex.c,v 1.17 2005/05/27 15:36:09 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/ex/if_ex.c,v 1.18 2005/06/14 11:41:37 joerg Exp $
  *
  * MAINTAINER: Matthew N. Dodd <winter@jurai.net>
  *                             <mdodd@FreeBSD.org>
@@ -284,9 +284,6 @@ ex_init(void *xsc)
 
        DODEBUG(Start_End, printf("ex_init%d: start\n", ifp->if_dunit););
 
-       if (TAILQ_FIRST(&ifp->if_addrhead) == NULL) {
-               return;
-       }
        s = splimp();
        ifp->if_timer = 0;
 
index 021290e..60ab3fa 100644 (file)
@@ -22,7 +22,7 @@
 
 /*
  * $FreeBSD: src/sys/dev/fe/if_fe.c,v 1.65.2.1 2000/09/22 10:01:47 nyan Exp $
- * $DragonFly: src/sys/dev/netif/fe/if_fe.c,v 1.16 2005/05/27 15:36:09 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/fe/if_fe.c,v 1.17 2005/06/14 11:41:37 joerg Exp $
  *
  * Device driver for Fujitsu MB86960A/MB86965A based Ethernet cards.
  * Contributed by M. Sekiguchi. <seki@sysrap.cs.fujitsu.co.jp>
@@ -1009,14 +1009,6 @@ fe_init (void * xsc)
        struct fe_softc *sc = xsc;
        int s;
 
-       /* We need an address. */
-       if (TAILQ_EMPTY(&sc->sc_if.if_addrhead)) { /* XXX unlikely */
-#ifdef DIAGNOSTIC
-               printf("fe%d: init() without any address\n", sc->sc_unit);
-#endif
-               return;
-       }
-
        /* Start initializing 86960.  */
        s = splimp();
 
index 4c1e12c..90132ea 100644 (file)
@@ -28,7 +28,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/lnc/if_lnc.c,v 1.89 2001/07/04 13:00:19 nyan Exp $
- * $DragonFly: src/sys/dev/netif/lnc/Attic/if_lnc.c,v 1.21 2005/06/13 22:55:15 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/lnc/Attic/if_lnc.c,v 1.22 2005/06/14 11:41:37 joerg Exp $
  */
 
 /*
@@ -894,14 +894,6 @@ lnc_init(xsc)
 
        crit_enter();
 
-       /* Check that interface has valid address */
-
-       if (TAILQ_EMPTY(&sc->arpcom.ac_if.if_addrhead)) { /* XXX unlikely */
-               printf("XXX no address?\n");
-               crit_exit();
-               return;
-       }
-
        /* Shut down interface */
        lnc_stop(sc);
        sc->arpcom.ac_if.if_flags |= IFF_BROADCAST | IFF_SIMPLEX | IFF_MULTICAST; /* XXX??? */
index bee57bf..6a544cb 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/i386/isa/if_rdp.c,v 1.6.2.2 2000/07/17 21:24:32 archie Exp $
- * $DragonFly: src/sys/dev/netif/rdp/if_rdp.c,v 1.16 2005/06/13 13:31:04 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/rdp/if_rdp.c,v 1.17 2005/06/14 11:41:37 joerg Exp $
  */
 
 /*
@@ -694,10 +694,6 @@ rdp_init(void *xsc)
 
        crit_enter();
 
-       /* address not known */
-       if (TAILQ_EMPTY(&ifp->if_addrhead))
-               return;
-
        ifp->if_timer = 0;
 
        /* program ethernet ID into the chip */
index 0c3f74c..fb0ff88 100644 (file)
@@ -25,7 +25,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/sbni/if_sbni.c,v 1.1.2.4 2002/08/11 09:32:00 fjoe Exp $
- * $DragonFly: src/sys/dev/netif/sbni/if_sbni.c,v 1.20 2005/06/13 13:47:50 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/sbni/if_sbni.c,v 1.21 2005/06/14 11:41:37 joerg Exp $
  */
 
 /*
@@ -266,10 +266,6 @@ sbni_init(void *xsc)
 
        crit_enter();
 
-       /* address not known */
-       if (TAILQ_EMPTY(&ifp->if_addrhead))
-               return;
-
        /*
         * kludge to avoid multiple initialization when more than once
         * protocols configured
index 2743bc0..861f145 100644 (file)
@@ -29,7 +29,7 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  *   $FreeBSD: src/sys/dev/sn/if_sn.c,v 1.7.2.3 2001/02/04 04:38:38 toshi Exp $
- *   $DragonFly: src/sys/dev/netif/sn/if_sn.c,v 1.18 2005/06/12 17:03:47 joerg Exp $
+ *   $DragonFly: src/sys/dev/netif/sn/if_sn.c,v 1.19 2005/06/14 11:41:37 joerg Exp $
  */
 
 /*
@@ -230,24 +230,6 @@ sn_attach(device_t dev)
 
        ether_ifattach(ifp, sc->arpcom.ac_enaddr);
 
-       /*
-        * Fill the hardware address into ifa_addr if we find an AF_LINK
-        * entry. We need to do this so bpf's can get the hardware addr of
-        * this card. netstat likes this too!
-        */
-       ifa = TAILQ_FIRST(&ifp->if_addrhead);
-       while ((ifa != 0) && (ifa->ifa_addr != 0) &&
-              (ifa->ifa_addr->sa_family != AF_LINK))
-               ifa = TAILQ_NEXT(ifa, ifa_link);
-
-       if ((ifa != 0) && (ifa->ifa_addr != 0)) {
-               sdl = (struct sockaddr_dl *) ifa->ifa_addr;
-               sdl->sdl_type = IFT_ETHER;
-               sdl->sdl_alen = ETHER_ADDR_LEN;
-               sdl->sdl_slen = 0;
-               bcopy(sc->arpcom.ac_enaddr, LLADDR(sdl), ETHER_ADDR_LEN);
-       }
-
        return 0;
 }
 
index fa537c3..8d826ca 100644 (file)
@@ -1,5 +1,5 @@
 /* $FreeBSD: src/sys/i386/isa/if_wl.c,v 1.27.2.2 2000/07/17 21:24:32 archie Exp $ */
-/* $DragonFly: src/sys/dev/netif/wl/if_wl.c,v 1.19 2005/06/06 23:12:07 okumoto Exp $ */
+/* $DragonFly: src/sys/dev/netif/wl/if_wl.c,v 1.20 2005/06/14 11:41:37 joerg Exp $ */
 /* 
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -673,13 +673,6 @@ wlinit(void *xsc)
     if (sc->wl_if.if_flags & IFF_DEBUG)
        printf("wl%d: entered wlinit()\n",sc->unit);
 #endif
-#if defined(__DragonFly__) || (defined(__FreeBSD__) && __FreeBSD_version >= 300000)
-    if (TAILQ_EMPTY(&ifp->if_addrhead)) {
-#else
-    if (ifp->if_addrlist == (struct ifaddr *)0) {
-#endif
-       return;
-    }
     crit_enter();
     if ((stat = wlhwrst(sc->unit)) == TRUE) {
        sc->wl_if.if_flags |= IFF_RUNNING;   /* same as DSF_RUNNING */
index 268d0b2..0a3e262 100644 (file)
@@ -25,7 +25,7 @@
  *
  *     $Id: if_xe.c,v 1.20 1999/06/13 19:17:40 scott Exp $
  * $FreeBSD: src/sys/dev/xe/if_xe.c,v 1.13.2.6 2003/02/05 22:03:57 mbr Exp $
- * $DragonFly: src/sys/dev/netif/xe/if_xe.c,v 1.20 2005/06/06 23:12:07 okumoto Exp $
+ * $DragonFly: src/sys/dev/netif/xe/if_xe.c,v 1.21 2005/06/14 11:41:37 joerg Exp $
  */
 
 /*
@@ -602,8 +602,6 @@ xe_init(void *xscp) {
   device_printf(scp->dev, "init\n");
 #endif
 
-  if (TAILQ_EMPTY(&scp->ifp->if_addrhead)) return;
-
   /* Reset transmitter flags */
   scp->tx_queued = 0;
   scp->tx_tpr = 0;