Author: davidcs
Date: Tue Oct 10 20:45:45 2017
New Revision: 324504
URL: https://svnweb.freebsd.org/changeset/base/324504

Log:
  Revert Commit r324290
    Add sanity checks in ql_hw_send() qla_send() to ensure that empty slots
    in Tx Ring map to empty slot in Tx_buf array before Transmits. If the
    checks fail further Transmission on that Tx Ring is prevented.

Modified:
  head/sys/dev/qlxgbe/ql_hw.c
  head/sys/dev/qlxgbe/ql_os.c

Modified: head/sys/dev/qlxgbe/ql_hw.c
==============================================================================
--- head/sys/dev/qlxgbe/ql_hw.c Tue Oct 10 20:08:30 2017        (r324503)
+++ head/sys/dev/qlxgbe/ql_hw.c Tue Oct 10 20:45:45 2017        (r324504)
@@ -2374,16 +2374,6 @@ ql_hw_send(qla_host_t *ha, bus_dma_segment_t *segs, in
                }
        }
 
-       for (i = 0; i < num_tx_cmds; i++) {
-               if (NULL != ha->tx_ring[txr_idx].tx_buf[(tx_idx+i)].m_head) {
-                       QL_ASSERT(ha, 0, \
-                               ("%s: txr_idx = %d tx_idx = %d mbuf = %p\n",\
-                               __func__, txr_idx, (tx_idx+i),\
-                               
ha->tx_ring[txr_idx].tx_buf[(tx_idx+i)].m_head));
-                       return (EINVAL);
-               }
-       }
-
        tx_cmd = &hw->tx_cntxt[txr_idx].tx_ring_base[tx_idx];
 
         if (!(mp->m_pkthdr.csum_flags & CSUM_TSO)) {

Modified: head/sys/dev/qlxgbe/ql_os.c
==============================================================================
--- head/sys/dev/qlxgbe/ql_os.c Tue Oct 10 20:08:30 2017        (r324503)
+++ head/sys/dev/qlxgbe/ql_os.c Tue Oct 10 20:45:45 2017        (r324504)
@@ -1232,17 +1232,6 @@ qla_send(qla_host_t *ha, struct mbuf **m_headp, uint32
        QL_DPRINT8(ha, (ha->pci_dev, "%s: enter\n", __func__));
 
        tx_idx = ha->hw.tx_cntxt[txr_idx].txr_next;
-
-       if (NULL != ha->tx_ring[txr_idx].tx_buf[tx_idx].m_head) {
-               QL_ASSERT(ha, 0, ("%s: txr_idx = %d tx_idx = %d mbuf = %p\n",\
-                       __func__, txr_idx, tx_idx,\
-                       ha->tx_ring[txr_idx].tx_buf[tx_idx].m_head));
-               if (m_head)
-                       m_freem(m_head);
-               *m_headp = NULL;
-               return (ret);
-       }
-
        map = ha->tx_ring[txr_idx].tx_buf[tx_idx].map;
 
        ret = bus_dmamap_load_mbuf_sg(ha->tx_tag, map, m_head, segs, &nsegs,
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to