The branch main has been updated by jhibbits:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=3f14814f05b5085b408ea25ae0bca68f4603c0ea

commit 3f14814f05b5085b408ea25ae0bca68f4603c0ea
Author:     Justin Hibbits <jhibb...@freebsd.org>
AuthorDate: 2021-10-26 16:45:58 +0000
Commit:     Justin Hibbits <jhibb...@freebsd.org>
CommitDate: 2023-02-06 17:32:05 +0000

    Mechanically convert ae(4) to IfAPI
    
    Sponsored by:   Juniper Networks, Inc.
    Differential Revision: https://reviews.freebsd.org/D37786
---
 sys/dev/ae/if_ae.c    | 163 +++++++++++++++++++++++++-------------------------
 sys/dev/ae/if_aevar.h |   2 +-
 2 files changed, 82 insertions(+), 83 deletions(-)

diff --git a/sys/dev/ae/if_ae.c b/sys/dev/ae/if_ae.c
index 1ffd8b06e522..72d4fdca6aa0 100644
--- a/sys/dev/ae/if_ae.c
+++ b/sys/dev/ae/if_ae.c
@@ -114,8 +114,8 @@ static int  ae_detach(device_t dev);
 static int     ae_miibus_readreg(device_t dev, int phy, int reg);
 static int     ae_miibus_writereg(device_t dev, int phy, int reg, int val);
 static void    ae_miibus_statchg(device_t dev);
-static void    ae_mediastatus(struct ifnet *ifp, struct ifmediareq *ifmr);
-static int     ae_mediachange(struct ifnet *ifp);
+static void    ae_mediastatus(if_t ifp, struct ifmediareq *ifmr);
+static int     ae_mediachange(if_t ifp);
 static void    ae_retrieve_address(ae_softc_t *sc);
 static void    ae_dmamap_cb(void *arg, bus_dma_segment_t *segs, int nsegs,
     int error);
@@ -128,8 +128,8 @@ static void ae_powersave_enable(ae_softc_t *sc);
 static int     ae_resume(device_t dev);
 static unsigned int    ae_tx_avail_size(ae_softc_t *sc);
 static int     ae_encap(ae_softc_t *sc, struct mbuf **m_head);
-static void    ae_start(struct ifnet *ifp);
-static void    ae_start_locked(struct ifnet *ifp);
+static void    ae_start(if_t ifp);
+static void    ae_start_locked(if_t ifp);
 static void    ae_link_task(void *arg, int pending);
 static void    ae_stop_rxmac(ae_softc_t *sc);
 static void    ae_stop_txmac(ae_softc_t *sc);
@@ -143,7 +143,7 @@ static void ae_watchdog(ae_softc_t *sc);
 static void    ae_tick(void *arg);
 static void    ae_rxfilter(ae_softc_t *sc);
 static void    ae_rxvlan(ae_softc_t *sc);
-static int     ae_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data);
+static int     ae_ioctl(if_t ifp, u_long cmd, caddr_t data);
 static void    ae_stop(ae_softc_t *sc);
 static int     ae_check_eeprom_present(ae_softc_t *sc, int *vpdc);
 static int     ae_vpd_read_word(ae_softc_t *sc, int reg, uint32_t *word);
@@ -238,7 +238,7 @@ static int
 ae_attach(device_t dev)
 {
        ae_softc_t *sc;
-       struct ifnet *ifp;
+       if_t ifp;
        uint8_t chiprev;
        uint32_t pcirev;
        int nmsi, pmc;
@@ -335,22 +335,21 @@ ae_attach(device_t dev)
                goto fail;
        }
 
-       ifp->if_softc = sc;
+       if_setsoftc(ifp, sc);
        if_initname(ifp, device_get_name(dev), device_get_unit(dev));
-       ifp->if_flags = IFF_BROADCAST | IFF_SIMPLEX | IFF_MULTICAST;
-       ifp->if_ioctl = ae_ioctl;
-       ifp->if_start = ae_start;
-       ifp->if_init = ae_init;
-       ifp->if_capabilities = IFCAP_VLAN_MTU | IFCAP_VLAN_HWTAGGING;
-       ifp->if_hwassist = 0;
-       ifp->if_snd.ifq_drv_maxlen = ifqmaxlen;
-       IFQ_SET_MAXLEN(&ifp->if_snd, ifp->if_snd.ifq_drv_maxlen);
-       IFQ_SET_READY(&ifp->if_snd);
+       if_setflags(ifp, IFF_BROADCAST | IFF_SIMPLEX | IFF_MULTICAST);
+       if_setioctlfn(ifp, ae_ioctl);
+       if_setstartfn(ifp, ae_start);
+       if_setinitfn(ifp, ae_init);
+       if_setcapabilities(ifp, IFCAP_VLAN_MTU | IFCAP_VLAN_HWTAGGING);
+       if_sethwassist(ifp, 0);
+       if_setsendqlen(ifp, ifqmaxlen);
+       if_setsendqready(ifp);
        if (pci_find_cap(dev, PCIY_PMG, &pmc) == 0) {
-               ifp->if_capabilities |= IFCAP_WOL_MAGIC;
+               if_setcapabilitiesbit(ifp, IFCAP_WOL_MAGIC, 0);
                sc->flags |= AE_FLAG_PMG;
        }
-       ifp->if_capenable = ifp->if_capabilities;
+       if_setcapenable(ifp, if_getcapabilities(ifp));
 
        /*
         * Configure and attach MII bus.
@@ -365,7 +364,7 @@ ae_attach(device_t dev)
 
        ether_ifattach(ifp, sc->eaddr);
        /* Tell the upper layer(s) we support long frames. */
-       ifp->if_hdrlen = sizeof(struct ether_vlan_header);
+       if_setifheaderlen(ifp, sizeof(struct ether_vlan_header));
 
        /*
         * Create and run all helper tasks.
@@ -557,7 +556,7 @@ ae_phy_init(ae_softc_t *sc)
 static int
 ae_init_locked(ae_softc_t *sc)
 {
-       struct ifnet *ifp;
+       if_t ifp;
        struct mii_data *mii;
        uint8_t eaddr[ETHER_ADDR_LEN];
        uint32_t val;
@@ -566,7 +565,7 @@ ae_init_locked(ae_softc_t *sc)
        AE_LOCK_ASSERT(sc);
 
        ifp = sc->ifp;
-       if ((ifp->if_drv_flags & IFF_DRV_RUNNING) != 0)
+       if ((if_getdrvflags(ifp) & IFF_DRV_RUNNING) != 0)
                return (0);
        mii = device_get_softc(sc->miibus);
 
@@ -584,7 +583,7 @@ ae_init_locked(ae_softc_t *sc)
        /*
         * Set the MAC address.
         */
-       bcopy(IF_LLADDR(ifp), eaddr, ETHER_ADDR_LEN);
+       bcopy(if_getlladdr(ifp), eaddr, ETHER_ADDR_LEN);
        val = eaddr[2] << 24 | eaddr[3] << 16 | eaddr[4] << 8 | eaddr[5];
        AE_WRITE_4(sc, AE_EADDR0_REG, val);
        val = eaddr[0] << 8 | eaddr[1];
@@ -653,7 +652,7 @@ ae_init_locked(ae_softc_t *sc)
        /*
         * Configure MTU.
         */
-       val = ifp->if_mtu + ETHER_HDR_LEN + ETHER_VLAN_ENCAP_LEN +
+       val = if_getmtu(ifp) + ETHER_HDR_LEN + ETHER_VLAN_ENCAP_LEN +
            ETHER_CRC_LEN;
        AE_WRITE_2(sc, AE_MTU_REG, val);
 
@@ -743,8 +742,8 @@ ae_init_locked(ae_softc_t *sc)
 
        callout_reset(&sc->tick_ch, hz, ae_tick, sc);
 
-       ifp->if_drv_flags |= IFF_DRV_RUNNING;
-       ifp->if_drv_flags &= ~IFF_DRV_OACTIVE;
+       if_setdrvflagbits(ifp, IFF_DRV_RUNNING, 0);
+       if_setdrvflagbits(ifp, 0, IFF_DRV_OACTIVE);
 
 #ifdef AE_DEBUG
        device_printf(sc->dev, "Initialization complete.\n");
@@ -757,7 +756,7 @@ static int
 ae_detach(device_t dev)
 {
        struct ae_softc *sc;
-       struct ifnet *ifp;
+       if_t ifp;
 
        sc = device_get_softc(dev);
        KASSERT(sc != NULL, ("[ae: %d]: sc is NULL", __LINE__));
@@ -882,12 +881,12 @@ ae_miibus_statchg(device_t dev)
 }
 
 static void
-ae_mediastatus(struct ifnet *ifp, struct ifmediareq *ifmr)
+ae_mediastatus(if_t ifp, struct ifmediareq *ifmr)
 {
        ae_softc_t *sc;
        struct mii_data *mii;
 
-       sc = ifp->if_softc;
+       sc = if_getsoftc(ifp);
        KASSERT(sc != NULL, ("[ae, %d]: sc is NULL", __LINE__));
 
        AE_LOCK(sc);
@@ -899,7 +898,7 @@ ae_mediastatus(struct ifnet *ifp, struct ifmediareq *ifmr)
 }
 
 static int
-ae_mediachange(struct ifnet *ifp)
+ae_mediachange(if_t ifp)
 {
        ae_softc_t *sc;
        struct mii_data *mii;
@@ -907,7 +906,7 @@ ae_mediachange(struct ifnet *ifp)
        int error;
 
        /* XXX: check IFF_UP ?? */
-       sc = ifp->if_softc;
+       sc = if_getsoftc(ifp);
        KASSERT(sc != NULL, ("[ae, %d]: sc is NULL", __LINE__));
        AE_LOCK(sc);
        mii = device_get_softc(sc->miibus);
@@ -1320,7 +1319,7 @@ ae_powersave_enable(ae_softc_t *sc)
 static void
 ae_pm_init(ae_softc_t *sc)
 {
-       struct ifnet *ifp;
+       if_t ifp;
        uint32_t val;
        uint16_t pmstat;
        struct mii_data *mii;
@@ -1338,7 +1337,7 @@ ae_pm_init(ae_softc_t *sc)
        /*
         * Configure WOL if enabled.
         */
-       if ((ifp->if_capenable & IFCAP_WOL) != 0) {
+       if ((if_getcapenable(ifp) & IFCAP_WOL) != 0) {
                mii = device_get_softc(sc->miibus);
                mii_pollstat(mii);
                if ((mii->mii_media_status & IFM_AVALID) != 0 &&
@@ -1386,7 +1385,7 @@ ae_pm_init(ae_softc_t *sc)
        if (pci_find_cap(sc->dev, PCIY_PMG, &pmc) == 0) {
                pmstat = pci_read_config(sc->dev, pmc + PCIR_POWER_STATUS, 2);
                pmstat &= ~(PCIM_PSTAT_PME | PCIM_PSTAT_PMEENABLE);
-               if ((ifp->if_capenable & IFCAP_WOL) != 0)
+               if ((if_getcapenable(ifp) & IFCAP_WOL) != 0)
                        pmstat |= PCIM_PSTAT_PME | PCIM_PSTAT_PMEENABLE;
                pci_write_config(sc->dev, pmc + PCIR_POWER_STATUS, pmstat, 2);
        }
@@ -1417,7 +1416,7 @@ ae_resume(device_t dev)
 
        AE_LOCK(sc);
        AE_READ_4(sc, AE_WOL_REG);      /* Clear WOL status. */
-       if ((sc->ifp->if_flags & IFF_UP) != 0)
+       if ((if_getflags(sc->ifp) & IFF_UP) != 0)
                ae_init_locked(sc);
        AE_UNLOCK(sc);
 
@@ -1512,25 +1511,25 @@ ae_encap(ae_softc_t *sc, struct mbuf **m_head)
 }
 
 static void
-ae_start(struct ifnet *ifp)
+ae_start(if_t ifp)
 {
        ae_softc_t *sc;
 
-       sc = ifp->if_softc;
+       sc = if_getsoftc(ifp);
        AE_LOCK(sc);
        ae_start_locked(ifp);
        AE_UNLOCK(sc);
 }
 
 static void
-ae_start_locked(struct ifnet *ifp)
+ae_start_locked(if_t ifp)
 {
        ae_softc_t *sc;
        unsigned int count;
        struct mbuf *m0;
        int error;
 
-       sc = ifp->if_softc;
+       sc = if_getsoftc(ifp);
        KASSERT(sc != NULL, ("[ae, %d]: sc is NULL", __LINE__));
        AE_LOCK_ASSERT(sc);
 
@@ -1538,21 +1537,21 @@ ae_start_locked(struct ifnet *ifp)
        if_printf(ifp, "Start called.\n");
 #endif
 
-       if ((ifp->if_drv_flags & (IFF_DRV_RUNNING | IFF_DRV_OACTIVE)) !=
+       if ((if_getdrvflags(ifp) & (IFF_DRV_RUNNING | IFF_DRV_OACTIVE)) !=
            IFF_DRV_RUNNING || (sc->flags & AE_FLAG_LINK) == 0)
                return;
 
        count = 0;
-       while (!IFQ_DRV_IS_EMPTY(&ifp->if_snd)) {
-               IFQ_DRV_DEQUEUE(&ifp->if_snd, m0);
+       while (!if_sendq_empty(ifp)) {
+               m0 = if_dequeue(ifp);
                if (m0 == NULL)
                        break;  /* Nothing to do. */
 
                error = ae_encap(sc, &m0);
                if (error != 0) {
                        if (m0 != NULL) {
-                               IFQ_DRV_PREPEND(&ifp->if_snd, m0);
-                               ifp->if_drv_flags |= IFF_DRV_OACTIVE;
+                               if_sendq_prepend(ifp, m0);
+                               if_setdrvflagbits(ifp, IFF_DRV_OACTIVE, 0);
 #ifdef AE_DEBUG
                                if_printf(ifp, "Setting OACTIVE.\n");
 #endif
@@ -1583,7 +1582,7 @@ ae_link_task(void *arg, int pending)
 {
        ae_softc_t *sc;
        struct mii_data *mii;
-       struct ifnet *ifp;
+       if_t ifp;
        uint32_t val;
 
        sc = (ae_softc_t *)arg;
@@ -1593,7 +1592,7 @@ ae_link_task(void *arg, int pending)
        ifp = sc->ifp;
        mii = device_get_softc(sc->miibus);
        if (mii == NULL || ifp == NULL ||
-           (ifp->if_drv_flags & IFF_DRV_RUNNING) == 0) {
+           (if_getdrvflags(ifp) & IFF_DRV_RUNNING) == 0) {
                AE_UNLOCK(sc);  /* XXX: could happen? */
                return;
        }
@@ -1753,7 +1752,7 @@ static void
 ae_int_task(void *arg, int pending)
 {
        ae_softc_t *sc;
-       struct ifnet *ifp;
+       if_t ifp;
        uint32_t val;
 
        sc = (ae_softc_t *)arg;
@@ -1777,10 +1776,10 @@ ae_int_task(void *arg, int pending)
        if_printf(ifp, "Interrupt received: 0x%08x\n", val);
 #endif
 
-       if ((ifp->if_drv_flags & IFF_DRV_RUNNING) != 0) {
+       if ((if_getdrvflags(ifp) & IFF_DRV_RUNNING) != 0) {
                if ((val & (AE_ISR_DMAR_TIMEOUT | AE_ISR_DMAW_TIMEOUT |
                    AE_ISR_PHY_LINKDOWN)) != 0) {
-                       ifp->if_drv_flags &= ~IFF_DRV_RUNNING;
+                       if_setdrvflagbits(ifp, 0, IFF_DRV_RUNNING);
                        ae_init_locked(sc);
                        AE_UNLOCK(sc);
                        return;
@@ -1795,7 +1794,7 @@ ae_int_task(void *arg, int pending)
                AE_WRITE_4(sc, AE_ISR_REG, 0);
 
                if ((sc->flags & AE_FLAG_TXAVAIL) != 0) {
-                       if (!IFQ_DRV_IS_EMPTY(&ifp->if_snd))
+                       if (!if_sendq_empty(ifp))
                                ae_start_locked(ifp);
                }
        }
@@ -1806,7 +1805,7 @@ ae_int_task(void *arg, int pending)
 static void
 ae_tx_intr(ae_softc_t *sc)
 {
-       struct ifnet *ifp;
+       if_t ifp;
        ae_txd_t *txd;
        ae_txs_t *txs;
        uint16_t flags;
@@ -1862,7 +1861,7 @@ ae_tx_intr(ae_softc_t *sc)
        }
 
        if ((sc->flags & AE_FLAG_TXAVAIL) != 0)
-               ifp->if_drv_flags &= ~IFF_DRV_OACTIVE;
+               if_setdrvflagbits(ifp, 0, IFF_DRV_OACTIVE);
        if (sc->tx_inproc < 0) {
                if_printf(ifp, "Received stray Tx interrupt(s).\n");
                sc->tx_inproc = 0;
@@ -1883,7 +1882,7 @@ ae_tx_intr(ae_softc_t *sc)
 static void
 ae_rxeof(ae_softc_t *sc, ae_rxd_t *rxd)
 {
-       struct ifnet *ifp;
+       if_t ifp;
        struct mbuf *m;
        unsigned int size;
        uint16_t flags;
@@ -1909,7 +1908,7 @@ ae_rxeof(ae_softc_t *sc, ae_rxd_t *rxd)
                return;
        }
 
-       if ((ifp->if_capenable & IFCAP_VLAN_HWTAGGING) != 0 &&
+       if ((if_getcapenable(ifp) & IFCAP_VLAN_HWTAGGING) != 0 &&
            (flags & AE_RXD_HAS_VLAN) != 0) {
                m->m_pkthdr.ether_vtag = AE_RXD_VLAN(le16toh(rxd->vlan));
                m->m_flags |= M_VLANTAG;
@@ -1920,7 +1919,7 @@ ae_rxeof(ae_softc_t *sc, ae_rxd_t *rxd)
         * Pass it through.
         */
        AE_UNLOCK(sc);
-       (*ifp->if_input)(ifp, m);
+       if_input(ifp, m);
        AE_LOCK(sc);
 }
 
@@ -1928,7 +1927,7 @@ static void
 ae_rx_intr(ae_softc_t *sc)
 {
        ae_rxd_t *rxd;
-       struct ifnet *ifp;
+       if_t ifp;
        uint16_t flags;
        int count;
 
@@ -1977,7 +1976,7 @@ ae_rx_intr(ae_softc_t *sc)
 static void
 ae_watchdog(ae_softc_t *sc)
 {
-       struct ifnet *ifp;
+       if_t ifp;
 
        KASSERT(sc != NULL, ("[ae, %d]: sc is NULL!", __LINE__));
        AE_LOCK_ASSERT(sc);
@@ -1992,9 +1991,9 @@ ae_watchdog(ae_softc_t *sc)
                if_printf(ifp, "watchdog timeout - resetting.\n");
 
        if_inc_counter(ifp, IFCOUNTER_OERRORS, 1);
-       ifp->if_drv_flags &= ~IFF_DRV_RUNNING;
+       if_setdrvflagbits(ifp, 0, IFF_DRV_RUNNING);
        ae_init_locked(sc);
-       if (!IFQ_DRV_IS_EMPTY(&ifp->if_snd))
+       if (!if_sendq_empty(ifp))
                ae_start_locked(ifp);
 }
 
@@ -2017,14 +2016,14 @@ ae_tick(void *arg)
 static void
 ae_rxvlan(ae_softc_t *sc)
 {
-       struct ifnet *ifp;
+       if_t ifp;
        uint32_t val;
 
        AE_LOCK_ASSERT(sc);
        ifp = sc->ifp;
        val = AE_READ_4(sc, AE_MAC_REG);
        val &= ~AE_MAC_RMVLAN_EN;
-       if ((ifp->if_capenable & IFCAP_VLAN_HWTAGGING) != 0)
+       if ((if_getcapenable(ifp) & IFCAP_VLAN_HWTAGGING) != 0)
                val |= AE_MAC_RMVLAN_EN;
        AE_WRITE_4(sc, AE_MAC_REG, val);
 }
@@ -2043,7 +2042,7 @@ ae_hash_maddr(void *arg, struct sockaddr_dl *sdl, u_int 
cnt)
 static void
 ae_rxfilter(ae_softc_t *sc)
 {
-       struct ifnet *ifp;
+       if_t ifp;
        uint32_t mchash[2];
        uint32_t rxcfg;
 
@@ -2056,11 +2055,11 @@ ae_rxfilter(ae_softc_t *sc)
        rxcfg = AE_READ_4(sc, AE_MAC_REG);
        rxcfg &= ~(AE_MAC_MCAST_EN | AE_MAC_BCAST_EN | AE_MAC_PROMISC_EN);
 
-       if ((ifp->if_flags & IFF_BROADCAST) != 0)
+       if ((if_getflags(ifp) & IFF_BROADCAST) != 0)
                rxcfg |= AE_MAC_BCAST_EN;
-       if ((ifp->if_flags & IFF_PROMISC) != 0)
+       if ((if_getflags(ifp) & IFF_PROMISC) != 0)
                rxcfg |= AE_MAC_PROMISC_EN;
-       if ((ifp->if_flags & IFF_ALLMULTI) != 0)
+       if ((if_getflags(ifp) & IFF_ALLMULTI) != 0)
                rxcfg |= AE_MAC_MCAST_EN;
 
        /*
@@ -2068,7 +2067,7 @@ ae_rxfilter(ae_softc_t *sc)
         */
        AE_WRITE_4(sc, AE_REG_MHT0, 0);
        AE_WRITE_4(sc, AE_REG_MHT1, 0);
-       if ((ifp->if_flags & (IFF_PROMISC | IFF_ALLMULTI)) != 0) {
+       if ((if_getflags(ifp) & (IFF_PROMISC | IFF_ALLMULTI)) != 0) {
                AE_WRITE_4(sc, AE_REG_MHT0, 0xffffffff);
                AE_WRITE_4(sc, AE_REG_MHT1, 0xffffffff);
                AE_WRITE_4(sc, AE_MAC_REG, rxcfg);
@@ -2086,14 +2085,14 @@ ae_rxfilter(ae_softc_t *sc)
 }
 
 static int
-ae_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data)
+ae_ioctl(if_t ifp, u_long cmd, caddr_t data)
 {
        struct ae_softc *sc;
        struct ifreq *ifr;
        struct mii_data *mii;
        int error, mask;
 
-       sc = ifp->if_softc;
+       sc = if_getsoftc(ifp);
        ifr = (struct ifreq *)data;
        error = 0;
 
@@ -2101,11 +2100,11 @@ ae_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data)
        case SIOCSIFMTU:
                if (ifr->ifr_mtu < ETHERMIN || ifr->ifr_mtu > ETHERMTU)
                        error = EINVAL;
-               else if (ifp->if_mtu != ifr->ifr_mtu) {
+               else if (if_getmtu(ifp) != ifr->ifr_mtu) {
                        AE_LOCK(sc);
-                       ifp->if_mtu = ifr->ifr_mtu;
-                       if ((ifp->if_drv_flags & IFF_DRV_RUNNING) != 0) {
-                               ifp->if_drv_flags &= ~IFF_DRV_RUNNING;
+                       if_setmtu(ifp, ifr->ifr_mtu);
+                       if ((if_getdrvflags(ifp) & IFF_DRV_RUNNING) != 0) {
+                               if_setdrvflagbits(ifp, 0, IFF_DRV_RUNNING);
                                ae_init_locked(sc);
                        }
                        AE_UNLOCK(sc);
@@ -2113,9 +2112,9 @@ ae_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data)
                break;
        case SIOCSIFFLAGS:
                AE_LOCK(sc);
-               if ((ifp->if_flags & IFF_UP) != 0) {
-                       if ((ifp->if_drv_flags & IFF_DRV_RUNNING) != 0) {
-                               if (((ifp->if_flags ^ sc->if_flags)
+               if ((if_getflags(ifp) & IFF_UP) != 0) {
+                       if ((if_getdrvflags(ifp) & IFF_DRV_RUNNING) != 0) {
+                               if (((if_getflags(ifp) ^ sc->if_flags)
                                    & (IFF_PROMISC | IFF_ALLMULTI)) != 0)
                                        ae_rxfilter(sc);
                        } else {
@@ -2123,16 +2122,16 @@ ae_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data)
                                        ae_init_locked(sc);
                        }
                } else {
-                       if ((ifp->if_drv_flags & IFF_DRV_RUNNING) != 0)
+                       if ((if_getdrvflags(ifp) & IFF_DRV_RUNNING) != 0)
                                ae_stop(sc);
                }
-               sc->if_flags = ifp->if_flags;
+               sc->if_flags = if_getflags(ifp);
                AE_UNLOCK(sc);
                break;
        case SIOCADDMULTI:
        case SIOCDELMULTI:
                AE_LOCK(sc);
-               if ((ifp->if_drv_flags & IFF_DRV_RUNNING) != 0)
+               if ((if_getdrvflags(ifp) & IFF_DRV_RUNNING) != 0)
                        ae_rxfilter(sc);
                AE_UNLOCK(sc);
                break;
@@ -2143,10 +2142,10 @@ ae_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data)
                break;
        case SIOCSIFCAP:
                AE_LOCK(sc);
-               mask = ifr->ifr_reqcap ^ ifp->if_capenable;
+               mask = ifr->ifr_reqcap ^ if_getcapenable(ifp);
                if ((mask & IFCAP_VLAN_HWTAGGING) != 0 &&
-                   (ifp->if_capabilities & IFCAP_VLAN_HWTAGGING) != 0) {
-                       ifp->if_capenable ^= IFCAP_VLAN_HWTAGGING;
+                   (if_getcapabilities(ifp) & IFCAP_VLAN_HWTAGGING) != 0) {
+                       if_togglecapenable(ifp, IFCAP_VLAN_HWTAGGING);
                        ae_rxvlan(sc);
                }
                VLAN_CAPABILITIES(ifp);
@@ -2162,13 +2161,13 @@ ae_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data)
 static void
 ae_stop(ae_softc_t *sc)
 {
-       struct ifnet *ifp;
+       if_t ifp;
        int i;
 
        AE_LOCK_ASSERT(sc);
 
        ifp = sc->ifp;
-       ifp->if_drv_flags &= ~(IFF_DRV_RUNNING | IFF_DRV_OACTIVE);
+       if_setdrvflagbits(ifp, 0, (IFF_DRV_RUNNING | IFF_DRV_OACTIVE));
        sc->flags &= ~AE_FLAG_LINK;
        sc->wd_timer = 0;       /* Cancel watchdog. */
        callout_stop(&sc->tick_ch);
diff --git a/sys/dev/ae/if_aevar.h b/sys/dev/ae/if_aevar.h
index b6010e30ec84..ec076ab38958 100644
--- a/sys/dev/ae/if_aevar.h
+++ b/sys/dev/ae/if_aevar.h
@@ -102,7 +102,7 @@ typedef struct ae_stats {
 
 /* Software state structure. */
 typedef struct ae_softc        {
-       struct ifnet            *ifp;
+       if_t                    ifp;
        device_t                dev;
        device_t                miibus;
        struct resource         *mem[1];

Reply via email to