diff options
author | Kevin Lo <kevlo@cvs.openbsd.org> | 2011-09-13 08:15:36 +0000 |
---|---|---|
committer | Kevin Lo <kevlo@cvs.openbsd.org> | 2011-09-13 08:15:36 +0000 |
commit | d08cf89f87ba4eb7f11ef8970bc0d1e239adec1d (patch) | |
tree | 440724cfe4df2cdac0541765fd64a179eb48d47c | |
parent | 3937e18c70e0094571184a9e9f79629c3b25624e (diff) |
Fix up ale_encap() / ale_start():
- Remove unnecessary nsegs variable from ale_encap() and
use map->dm_nsegs. Also remove unnecessary FreeBSD check
for 0 DMA segments check.
- Remove printfs in ale_encap() failure paths that shouldn't
be there.
- Add missing IF_PREPEND() from failure path coming off of
ale_encap() within ale_start().
- Fix error handling within ale_encap(). Previously ale_encap()
was attempting to unload a DMA map upon failure from
bus_dmamap_load_mbuf() even though one wasn't loaded at that
point and then always forcing mbufs through the EFBIG path.
Tested by Johan Torin.
From Brad
-rw-r--r-- | sys/dev/pci/if_ale.c | 57 |
1 files changed, 23 insertions, 34 deletions
diff --git a/sys/dev/pci/if_ale.c b/sys/dev/pci/if_ale.c index 92c9d936849..a45b91b9727 100644 --- a/sys/dev/pci/if_ale.c +++ b/sys/dev/pci/if_ale.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if_ale.c,v 1.19 2011/09/05 05:44:36 kevlo Exp $ */ +/* $OpenBSD: if_ale.c,v 1.20 2011/09/13 08:15:35 kevlo Exp $ */ /*- * Copyright (c) 2008, Pyun YongHyeon <yongari@FreeBSD.org> * All rights reserved. @@ -879,7 +879,7 @@ ale_encap(struct ale_softc *sc, struct mbuf **m_head) struct mbuf *m; bus_dmamap_t map; uint32_t cflags, poff, vtag; - int error, i, nsegs, prod; + int error, i, prod; m = *m_head; cflags = vtag = 0; @@ -891,46 +891,25 @@ ale_encap(struct ale_softc *sc, struct mbuf **m_head) map = txd->tx_dmamap; error = bus_dmamap_load_mbuf(sc->sc_dmat, map, *m_head, BUS_DMA_NOWAIT); - + if (error != 0 && error != EFBIG) + goto drop; if (error != 0) { - bus_dmamap_unload(sc->sc_dmat, map); - error = EFBIG; - } - if (error == EFBIG) { if (m_defrag(*m_head, M_DONTWAIT)) { - printf("%s: can't defrag TX mbuf\n", - sc->sc_dev.dv_xname); - m_freem(*m_head); - *m_head = NULL; - return (ENOBUFS); + error = ENOBUFS; + goto drop; } error = bus_dmamap_load_mbuf(sc->sc_dmat, map, *m_head, BUS_DMA_NOWAIT); - if (error != 0) { - printf("%s: could not load defragged TX mbuf\n", - sc->sc_dev.dv_xname); - m_freem(*m_head); - *m_head = NULL; - return (error); - } - } else if (error) { - printf("%s: could not load TX mbuf\n", sc->sc_dev.dv_xname); - return (error); - } - - nsegs = map->dm_nsegs; - - if (nsegs == 0) { - m_freem(*m_head); - *m_head = NULL; - return (EIO); + if (error != 0) + goto drop; } /* Check descriptor overrun. */ - if (sc->ale_cdata.ale_tx_cnt + nsegs >= ALE_TX_RING_CNT - 2) { + if (sc->ale_cdata.ale_tx_cnt + map->dm_nsegs >= ALE_TX_RING_CNT - 2) { bus_dmamap_unload(sc->sc_dmat, map); return (ENOBUFS); } + bus_dmamap_sync(sc->sc_dmat, map, 0, map->dm_mapsize, BUS_DMASYNC_PREWRITE); @@ -974,7 +953,7 @@ ale_encap(struct ale_softc *sc, struct mbuf **m_head) #endif desc = NULL; - for (i = 0; i < nsegs; i++) { + for (i = 0; i < map->dm_nsegs; i++) { desc = &sc->ale_cdata.ale_tx_ring[prod]; desc->addr = htole64(map->dm_segs[i].ds_addr); desc->len = @@ -983,6 +962,7 @@ ale_encap(struct ale_softc *sc, struct mbuf **m_head) sc->ale_cdata.ale_tx_cnt++; ALE_DESC_INC(prod, ALE_TX_RING_CNT); } + /* Update producer index. */ sc->ale_cdata.ale_tx_prod = prod; @@ -1003,6 +983,11 @@ ale_encap(struct ale_softc *sc, struct mbuf **m_head) sc->ale_cdata.ale_tx_ring_map->dm_mapsize, BUS_DMASYNC_PREWRITE); return (0); + + drop: + m_freem(*m_head); + *m_head = NULL; + return (error); } void @@ -1035,11 +1020,15 @@ ale_start(struct ifnet *ifp) * for the NIC to drain the ring. */ if (ale_encap(sc, &m_head)) { - if (m_head == NULL) - break; + if (m_head == NULL) { + ifp->if_oerrors++; + break; + } + IF_PREPEND(&ifp->if_snd, m_head); ifp->if_flags |= IFF_OACTIVE; break; } + enq = 1; #if NBPFILTER > 0 |