Use queue(3) macros for if_multiaddrs.
authorJoerg Sonnenberger <joerg@dragonflybsd.org>
Mon, 20 Jun 2005 15:10:41 +0000 (15:10 +0000)
committerJoerg Sonnenberger <joerg@dragonflybsd.org>
Mon, 20 Jun 2005 15:10:41 +0000 (15:10 +0000)
15 files changed:
sys/dev/netif/dc/if_dc.c
sys/dev/netif/de/if_de.c
sys/dev/netif/ed/if_ed.c
sys/dev/netif/fe/if_fe.c
sys/dev/netif/ie/if_ie.c
sys/dev/netif/le/if_le.c
sys/dev/netif/pcn/if_pcn.c
sys/dev/netif/pdq_layer/pdq_ifsubr.c
sys/dev/netif/ray/if_ray.c
sys/dev/netif/sn/if_sn.c
sys/dev/netif/snc/dp83932.c
sys/dev/netif/ste/if_ste.c
sys/dev/netif/txp/if_txp.c
sys/dev/netif/wl/if_wl.c
sys/dev/netif/xl/if_xl.c

index c210b41..1385503 100644 (file)
@@ -30,7 +30,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/pci/if_dc.c,v 1.9.2.45 2003/06/08 14:31:53 mux Exp $
- * $DragonFly: src/sys/dev/netif/dc/if_dc.c,v 1.34 2005/06/14 14:19:22 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/dc/if_dc.c,v 1.35 2005/06/20 15:10:40 joerg Exp $
  */
 
 /*
@@ -1065,8 +1065,7 @@ void dc_setfilt_21143(sc)
                DC_CLRBIT(sc, DC_NETCFG, DC_NETCFG_RX_ALLMULTI);
 
        crc_mask = dc_crc_mask(sc);
-       for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-           ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
                h = ether_crc32_le(
@@ -1144,8 +1143,7 @@ void dc_setfilt_admtek(sc)
                crc_mask = dc_crc_mask(sc);
        else
                crc_mask = 0x3f;
-       for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-           ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
                if (DC_IS_CENTAUR(sc)) {
@@ -1222,8 +1220,7 @@ void dc_setfilt_asix(sc)
                return;
 
        /* now program new ones */
-       for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-           ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
                h = ether_crc32_be(
index 2976cbc..a6ff7c8 100644 (file)
@@ -1,7 +1,7 @@
 /*     $NetBSD: if_de.c,v 1.86 1999/06/01 19:17:59 thorpej Exp $       */
 
 /* $FreeBSD: src/sys/pci/if_de.c,v 1.123.2.4 2000/08/04 23:25:09 peter Exp $ */
-/* $DragonFly: src/sys/dev/netif/de/if_de.c,v 1.38 2005/06/13 23:03:15 joerg Exp $ */
+/* $DragonFly: src/sys/dev/netif/de/if_de.c,v 1.39 2005/06/20 15:10:40 joerg Exp $ */
 
 /*-
  * Copyright (c) 1994-1997 Matt Thomas (matt@3am-software.com)
@@ -2758,9 +2758,7 @@ tulip_addr_filter(tulip_softc_t *sc)
 #endif
 
     multicnt = 0;
-    for (ifma = sc->tulip_if.if_multiaddrs.lh_first; ifma != NULL;
-        ifma = ifma->ifma_link.le_next) {
-
+    LIST_FOREACH(ifma, &sc->tulip_if.if_multiaddrs, ifma_link) {
            if (ifma->ifma_addr->sa_family == AF_LINK)
                multicnt++;
     }
@@ -2787,9 +2785,7 @@ tulip_addr_filter(tulip_softc_t *sc)
         */
        bzero(sc->tulip_setupdata, sizeof(sc->tulip_setupdata));
 
-       for (ifma = sc->tulip_if.if_multiaddrs.lh_first; ifma != NULL;
-            ifma = ifma->ifma_link.le_next) {
-
+       LIST_FOREACH(ifma, &sc->tulip_if.if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
 
@@ -2838,8 +2834,7 @@ tulip_addr_filter(tulip_softc_t *sc)
            /*
             * Else can get perfect filtering for 16 addresses.
             */
-           for (ifma = sc->tulip_if.if_multiaddrs.lh_first; ifma != NULL;
-                ifma = ifma->ifma_link.le_next) {
+           LIST_FOREACH(ifma, &sc->tulip_if.if_multiaddrs, ifma_link) {
                    if (ifma->ifma_addr->sa_family != AF_LINK)
                            continue;
                    addrp = LLADDR((struct sockaddr_dl *)ifma->ifma_addr);
index 177617e..06f36e2 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.24 2005/06/14 11:41:37 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/ed/if_ed.c,v 1.25 2005/06/20 15:10:40 joerg Exp $
  */
 
 /*
@@ -3564,8 +3564,7 @@ ds_getmcaf(sc, mcaf)
        mcaf[0] = 0;
        mcaf[1] = 0;
 
-       for (ifma = sc->arpcom.ac_if.if_multiaddrs.lh_first; ifma;
-            ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &sc->arpcom.ac_if.if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
                index = ds_mchash(LLADDR((struct sockaddr_dl *)ifma->ifma_addr))
index 2fb9a6d..d769209 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.18 2005/06/14 15:19:28 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/fe/if_fe.c,v 1.19 2005/06/20 15:10:40 joerg Exp $
  *
  * Device driver for Fujitsu MB86960A/MB86965A based Ethernet cards.
  * Contributed by M. Sekiguchi. <seki@sysrap.cs.fujitsu.co.jp>
@@ -2088,8 +2088,7 @@ fe_mcaf ( struct fe_softc *sc )
        struct ifmultiaddr *ifma;
 
        filter = fe_filter_nothing;
-       for (ifma = sc->arpcom.ac_if.if_multiaddrs.lh_first; ifma;
-            ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &sc->arpcom.ac_if.if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
                index = fe_hash(LLADDR((struct sockaddr_dl *)ifma->ifma_addr));
index 4d5ab72..8a2d569 100644 (file)
@@ -48,7 +48,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/ie/if_ie.c,v 1.72.2.4 2003/03/27 21:01:49 mdodd Exp $
- * $DragonFly: src/sys/dev/netif/ie/if_ie.c,v 1.19 2005/06/14 15:30:58 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/ie/if_ie.c,v 1.20 2005/06/20 15:10:41 joerg Exp $
  */
 
 /*
@@ -2165,8 +2165,7 @@ ie_mc_reset(int unit)
         * Step through the list of addresses.
         */
        ie->mcast_count = 0;
-       for (ifma = ie->arpcom.ac_if.if_multiaddrs.lh_first; ifma;
-            ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ie->arpcom.ac_if.if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
 
index 8e78b2d..2ef14e6 100644 (file)
@@ -22,7 +22,7 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/i386/isa/if_le.c,v 1.56.2.4 2002/06/05 23:24:10 paul Exp $
- * $DragonFly: src/sys/dev/netif/le/if_le.c,v 1.25 2005/06/14 15:11:23 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/le/if_le.c,v 1.26 2005/06/20 15:10:41 joerg Exp $
  */
 
 /*
@@ -482,8 +482,7 @@ le_multi_filter(struct le_softc *sc)
 
     sc->le_flags |= IFF_MULTICAST;
 
-    for (ifma = ifp->if_multiaddrs.lh_first; ifma;
-        ifma = ifma->ifma_link.le_next) {
+    LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
            if (ifma->ifma_addr->sa_family != AF_LINK)
                    continue;
 
index 8a94720..2fe095b 100644 (file)
@@ -31,7 +31,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/pci/if_pcn.c,v 1.5.2.10 2003/03/05 18:42:33 njl Exp $
- * $DragonFly: src/sys/dev/netif/pcn/if_pcn.c,v 1.21 2005/06/13 18:43:58 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/pcn/if_pcn.c,v 1.22 2005/06/20 15:10:41 joerg Exp $
  */
 
 /*
@@ -342,8 +342,7 @@ static void pcn_setmulti(sc)
                pcn_csr_write(sc, PCN_CSR_MAR0 + i, 0);
 
        /* now program new ones */
-       for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-           ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
                h = pcn_crc(LLADDR((struct sockaddr_dl *)ifma->ifma_addr));
index 6c3789b..6bd817b 100644 (file)
@@ -22,7 +22,7 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/pdq/pdq_ifsubr.c,v 1.11.2.1 2000/08/02 22:39:30 peter Exp $
- * $DragonFly: src/sys/dev/netif/pdq_layer/Attic/pdq_ifsubr.c,v 1.12 2005/06/13 18:13:20 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/pdq_layer/Attic/pdq_ifsubr.c,v 1.13 2005/06/20 15:10:41 joerg Exp $
  *
  */
 
@@ -222,8 +222,7 @@ pdq_os_addr_fill(
     pdq_softc_t *sc = (pdq_softc_t *) pdq->pdq_os_ctx;
     struct ifmultiaddr *ifma;
 
-    for (ifma = sc->sc_if.if_multiaddrs.lh_first; ifma && num_addrs > 0;
-        ifma = ifma->ifma_link.le_next) {
+    LIST_FOREACH(ifma, &sc->sc_if.if_multiaddrs, ifma_link) {
            char *mcaddr;
            if (ifma->ifma_addr->sa_family != AF_LINK)
                    continue;
index 652a70a..1c10be0 100644 (file)
@@ -29,7 +29,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/ray/if_ray.c,v 1.47.2.4 2001/08/14 22:54:05 dmlb Exp $
- * $DragonFly: src/sys/dev/netif/ray/Attic/if_ray.c,v 1.22 2005/06/13 19:19:19 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/ray/Attic/if_ray.c,v 1.23 2005/06/20 15:10:41 joerg Exp $
  *
  */
 
@@ -2685,8 +2685,7 @@ ray_mcast(struct ray_softc *sc, struct ray_comq_entry *com)
         * The multicast list is only 16 items long so use promiscuous
         * mode and don't bother updating the multicast list.
         */
-       for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-           ifma = ifma->ifma_link.le_next)
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link)
                count++;
        if (count == 0) {
                ray_com_runq_done(sc);
@@ -2705,8 +2704,7 @@ ray_mcast(struct ray_softc *sc, struct ray_comq_entry *com)
        SRAM_WRITE_FIELD_1(sc, com->c_ccs,
            ray_cmd_update_mcast, c_nmcast, count);
        bufp = RAY_HOST_TO_ECF_BASE;
-       for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-           ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
                SRAM_WRITE_REGION(
                    sc,
                    bufp,
index 6970d93..6117da8 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.20 2005/06/16 14:51:55 joerg Exp $
+ *   $DragonFly: src/sys/dev/netif/sn/if_sn.c,v 1.21 2005/06/20 15:10:41 joerg Exp $
  */
 
 /*
@@ -1415,8 +1415,7 @@ sn_getmcf(struct arpcom *ac, u_char *mcf)
 
        bzero(mcf, MCFSZ);
 
-       for (ifma = ac->ac_if.if_multiaddrs.lh_first; ifma;
-            ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ac->ac_if.if_multiaddrs, ifma_link) {
            if (ifma->ifma_addr->sa_family != AF_LINK)
                return 0;
            index = smc_crc(LLADDR((struct sockaddr_dl *)ifma->ifma_addr)) & 0x3f;
index bed09e1..cd5e14e 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/dev/snc/dp83932.c,v 1.1.2.2 2003/02/11 08:52:00 nyan Exp $    */
-/*     $DragonFly: src/sys/dev/netif/snc/Attic/dp83932.c,v 1.15 2005/06/12 16:31:50 joerg Exp $        */
+/*     $DragonFly: src/sys/dev/netif/snc/Attic/dp83932.c,v 1.16 2005/06/20 15:10:41 joerg Exp $        */
 /*     $NecBSD: dp83932.c,v 1.5 1999/07/29 05:08:44 kmatsuda Exp $     */
 /*     $NetBSD: if_snc.c,v 1.18 1998/04/25 21:27:40 scottr Exp $       */
 
@@ -680,8 +680,7 @@ camprogram(sc)
        ifp->if_flags &= ~IFF_ALLMULTI;
 
        /* Loop through multicast addresses */
-        for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-                                ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
                 if (ifma->ifma_addr->sa_family != AF_LINK)
                         continue;
                if (mcount == MAXCAM) {
index 4e581c8..2d7b466 100644 (file)
@@ -30,7 +30,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/pci/if_ste.c,v 1.14.2.9 2003/02/05 22:03:57 mbr Exp $
- * $DragonFly: src/sys/dev/netif/ste/if_ste.c,v 1.27 2005/06/14 14:19:22 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/ste/if_ste.c,v 1.28 2005/06/20 15:10:41 joerg Exp $
  */
 
 #include <sys/param.h>
@@ -573,8 +573,7 @@ static void ste_setmulti(sc)
        CSR_WRITE_2(sc, STE_MAR3, 0);
 
        /* now program new ones */
-       for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-           ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
                h = ether_crc32_be(
index a7e982f..6f18293 100644 (file)
@@ -1,6 +1,6 @@
 /*     $OpenBSD: if_txp.c,v 1.48 2001/06/27 06:34:50 kjc Exp $ */
 /*     $FreeBSD: src/sys/dev/txp/if_txp.c,v 1.4.2.4 2001/12/14 19:50:43 jlemon Exp $ */
-/*     $DragonFly: src/sys/dev/netif/txp/if_txp.c,v 1.29 2005/06/20 13:56:08 joerg Exp $ */
+/*     $DragonFly: src/sys/dev/netif/txp/if_txp.c,v 1.30 2005/06/20 15:10:41 joerg Exp $ */
 
 /*
  * Copyright (c) 2001
@@ -1728,8 +1728,7 @@ txp_set_filter(sc)
 
                hash[0] = hash[1] = 0;
 
-               for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-                   ifma = ifma->ifma_link.le_next) {
+               LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
                        if (ifma->ifma_addr->sa_family != AF_LINK)
                                continue;
 
index 8d826ca..3f09c70 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.20 2005/06/14 11:41:37 joerg Exp $ */
+/* $DragonFly: src/sys/dev/netif/wl/if_wl.c,v 1.21 2005/06/20 15:10:41 joerg Exp $ */
 /* 
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -2037,8 +2037,7 @@ wlconfig(int unit)
     outw(PIOP1(base), AC_MCSETUP|AC_CW_EL);            /* ac_command */
     outw(PIOR1(base), OFFSET_CU + 8);
 #if defined(__DragonFly__) || (defined(__FreeBSD__) && __FreeBSD_version >= 300000)
-    for (ifma = sc->wl_if.if_multiaddrs.lh_first; ifma;
-        ifma = ifma->ifma_link.le_next) {
+    LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
        if (ifma->ifma_addr->sa_family != AF_LINK)
            continue;
        
index c202c4a..1d37797 100644 (file)
@@ -30,7 +30,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/pci/if_xl.c,v 1.72.2.28 2003/10/08 06:01:57 murray Exp $
- * $DragonFly: src/sys/dev/netif/xl/if_xl.c,v 1.26 2005/06/14 14:19:22 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/xl/if_xl.c,v 1.27 2005/06/20 15:10:41 joerg Exp $
  */
 
 /*
@@ -851,8 +851,7 @@ xl_setmulti(sc)
                return;
        }
 
-       for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-                               ifma = ifma->ifma_link.le_next)
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link)
                mcnt++;
 
        if (mcnt)
@@ -896,8 +895,7 @@ xl_setmulti_hash(sc)
                CSR_WRITE_2(sc, XL_COMMAND, XL_CMD_RX_SET_HASH|i);
 
        /* now program new ones */
-        for (ifma = ifp->if_multiaddrs.lh_first; ifma != NULL;
-                                ifma = ifma->ifma_link.le_next) {
+       LIST_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link) {
                if (ifma->ifma_addr->sa_family != AF_LINK)
                        continue;
                h = xl_calchash(LLADDR((struct sockaddr_dl *)ifma->ifma_addr));