 @@ -1,1232 +1,1232 @@
-/* $NetBSD: ix_txrx.c,v 1.24.2.13 2018/09/07 12:37:20 martin Exp $ */
+/* $NetBSD: ix_txrx.c,v 1.24.2.14 2018/12/20 11:34:33 martin Exp $ */
 /******************************************************************************
   Copyright (c) 2001-2017, Intel Corporation
   All rights reserved.
   Redistribution and use in source and binary forms, with or without
   modification, are permitted provided that the following conditions are met:
 . Redistributions of source code must retain the above copyright notice,
       this list of conditions and the following disclaimer.
 . Redistributions in binary form must reproduce the above copyright
       notice, this list of conditions and the following disclaimer in the
       documentation and/or other materials provided with the distribution.
 . Neither the name of the Intel Corporation nor the names of its
       contributors may be used to endorse or promote products derived from
       this software without specific prior written permission.
   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
   AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
   IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
   ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
   LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
   CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
   SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
   INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
   CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
   ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
   POSSIBILITY OF SUCH DAMAGE.
 ******************************************************************************/
 /*$FreeBSD: head/sys/dev/ixgbe/ix_txrx.c 327031 2017-12-20 18:15:06Z erj $*/
 /*
  * Copyright (c) 2011 The NetBSD Foundation, Inc.
  * All rights reserved.
+ *
  * This code is derived from software contributed to The NetBSD Foundation
  * by Coyote Point Systems, Inc.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. AND CONTRIBUTORS
  * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
  * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
  * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
  * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
  * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
  * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
  * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
  * POSSIBILITY OF SUCH DAMAGE.
  */
 #include "opt_inet.h"
 #include "opt_inet6.h"
 #include "ixgbe.h"
 /*
  * HW RSC control:
  *  this feature only works with
  *  IPv4, and only on 82599 and later.
  *  Also this will cause IP forwarding to
  *  fail and that can't be controlled by
  *  the stack as LRO can. For all these
  *  reasons I've deemed it best to leave
  *  this off and not bother with a tuneable
  *  interface, this would need to be compiled
  *  to enable.
  */
 static bool ixgbe_rsc_enable = FALSE;
 /*
  * For Flow Director: this is the
  * number of TX packets we sample
  * for the filter pool, this means
  * every 20th packet will be probed.
+ *
  * This feature can be disabled by
  * setting this to 0.
  */
 static int atr_sample_rate = 20;
 /************************************************************************
  *  Local Function prototypes
  ************************************************************************/
 static void          ixgbe_setup_transmit_ring(struct tx_ring *);
 static void          ixgbe_free_transmit_buffers(struct tx_ring *);
 static int           ixgbe_setup_receive_ring(struct rx_ring *);
 static void          ixgbe_free_receive_buffers(struct rx_ring *);
 static void          ixgbe_rx_checksum(u32, struct mbuf *, u32,
                                        struct ixgbe_hw_stats *);
 static void          ixgbe_refresh_mbufs(struct rx_ring *, int);
 static void          ixgbe_drain(struct ifnet *, struct tx_ring *);
 static int           ixgbe_xmit(struct tx_ring *, struct mbuf *);
 static int           ixgbe_tx_ctx_setup(struct tx_ring *,
                                         struct mbuf *, u32 *, u32 *);
 static int           ixgbe_tso_setup(struct tx_ring *,
                                      struct mbuf *, u32 *, u32 *);
 static __inline void ixgbe_rx_discard(struct rx_ring *, int);
 static __inline void ixgbe_rx_input(struct rx_ring *, struct ifnet *,
                                     struct mbuf *, u32);
 static int           ixgbe_dma_malloc(struct adapter *, bus_size_t,
                                       struct ixgbe_dma_alloc *, int);
 static void          ixgbe_dma_free(struct adapter *, struct ixgbe_dma_alloc *);
 static void	ixgbe_setup_hw_rsc(struct rx_ring *);
 /************************************************************************
  * ixgbe_legacy_start_locked - Transmit entry point
+ *
  *   Called by the stack to initiate a transmit.
  *   The driver will remain in this routine as long as there are
  *   packets to transmit and transmit resources are available.
  *   In case resources are not available, the stack is notified
  *   and the packet is requeued.
  ************************************************************************/
 int
 ixgbe_legacy_start_locked(struct ifnet *ifp, struct tx_ring *txr)
+{
 	int rc;
 	struct mbuf    *m_head;
 	struct adapter *adapter = txr->adapter;
 	IXGBE_TX_LOCK_ASSERT(txr);
 	if (!adapter->link_active) {
 		/*
 		 * discard all packets buffered in IFQ to avoid
 		 * sending old packets at next link up timing.
 		 */
 		ixgbe_drain(ifp, txr);
 		return (ENETDOWN);
+	}
 	if ((ifp->if_flags & IFF_RUNNING) == 0)
 		return (ENETDOWN);
 	if (txr->txr_no_space)
 		return (ENETDOWN);
 	while (!IFQ_IS_EMPTY(&ifp->if_snd)) {
 		if (txr->tx_avail <= IXGBE_QUEUE_MIN_FREE)
 			break;
 		IFQ_POLL(&ifp->if_snd, m_head);
 		if (m_head == NULL)
 			break;
 		if ((rc = ixgbe_xmit(txr, m_head)) == EAGAIN) {
 			break;
+		}
 		IFQ_DEQUEUE(&ifp->if_snd, m_head);
 		if (rc != 0) {
 			m_freem(m_head);
 			continue;
+		}
 		/* Send a copy of the frame to the BPF listener */
 		bpf_mtap(ifp, m_head);
+	}
 	return IXGBE_SUCCESS;
 } /* ixgbe_legacy_start_locked */
 /************************************************************************
  * ixgbe_legacy_start
+ *
  *   Called by the stack, this always uses the first tx ring,
  *   and should not be used with multiqueue tx enabled.
  ************************************************************************/
 void
 ixgbe_legacy_start(struct ifnet *ifp)
+{
 	struct adapter *adapter = ifp->if_softc;
 	struct tx_ring *txr = adapter->tx_rings;
 	if (ifp->if_flags & IFF_RUNNING) {
 		IXGBE_TX_LOCK(txr);
 		ixgbe_legacy_start_locked(ifp, txr);
 		IXGBE_TX_UNLOCK(txr);
+	}
 } /* ixgbe_legacy_start */
 /************************************************************************
  * ixgbe_mq_start - Multiqueue Transmit Entry Point
+ *
  *   (if_transmit function)
  ************************************************************************/
 int
 ixgbe_mq_start(struct ifnet *ifp, struct mbuf *m)
+{
 	struct adapter	*adapter = ifp->if_softc;
 	struct tx_ring	*txr;
 	int 		i;
 #ifdef RSS
 	uint32_t bucket_id;
 #endif
 	/*
 	 * When doing RSS, map it to the same outbound queue
 	 * as the incoming flow would be mapped to.
+	 *
 	 * If everything is setup correctly, it should be the
 	 * same bucket that the current CPU we're on is.
 	 */
 #ifdef RSS
 	if (M_HASHTYPE_GET(m) != M_HASHTYPE_NONE) {
 		if ((adapter->feat_en & IXGBE_FEATURE_RSS) &&
 		    (rss_hash2bucket(m->m_pkthdr.flowid, M_HASHTYPE_GET(m),
 		    &bucket_id) == 0)) {
 			i = bucket_id % adapter->num_queues;
 #ifdef IXGBE_DEBUG
 			if (bucket_id > adapter->num_queues)
 				if_printf(ifp,
 				    "bucket_id (%d) > num_queues (%d)\n",
 				    bucket_id, adapter->num_queues);
 #endif
 		} else
 			i = m->m_pkthdr.flowid % adapter->num_queues;
 	} else
 #endif /* 0 */
-		i = cpu_index(curcpu()) % adapter->num_queues;
+		i = (cpu_index(curcpu()) % ncpu) % adapter->num_queues;
 	/* Check for a hung queue and pick alternative */
 	if (((1 << i) & adapter->active_queues) == 0)
 		i = ffs64(adapter->active_queues);
 	txr = &adapter->tx_rings[i];
 	if (__predict_false(!pcq_put(txr->txr_interq, m))) {
 		m_freem(m);
 		txr->pcq_drops.ev_count++;
 		return ENOBUFS;
+	}
 	if (IXGBE_TX_TRYLOCK(txr)) {
 		ixgbe_mq_start_locked(ifp, txr);
 		IXGBE_TX_UNLOCK(txr);
 	} else {
 		if (adapter->txrx_use_workqueue) {
 			u_int *enqueued;
 			/*
 			 * This function itself is not called in interrupt
 			 * context, however it can be called in fast softint
 			 * context right after receiving forwarding packets.
 			 * So, it is required to protect workqueue from twice
 			 * enqueuing when the machine uses both spontaneous
 			 * packets and forwarding packets.
 			 */
 			enqueued = percpu_getref(adapter->txr_wq_enqueued);
 			if (*enqueued == 0) {
 				*enqueued = 1;
 				percpu_putref(adapter->txr_wq_enqueued);
 				workqueue_enqueue(adapter->txr_wq,
 				    &txr->wq_cookie, curcpu());
 			} else
 				percpu_putref(adapter->txr_wq_enqueued);
 		} else
 			softint_schedule(txr->txr_si);
+	}
 	return (0);
 } /* ixgbe_mq_start */
 /************************************************************************
  * ixgbe_mq_start_locked
  ************************************************************************/
 int
 ixgbe_mq_start_locked(struct ifnet *ifp, struct tx_ring *txr)
+{
 	struct mbuf    *next;
 	int            enqueued = 0, err = 0;
 	if (!txr->adapter->link_active) {
 		/*
 		 * discard all packets buffered in txr_interq to avoid
 		 * sending old packets at next link up timing.
 		 */
 		ixgbe_drain(ifp, txr);
 		return (ENETDOWN);
+	}
 	if ((ifp->if_flags & IFF_RUNNING) == 0)
 		return (ENETDOWN);
 	if (txr->txr_no_space)
 		return (ENETDOWN);
 	/* Process the queue */
 	while ((next = pcq_get(txr->txr_interq)) != NULL) {
 		if ((err = ixgbe_xmit(txr, next)) != 0) {
 			m_freem(next);
 			/* All errors are counted in ixgbe_xmit() */
 			break;
+		}
 		enqueued++;
 #if __FreeBSD_version >= 1100036
 		/*
 		 * Since we're looking at the tx ring, we can check
 		 * to see if we're a VF by examing our tail register
 		 * address.
 		 */
 		if ((txr->adapter->feat_en & IXGBE_FEATURE_VF) &&
 		    (next->m_flags & M_MCAST))
 			if_inc_counter(ifp, IFCOUNTER_OMCASTS, 1);
 #endif
 		/* Send a copy of the frame to the BPF listener */
 		bpf_mtap(ifp, next);
 		if ((ifp->if_flags & IFF_RUNNING) == 0)
 			break;
+	}
 	if (txr->tx_avail < IXGBE_TX_CLEANUP_THRESHOLD(txr->adapter))
 		ixgbe_txeof(txr);
 	return (err);
 } /* ixgbe_mq_start_locked */
 /************************************************************************
  * ixgbe_deferred_mq_start
+ *
  *   Called from a softint and workqueue (indirectly) to drain queued
  *   transmit packets.
  ************************************************************************/
 void
 ixgbe_deferred_mq_start(void *arg)
+{
 	struct tx_ring *txr = arg;
 	struct adapter *adapter = txr->adapter;
 	struct ifnet   *ifp = adapter->ifp;
 	IXGBE_TX_LOCK(txr);
 	if (pcq_peek(txr->txr_interq) != NULL)
 		ixgbe_mq_start_locked(ifp, txr);
 	IXGBE_TX_UNLOCK(txr);
 } /* ixgbe_deferred_mq_start */
 /************************************************************************
  * ixgbe_deferred_mq_start_work
+ *
  *   Called from a workqueue to drain queued transmit packets.
  ************************************************************************/
 void
 ixgbe_deferred_mq_start_work(struct work *wk, void *arg)
+{
 	struct tx_ring *txr = container_of(wk, struct tx_ring, wq_cookie);
 	struct adapter *adapter = txr->adapter;
 	u_int *enqueued = percpu_getref(adapter->txr_wq_enqueued);
 	*enqueued = 0;
 	percpu_putref(adapter->txr_wq_enqueued);
 	ixgbe_deferred_mq_start(txr);
 } /* ixgbe_deferred_mq_start */
 /************************************************************************
  * ixgbe_drain_all
  ************************************************************************/
 void
 ixgbe_drain_all(struct adapter *adapter)
+{
 	struct ifnet *ifp = adapter->ifp;
 	struct ix_queue *que = adapter->queues;
 	for (int i = 0; i < adapter->num_queues; i++, que++) {
 		struct tx_ring  *txr = que->txr;
 		IXGBE_TX_LOCK(txr);
 		ixgbe_drain(ifp, txr);
 		IXGBE_TX_UNLOCK(txr);
+	}
+}
 /************************************************************************
  * ixgbe_xmit
+ *
  *   Maps the mbufs to tx descriptors, allowing the
  *   TX engine to transmit the packets.
+ *
  *   Return 0 on success, positive on failure
  ************************************************************************/
 static int
 ixgbe_xmit(struct tx_ring *txr, struct mbuf *m_head)
+{
 	struct adapter          *adapter = txr->adapter;
 	struct ixgbe_tx_buf     *txbuf;
 	union ixgbe_adv_tx_desc *txd = NULL;
 	struct ifnet	        *ifp = adapter->ifp;
 	int                     i, j, error;
 	int                     first;
 	u32                     olinfo_status = 0, cmd_type_len;
 	bool                    remap = TRUE;
 	bus_dmamap_t            map;
 	/* Basic descriptor defines */
 	cmd_type_len = (IXGBE_ADVTXD_DTYP_DATA |
 	    IXGBE_ADVTXD_DCMD_IFCS | IXGBE_ADVTXD_DCMD_DEXT);
 	if (vlan_has_tag(m_head))
 		cmd_type_len |= IXGBE_ADVTXD_DCMD_VLE;
 	/*
 	 * Important to capture the first descriptor
 	 * used because it will contain the index of
 	 * the one we tell the hardware to report back
 	 */
 	first = txr->next_avail_desc;
 	txbuf = &txr->tx_buffers[first];
 	map = txbuf->map;
 	/*
 	 * Map the packet for DMA.
 	 */
 retry:
 	error = bus_dmamap_load_mbuf(txr->txtag->dt_dmat, map, m_head,
 	    BUS_DMA_NOWAIT);
 	if (__predict_false(error)) {
 		struct mbuf *m;
 		switch (error) {
 		case EAGAIN:
 			txr->q_eagain_tx_dma_setup++;
 			return EAGAIN;
 		case ENOMEM:
 			txr->q_enomem_tx_dma_setup++;
 			return EAGAIN;
 		case EFBIG:
 			/* Try it again? - one try */
 			if (remap == TRUE) {
 				remap = FALSE;
 				/*
 				 * XXX: m_defrag will choke on
 				 * non-MCLBYTES-sized clusters
 				 */
 				txr->q_efbig_tx_dma_setup++;
 				m = m_defrag(m_head, M_NOWAIT);
 				if (m == NULL) {
 					txr->q_mbuf_defrag_failed++;
 					return ENOBUFS;
+				}
 				m_head = m;
 				goto retry;
 			} else {
 				txr->q_efbig2_tx_dma_setup++;
 				return error;
+			}
 		case EINVAL:
 			txr->q_einval_tx_dma_setup++;
 			return error;
 		default:
 			txr->q_other_tx_dma_setup++;
 			return error;
+		}
+	}
 	/* Make certain there are enough descriptors */
 	if (txr->tx_avail < (map->dm_nsegs + 2)) {
 		txr->txr_no_space = true;
 		txr->no_desc_avail.ev_count++;
 		ixgbe_dmamap_unload(txr->txtag, txbuf->map);
 		return EAGAIN;
+	}
 	/*
 	 * Set up the appropriate offload context
 	 * this will consume the first descriptor
 	 */
 	error = ixgbe_tx_ctx_setup(txr, m_head, &cmd_type_len, &olinfo_status);
 	if (__predict_false(error)) {
 		return (error);
+	}
 	/* Do the flow director magic */
 	if ((adapter->feat_en & IXGBE_FEATURE_FDIR) &&
 	    (txr->atr_sample) && (!adapter->fdir_reinit)) {
 		++txr->atr_count;
 		if (txr->atr_count >= atr_sample_rate) {
 			ixgbe_atr(txr, m_head);
 			txr->atr_count = 0;
+		}
+	}
 	olinfo_status |= IXGBE_ADVTXD_CC;
 	i = txr->next_avail_desc;
 	for (j = 0; j < map->dm_nsegs; j++) {
 		bus_size_t seglen;
 		bus_addr_t segaddr;
 		txbuf = &txr->tx_buffers[i];
 		txd = &txr->tx_base[i];
 		seglen = map->dm_segs[j].ds_len;
 		segaddr = htole64(map->dm_segs[j].ds_addr);
 		txd->read.buffer_addr = segaddr;
 		txd->read.cmd_type_len = htole32(cmd_type_len | seglen);
 		txd->read.olinfo_status = htole32(olinfo_status);
 		if (++i == txr->num_desc)
 			i = 0;
+	}
 	txd->read.cmd_type_len |= htole32(IXGBE_TXD_CMD_EOP | IXGBE_TXD_CMD_RS);
 	txr->tx_avail -= map->dm_nsegs;
 	txr->next_avail_desc = i;
 	txbuf->m_head = m_head;
 	/*
 	 * Here we swap the map so the last descriptor,
 	 * which gets the completion interrupt has the
 	 * real map, and the first descriptor gets the
 	 * unused map from this descriptor.
 	 */
 	txr->tx_buffers[first].map = txbuf->map;
 	txbuf->map = map;
 	bus_dmamap_sync(txr->txtag->dt_dmat, map, 0, m_head->m_pkthdr.len,
 	    BUS_DMASYNC_PREWRITE);
 	/* Set the EOP descriptor that will be marked done */
 	txbuf = &txr->tx_buffers[first];
 	txbuf->eop = txd;
 	ixgbe_dmamap_sync(txr->txdma.dma_tag, txr->txdma.dma_map,
 	    BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
 	/*
 	 * Advance the Transmit Descriptor Tail (Tdt), this tells the
 	 * hardware that this frame is available to transmit.
 	 */
 	++txr->total_packets.ev_count;
 	IXGBE_WRITE_REG(&adapter->hw, txr->tail, i);
 	/*
 	 * XXXX NOMPSAFE: ifp->if_data should be percpu.
 	 */
 	ifp->if_obytes += m_head->m_pkthdr.len;
 	if (m_head->m_flags & M_MCAST)
 		ifp->if_omcasts++;
 	/* Mark queue as having work */
 	if (txr->busy == 0)
 		txr->busy = 1;
 	return (0);
 } /* ixgbe_xmit */
 /************************************************************************
  * ixgbe_drain
  ************************************************************************/
 static void
 ixgbe_drain(struct ifnet *ifp, struct tx_ring *txr)
+{
 	struct mbuf *m;
 	IXGBE_TX_LOCK_ASSERT(txr);
 	if (txr->me == 0) {
 		while (!IFQ_IS_EMPTY(&ifp->if_snd)) {
 			IFQ_DEQUEUE(&ifp->if_snd, m);
 			m_freem(m);
 			IF_DROP(&ifp->if_snd);
+		}
+	}
 	while ((m = pcq_get(txr->txr_interq)) != NULL) {
 		m_freem(m);
 		txr->pcq_drops.ev_count++;
+	}
+}
 /************************************************************************
  * ixgbe_allocate_transmit_buffers
+ *
  *   Allocate memory for tx_buffer structures. The tx_buffer stores all
  *   the information needed to transmit a packet on the wire. This is
  *   called only once at attach, setup is done every reset.
  ************************************************************************/
 static int
 ixgbe_allocate_transmit_buffers(struct tx_ring *txr)
+{
 	struct adapter      *adapter = txr->adapter;
 	device_t            dev = adapter->dev;
 	struct ixgbe_tx_buf *txbuf;
 	int                 error, i;
 	/*
 	 * Setup DMA descriptor areas.
 	 */
 	error = ixgbe_dma_tag_create(
 	         /*      parent */ adapter->osdep.dmat,
 	         /*   alignment */ 1,
 	         /*      bounds */ 0,
 	         /*     maxsize */ IXGBE_TSO_SIZE,
 	         /*   nsegments */ adapter->num_segs,
 	         /*  maxsegsize */ PAGE_SIZE,
 	         /*       flags */ 0,
 	                           &txr->txtag);
 	if (error != 0) {
 		aprint_error_dev(dev,"Unable to allocate TX DMA tag\n");
 		goto fail;
+	}
 	txr->tx_buffers =
 	    (struct ixgbe_tx_buf *) malloc(sizeof(struct ixgbe_tx_buf) *
 	    adapter->num_tx_desc, M_DEVBUF, M_NOWAIT | M_ZERO);
 	if (txr->tx_buffers == NULL) {
 		aprint_error_dev(dev, "Unable to allocate tx_buffer memory\n");
 		error = ENOMEM;
 		goto fail;
+	}
 	/* Create the descriptor buffer dma maps */
 	txbuf = txr->tx_buffers;
 	for (i = 0; i < adapter->num_tx_desc; i++, txbuf++) {
 		error = ixgbe_dmamap_create(txr->txtag, 0, &txbuf->map);
 		if (error != 0) {
 			aprint_error_dev(dev,
 			    "Unable to create TX DMA map (%d)\n", error);
 			goto fail;
+		}
+	}
 	return 0;
 fail:
 	/* We free all, it handles case where we are in the middle */
 #if 0 /* XXX was FreeBSD */
 	ixgbe_free_transmit_structures(adapter);
 #else
 	ixgbe_free_transmit_buffers(txr);
 #endif
 	return (error);
 } /* ixgbe_allocate_transmit_buffers */
 /************************************************************************
  * ixgbe_setup_transmit_ring - Initialize a transmit ring.
  ************************************************************************/
 static void
 ixgbe_setup_transmit_ring(struct tx_ring *txr)
+{
 	struct adapter        *adapter = txr->adapter;
 	struct ixgbe_tx_buf   *txbuf;
 #ifdef DEV_NETMAP
 	struct netmap_adapter *na = NA(adapter->ifp);
 	struct netmap_slot    *slot;
 #endif /* DEV_NETMAP */
 	/* Clear the old ring contents */
 	IXGBE_TX_LOCK(txr);
 #ifdef DEV_NETMAP
 	if (adapter->feat_en & IXGBE_FEATURE_NETMAP) {
 		/*
 		 * (under lock): if in netmap mode, do some consistency
 		 * checks and set slot to entry 0 of the netmap ring.
 		 */
 		slot = netmap_reset(na, NR_TX, txr->me, 0);
+	}
 #endif /* DEV_NETMAP */
 	bzero((void *)txr->tx_base,
 	    (sizeof(union ixgbe_adv_tx_desc)) * adapter->num_tx_desc);
 	/* Reset indices */
 	txr->next_avail_desc = 0;
 	txr->next_to_clean = 0;
 	/* Free any existing tx buffers. */
 	txbuf = txr->tx_buffers;
 	for (int i = 0; i < txr->num_desc; i++, txbuf++) {
 		if (txbuf->m_head != NULL) {
 			bus_dmamap_sync(txr->txtag->dt_dmat, txbuf->map,
 , txbuf->m_head->m_pkthdr.len,
 			    BUS_DMASYNC_POSTWRITE);
 			ixgbe_dmamap_unload(txr->txtag, txbuf->map);
 			m_freem(txbuf->m_head);
 			txbuf->m_head = NULL;
+		}
 #ifdef DEV_NETMAP
 		/*
 		 * In netmap mode, set the map for the packet buffer.
 		 * NOTE: Some drivers (not this one) also need to set
 		 * the physical buffer address in the NIC ring.
 		 * Slots in the netmap ring (indexed by "si") are
 		 * kring->nkr_hwofs positions "ahead" wrt the
 		 * corresponding slot in the NIC ring. In some drivers
 		 * (not here) nkr_hwofs can be negative. Function
 		 * netmap_idx_n2k() handles wraparounds properly.
 		 */
 		if ((adapter->feat_en & IXGBE_FEATURE_NETMAP) && slot) {
 			int si = netmap_idx_n2k(&na->tx_rings[txr->me], i);
 			netmap_load_map(na, txr->txtag,
 			    txbuf->map, NMB(na, slot + si));
+		}
 #endif /* DEV_NETMAP */
 		/* Clear the EOP descriptor pointer */
 		txbuf->eop = NULL;
+	}
 	/* Set the rate at which we sample packets */
 	if (adapter->feat_en & IXGBE_FEATURE_FDIR)
 		txr->atr_sample = atr_sample_rate;
 	/* Set number of descriptors available */
 	txr->tx_avail = adapter->num_tx_desc;
 	ixgbe_dmamap_sync(txr->txdma.dma_tag, txr->txdma.dma_map,
 	    BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
 	IXGBE_TX_UNLOCK(txr);
 } /* ixgbe_setup_transmit_ring */
 /************************************************************************
  * ixgbe_setup_transmit_structures - Initialize all transmit rings.
  ************************************************************************/
 int
 ixgbe_setup_transmit_structures(struct adapter *adapter)
+{
 	struct tx_ring *txr = adapter->tx_rings;
 	for (int i = 0; i < adapter->num_queues; i++, txr++)
 		ixgbe_setup_transmit_ring(txr);
 	return (0);
 } /* ixgbe_setup_transmit_structures */
 /************************************************************************
  * ixgbe_free_transmit_structures - Free all transmit rings.
  ************************************************************************/
 void
 ixgbe_free_transmit_structures(struct adapter *adapter)
+{
 	struct tx_ring *txr = adapter->tx_rings;
 	for (int i = 0; i < adapter->num_queues; i++, txr++) {
 		ixgbe_free_transmit_buffers(txr);
 		ixgbe_dma_free(adapter, &txr->txdma);
 		IXGBE_TX_LOCK_DESTROY(txr);
+	}
 	free(adapter->tx_rings, M_DEVBUF);
 } /* ixgbe_free_transmit_structures */
 /************************************************************************
  * ixgbe_free_transmit_buffers
+ *
  *   Free transmit ring related data structures.
  ************************************************************************/
 static void
 ixgbe_free_transmit_buffers(struct tx_ring *txr)
+{
 	struct adapter      *adapter = txr->adapter;
 	struct ixgbe_tx_buf *tx_buffer;
 	int                 i;
 	INIT_DEBUGOUT("ixgbe_free_transmit_buffers: begin");
 	if (txr->tx_buffers == NULL)
 		return;
 	tx_buffer = txr->tx_buffers;
 	for (i = 0; i < adapter->num_tx_desc; i++, tx_buffer++) {
 		if (tx_buffer->m_head != NULL) {
 			bus_dmamap_sync(txr->txtag->dt_dmat, tx_buffer->map,
 , tx_buffer->m_head->m_pkthdr.len,
 			    BUS_DMASYNC_POSTWRITE);
 			ixgbe_dmamap_unload(txr->txtag, tx_buffer->map);
 			m_freem(tx_buffer->m_head);
 			tx_buffer->m_head = NULL;
 			if (tx_buffer->map != NULL) {
 				ixgbe_dmamap_destroy(txr->txtag,
 				    tx_buffer->map);
 				tx_buffer->map = NULL;
+			}
 		} else if (tx_buffer->map != NULL) {
 			ixgbe_dmamap_unload(txr->txtag, tx_buffer->map);
 			ixgbe_dmamap_destroy(txr->txtag, tx_buffer->map);
 			tx_buffer->map = NULL;
+		}
+	}
 	if (txr->txr_interq != NULL) {
 		struct mbuf *m;
 		while ((m = pcq_get(txr->txr_interq)) != NULL)
 			m_freem(m);
 		pcq_destroy(txr->txr_interq);
+	}
 	if (txr->tx_buffers != NULL) {
 		free(txr->tx_buffers, M_DEVBUF);
 		txr->tx_buffers = NULL;
+	}
 	if (txr->txtag != NULL) {
 		ixgbe_dma_tag_destroy(txr->txtag);
 		txr->txtag = NULL;
+	}
 } /* ixgbe_free_transmit_buffers */
 /************************************************************************
  * ixgbe_tx_ctx_setup
+ *
  *   Advanced Context Descriptor setup for VLAN, CSUM or TSO
  ************************************************************************/
 static int
 ixgbe_tx_ctx_setup(struct tx_ring *txr, struct mbuf *mp,
     u32 *cmd_type_len, u32 *olinfo_status)
+{
 	struct adapter                   *adapter = txr->adapter;
 	struct ixgbe_adv_tx_context_desc *TXD;
 	struct ether_vlan_header         *eh;
 #ifdef INET
 	struct ip                        *ip;
 #endif
 #ifdef INET6
 	struct ip6_hdr                   *ip6;
 #endif
 	int                              ehdrlen, ip_hlen = 0;
 	int                              offload = TRUE;
 	int                              ctxd = txr->next_avail_desc;
 	u32                              vlan_macip_lens = 0;
 	u32                              type_tucmd_mlhl = 0;
 	u16                              vtag = 0;
 	u16                              etype;
 	u8                               ipproto = 0;
 	char                             *l3d;
 	/* First check if TSO is to be used */
 	if (mp->m_pkthdr.csum_flags & (M_CSUM_TSOv4 | M_CSUM_TSOv6)) {
 		int rv = ixgbe_tso_setup(txr, mp, cmd_type_len, olinfo_status);
 		if (rv != 0)
 			++adapter->tso_err.ev_count;
 		return rv;
+	}
 	if ((mp->m_pkthdr.csum_flags & M_CSUM_OFFLOAD) == 0)
 		offload = FALSE;
 	/* Indicate the whole packet as payload when not doing TSO */
 	*olinfo_status |= mp->m_pkthdr.len << IXGBE_ADVTXD_PAYLEN_SHIFT;
 	/* Now ready a context descriptor */
 	TXD = (struct ixgbe_adv_tx_context_desc *)&txr->tx_base[ctxd];
 	/*
 	 * In advanced descriptors the vlan tag must
 	 * be placed into the context descriptor. Hence
 	 * we need to make one even if not doing offloads.
 	 */
 	if (vlan_has_tag(mp)) {
 		vtag = htole16(vlan_get_tag(mp));
 		vlan_macip_lens |= (vtag << IXGBE_ADVTXD_VLAN_SHIFT);
 	} else if (!(txr->adapter->feat_en & IXGBE_FEATURE_NEEDS_CTXD) &&
 	           (offload == FALSE))
 		return (0);
 	/*
 	 * Determine where frame payload starts.
 	 * Jump over vlan headers if already present,
 	 * helpful for QinQ too.
 	 */
 	KASSERT(mp->m_len >= offsetof(struct ether_vlan_header, evl_tag));
 	eh = mtod(mp, struct ether_vlan_header *);
 	if (eh->evl_encap_proto == htons(ETHERTYPE_VLAN)) {
 		KASSERT(mp->m_len >= sizeof(struct ether_vlan_header));
 		etype = ntohs(eh->evl_proto);
 		ehdrlen = ETHER_HDR_LEN + ETHER_VLAN_ENCAP_LEN;
 	} else {
 		etype = ntohs(eh->evl_encap_proto);
 		ehdrlen = ETHER_HDR_LEN;
+	}
 	/* Set the ether header length */
 	vlan_macip_lens |= ehdrlen << IXGBE_ADVTXD_MACLEN_SHIFT;
 	if (offload == FALSE)
 		goto no_offloads;
 	/*
 	 * If the first mbuf only includes the ethernet header,
 	 * jump to the next one
 	 * XXX: This assumes the stack splits mbufs containing headers
 	 *      on header boundaries
 	 * XXX: And assumes the entire IP header is contained in one mbuf
 	 */
 	if (mp->m_len == ehdrlen && mp->m_next)
 		l3d = mtod(mp->m_next, char *);
 	else
 		l3d = mtod(mp, char *) + ehdrlen;
 	switch (etype) {
 #ifdef INET
 	case ETHERTYPE_IP:
 		ip = (struct ip *)(l3d);
 		ip_hlen = ip->ip_hl << 2;
 		ipproto = ip->ip_p;
 		type_tucmd_mlhl |= IXGBE_ADVTXD_TUCMD_IPV4;
 		KASSERT((mp->m_pkthdr.csum_flags & M_CSUM_IPv4) == 0 ||
 		    ip->ip_sum == 0);
 		break;
 #endif
 #ifdef INET6
 	case ETHERTYPE_IPV6:
 		ip6 = (struct ip6_hdr *)(l3d);
 		ip_hlen = sizeof(struct ip6_hdr);
 		ipproto = ip6->ip6_nxt;
 		type_tucmd_mlhl |= IXGBE_ADVTXD_TUCMD_IPV6;
 		break;
 #endif
 	default:
 		offload = false;
 		break;
+	}
 	if ((mp->m_pkthdr.csum_flags & M_CSUM_IPv4) != 0)
 		*olinfo_status |= IXGBE_TXD_POPTS_IXSM << 8;
 	vlan_macip_lens |= ip_hlen;
 	/* No support for offloads for non-L4 next headers */
  	switch (ipproto) {
 	case IPPROTO_TCP:
 		if (mp->m_pkthdr.csum_flags &
 		    (M_CSUM_TCPv4 | M_CSUM_TCPv6))
 			type_tucmd_mlhl |= IXGBE_ADVTXD_TUCMD_L4T_TCP;
 		else
 			offload = false;
 		break;
 	case IPPROTO_UDP:
 		if (mp->m_pkthdr.csum_flags &
 		    (M_CSUM_UDPv4 | M_CSUM_UDPv6))
 			type_tucmd_mlhl |= IXGBE_ADVTXD_TUCMD_L4T_UDP;
 		else
 			offload = false;
 		break;
 	default:
 		offload = false;
 		break;
+	}
 	if (offload) /* Insert L4 checksum into data descriptors */
 		*olinfo_status |= IXGBE_TXD_POPTS_TXSM << 8;
 no_offloads:
 	type_tucmd_mlhl |= IXGBE_ADVTXD_DCMD_DEXT | IXGBE_ADVTXD_DTYP_CTXT;
 	/* Now copy bits into descriptor */
 	TXD->vlan_macip_lens = htole32(vlan_macip_lens);
 	TXD->type_tucmd_mlhl = htole32(type_tucmd_mlhl);
 	TXD->seqnum_seed = htole32(0);
 	TXD->mss_l4len_idx = htole32(0);
 	/* We've consumed the first desc, adjust counters */
 	if (++ctxd == txr->num_desc)
 		ctxd = 0;
 	txr->next_avail_desc = ctxd;
 	--txr->tx_avail;
 	return (0);
 } /* ixgbe_tx_ctx_setup */
 /************************************************************************
  * ixgbe_tso_setup
+ *
  *   Setup work for hardware segmentation offload (TSO) on
  *   adapters using advanced tx descriptors
  ************************************************************************/
 static int
 ixgbe_tso_setup(struct tx_ring *txr, struct mbuf *mp, u32 *cmd_type_len,
     u32 *olinfo_status)
+{
 	struct ixgbe_adv_tx_context_desc *TXD;
 	struct ether_vlan_header         *eh;
 #ifdef INET6
 	struct ip6_hdr                   *ip6;
 #endif
 #ifdef INET
 	struct ip                        *ip;
 #endif
 	struct tcphdr                    *th;
 	int                              ctxd, ehdrlen, ip_hlen, tcp_hlen;
 	u32                              vlan_macip_lens = 0;
 	u32                              type_tucmd_mlhl = 0;
 	u32                              mss_l4len_idx = 0, paylen;
 	u16                              vtag = 0, eh_type;
 	/*
 	 * Determine where frame payload starts.
 	 * Jump over vlan headers if already present
 	 */
 	eh = mtod(mp, struct ether_vlan_header *);
 	if (eh->evl_encap_proto == htons(ETHERTYPE_VLAN)) {
 		ehdrlen = ETHER_HDR_LEN + ETHER_VLAN_ENCAP_LEN;
 		eh_type = eh->evl_proto;
 	} else {
 		ehdrlen = ETHER_HDR_LEN;
 		eh_type = eh->evl_encap_proto;
+	}
 	switch (ntohs(eh_type)) {
 #ifdef INET
 	case ETHERTYPE_IP:
 		ip = (struct ip *)(mp->m_data + ehdrlen);
 		if (ip->ip_p != IPPROTO_TCP)
 			return (ENXIO);
 		ip->ip_sum = 0;
 		ip_hlen = ip->ip_hl << 2;
 		th = (struct tcphdr *)((char *)ip + ip_hlen);
 		th->th_sum = in_cksum_phdr(ip->ip_src.s_addr,
 		    ip->ip_dst.s_addr, htons(IPPROTO_TCP));
 		type_tucmd_mlhl |= IXGBE_ADVTXD_TUCMD_IPV4;
 		/* Tell transmit desc to also do IPv4 checksum. */
 		*olinfo_status |= IXGBE_TXD_POPTS_IXSM << 8;
 		break;
 #endif
 #ifdef INET6
 	case ETHERTYPE_IPV6:
 		ip6 = (struct ip6_hdr *)(mp->m_data + ehdrlen);
 		/* XXX-BZ For now we do not pretend to support ext. hdrs. */
 		if (ip6->ip6_nxt != IPPROTO_TCP)
 			return (ENXIO);
 		ip_hlen = sizeof(struct ip6_hdr);
 		ip6 = (struct ip6_hdr *)(mp->m_data + ehdrlen);
 		th = (struct tcphdr *)((char *)ip6 + ip_hlen);
 		th->th_sum = in6_cksum_phdr(&ip6->ip6_src,
 		    &ip6->ip6_dst, 0, htonl(IPPROTO_TCP));
 		type_tucmd_mlhl |= IXGBE_ADVTXD_TUCMD_IPV6;
 		break;
 #endif
 	default:
 		panic("%s: CSUM_TSO but no supported IP version (0x%04x)",
 		    __func__, ntohs(eh_type));
 		break;
+	}
 	ctxd = txr->next_avail_desc;
 	TXD = (struct ixgbe_adv_tx_context_desc *)&txr->tx_base[ctxd];
 	tcp_hlen = th->th_off << 2;
 	/* This is used in the transmit desc in encap */
 	paylen = mp->m_pkthdr.len - ehdrlen - ip_hlen - tcp_hlen;
 	/* VLAN MACLEN IPLEN */
 	if (vlan_has_tag(mp)) {
 		vtag = htole16(vlan_get_tag(mp));
 		vlan_macip_lens |= (vtag << IXGBE_ADVTXD_VLAN_SHIFT);
+	}
 	vlan_macip_lens |= ehdrlen << IXGBE_ADVTXD_MACLEN_SHIFT;
 	vlan_macip_lens |= ip_hlen;
 	TXD->vlan_macip_lens = htole32(vlan_macip_lens);
 	/* ADV DTYPE TUCMD */
 	type_tucmd_mlhl |= IXGBE_ADVTXD_DCMD_DEXT | IXGBE_ADVTXD_DTYP_CTXT;
 	type_tucmd_mlhl |= IXGBE_ADVTXD_TUCMD_L4T_TCP;
 	TXD->type_tucmd_mlhl = htole32(type_tucmd_mlhl);
 	/* MSS L4LEN IDX */
 	mss_l4len_idx |= (mp->m_pkthdr.segsz << IXGBE_ADVTXD_MSS_SHIFT);
 	mss_l4len_idx |= (tcp_hlen << IXGBE_ADVTXD_L4LEN_SHIFT);
 	TXD->mss_l4len_idx = htole32(mss_l4len_idx);
 	TXD->seqnum_seed = htole32(0);
 	if (++ctxd == txr->num_desc)
 		ctxd = 0;
 	txr->tx_avail--;
 	txr->next_avail_desc = ctxd;
 	*cmd_type_len |= IXGBE_ADVTXD_DCMD_TSE;
 	*olinfo_status |= IXGBE_TXD_POPTS_TXSM << 8;
 	*olinfo_status |= paylen << IXGBE_ADVTXD_PAYLEN_SHIFT;
 	++txr->tso_tx.ev_count;
 	return (0);
 } /* ixgbe_tso_setup */
 /************************************************************************
  * ixgbe_txeof
+ *
  *   Examine each tx_buffer in the used queue. If the hardware is done
  *   processing the packet then free associated resources. The
  *   tx_buffer is put back on the free queue.
  ************************************************************************/
 bool
 ixgbe_txeof(struct tx_ring *txr)
+{
 	struct adapter		*adapter = txr->adapter;
 	struct ifnet		*ifp = adapter->ifp;
 	struct ixgbe_tx_buf	*buf;
 	union ixgbe_adv_tx_desc *txd;
 	u32			work, processed = 0;
 	u32			limit = adapter->tx_process_limit;
 	KASSERT(mutex_owned(&txr->tx_mtx));
 #ifdef DEV_NETMAP
 	if ((adapter->feat_en & IXGBE_FEATURE_NETMAP) &&
 	    (adapter->ifp->if_capenable & IFCAP_NETMAP)) {
 		struct netmap_adapter *na = NA(adapter->ifp);
 		struct netmap_kring *kring = &na->tx_rings[txr->me];
 		txd = txr->tx_base;
 		bus_dmamap_sync(txr->txdma.dma_tag, txr->txdma.dma_map,
 		    BUS_DMASYNC_POSTREAD);
 		/*
 		 * In netmap mode, all the work is done in the context
 		 * of the client thread. Interrupt handlers only wake up
 		 * clients, which may be sleeping on individual rings
 		 * or on a global resource for all rings.
 		 * To implement tx interrupt mitigation, we wake up the client
 		 * thread roughly every half ring, even if the NIC interrupts
 		 * more frequently. This is implemented as follows:
 		 * - ixgbe_txsync() sets kring->nr_kflags with the index of
 		 *   the slot that should wake up the thread (nkr_num_slots
 		 *   means the user thread should not be woken up);
 		 * - the driver ignores tx interrupts unless netmap_mitigate=0
 		 *   or the slot has the DD bit set.
 		 */
 		if (!netmap_mitigate ||
 		    (kring->nr_kflags < kring->nkr_num_slots &&
 		     txd[kring->nr_kflags].wb.status & IXGBE_TXD_STAT_DD)) {
 			netmap_tx_irq(ifp, txr->me);
+		}
 		return false;
+	}
 #endif /* DEV_NETMAP */
 	if (txr->tx_avail == txr->num_desc) {
 		txr->busy = 0;
 		return false;
+	}
 	/* Get work starting point */
 	work = txr->next_to_clean;
 	buf = &txr->tx_buffers[work];
 	txd = &txr->tx_base[work];
 	work -= txr->num_desc; /* The distance to ring end */
 	ixgbe_dmamap_sync(txr->txdma.dma_tag, txr->txdma.dma_map,
 	    BUS_DMASYNC_POSTREAD);
 	do {
 		union ixgbe_adv_tx_desc *eop = buf->eop;
 		if (eop == NULL) /* No work */
 			break;
 		if ((eop->wb.status & IXGBE_TXD_STAT_DD) == 0)
 			break;	/* I/O not complete */
 		if (buf->m_head) {
 			txr->bytes += buf->m_head->m_pkthdr.len;
 			bus_dmamap_sync(txr->txtag->dt_dmat, buf->map,
 , buf->m_head->m_pkthdr.len,
 			    BUS_DMASYNC_POSTWRITE);
 			ixgbe_dmamap_unload(txr->txtag, buf->map);
 			m_freem(buf->m_head);
 			buf->m_head = NULL;
+		}
 		buf->eop = NULL;
 		txr->txr_no_space = false;
 		++txr->tx_avail;
 		/* We clean the range if multi segment */
 		while (txd != eop) {
 			++txd;
 			++buf;
 			++work;
 			/* wrap the ring? */
 			if (__predict_false(!work)) {
 				work -= txr->num_desc;
 				buf = txr->tx_buffers;
 				txd = txr->tx_base;
+			}
 			if (buf->m_head) {
 				txr->bytes +=
 				    buf->m_head->m_pkthdr.len;
 				bus_dmamap_sync(txr->txtag->dt_dmat,
 				    buf->map,
 , buf->m_head->m_pkthdr.len,
 				    BUS_DMASYNC_POSTWRITE);
 				ixgbe_dmamap_unload(txr->txtag,
 				    buf->map);
 				m_freem(buf->m_head);
 				buf->m_head = NULL;
+			}
 			++txr->tx_avail;
 			buf->eop = NULL;
+		}
 		++txr->packets;
 		++processed;
 		++ifp->if_opackets;
 		/* Try the next packet */
 		++txd;
 		++buf;
 		++work;
 		/* reset with a wrap */
 		if (__predict_false(!work)) {
 			work -= txr->num_desc;
 			buf = txr->tx_buffers;
 			txd = txr->tx_base;
+		}
 		prefetch(txd);
 	} while (__predict_true(--limit));
 	ixgbe_dmamap_sync(txr->txdma.dma_tag, txr->txdma.dma_map,
 	    BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
 	work += txr->num_desc;
 	txr->next_to_clean = work;
 	/*
 	 * Queue Hang detection, we know there's
 	 * work outstanding or the first return
 	 * would have been taken, so increment busy
 	 * if nothing managed to get cleaned, then
 	 * in local_timer it will be checked and
 	 * marked as HUNG if it exceeds a MAX attempt.
 	 */
 	if ((processed == 0) && (txr->busy != IXGBE_QUEUE_HUNG))
 		++txr->busy;
 	/*
 	 * If anything gets cleaned we reset state to 1,
 	 * note this will turn off HUNG if its set.
 	 */