d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Couple minor if_vmx tweaks

Browse Source 
- Use queue size fields from the Tx/Rx queues in various places
    instead of (currently the same values) from the softc.
  - Fix potential crash in detach if the attached failed to alloc
    queue memory.
  - Move the VMXNET3_MAX_RX_SEGS define to a better spot.
  - Tweak frame size calculation w.r.t. ETHER_ALIGN. This could be
    tweaked some more, or removed since it probably doesn't matter
    much for x86 (and the x86 class of machines this driver will
    be used on).
This commit is contained in:
Bryan Venteicher 2013-08-27 04:05:18 +00:00
parent 5a5347c3f1
commit 3c9657753e
2 changed files with 32 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
sys/dev/vmware/vmxnet3/if_vmx.c
View File

@ -437,14 +437,14 @@ vmxnet3_check_version(struct vmxnet3_softc *sc)
device_printf(dev, "unsupported hardware version %#x\n", device_printf(dev, "unsupported hardware version %#x\n",
version); version);
return (ENOTSUP); return (ENOTSUP);
} else }
vmxnet3_write_bar1(sc, VMXNET3_BAR1_VRRS, 1); vmxnet3_write_bar1(sc, VMXNET3_BAR1_VRRS, 1);
version = vmxnet3_read_bar1(sc, VMXNET3_BAR1_UVRS); version = vmxnet3_read_bar1(sc, VMXNET3_BAR1_UVRS);
if ((version & 0x01) == 0) { if ((version & 0x01) == 0) {
device_printf(dev, "unsupported UPT version %#x\n", version); device_printf(dev, "unsupported UPT version %#x\n", version);
return (ENOTSUP); return (ENOTSUP);
} else }
vmxnet3_write_bar1(sc, VMXNET3_BAR1_UVRS, 1); vmxnet3_write_bar1(sc, VMXNET3_BAR1_UVRS, 1);
return (0); return (0);
@ -781,10 +781,9 @@ vmxnet3_init_rxq(struct vmxnet3_softc *sc, int q)
sizeof(struct vmxnet3_rxbuf), M_DEVBUF, M_NOWAIT | M_ZERO); sizeof(struct vmxnet3_rxbuf), M_DEVBUF, M_NOWAIT | M_ZERO);
if (rxr->vxrxr_rxbuf == NULL) if (rxr->vxrxr_rxbuf == NULL)
return (ENOMEM); return (ENOMEM);
}
rxq->vxrxq_comp_ring.vxcr_ndesc = rxq->vxrxq_comp_ring.vxcr_ndesc += sc->vmx_nrxdescs;
sc->vmx_nrxdescs * VMXNET3_RXRINGS_PERQ; }
return (0); return (0);
} }
@ -1240,7 +1239,10 @@ static void
vmxnet3_free_queue_data(struct vmxnet3_softc *sc) vmxnet3_free_queue_data(struct vmxnet3_softc *sc)
{ {
if (sc->vmx_rxq != NULL)
vmxnet3_free_rxq_data(sc); vmxnet3_free_rxq_data(sc);
if (sc->vmx_txq != NULL)
vmxnet3_free_txq_data(sc); vmxnet3_free_txq_data(sc);
} }
@ -1325,9 +1327,9 @@ vmxnet3_init_shared_data(struct vmxnet3_softc *sc)
txs = txq->vxtxq_ts; txs = txq->vxtxq_ts;
txs->cmd_ring = txq->vxtxq_cmd_ring.vxtxr_dma.dma_paddr; txs->cmd_ring = txq->vxtxq_cmd_ring.vxtxr_dma.dma_paddr;
txs->cmd_ring_len = sc->vmx_ntxdescs; txs->cmd_ring_len = txq->vxtxq_cmd_ring.vxtxr_ndesc;
txs->comp_ring = txq->vxtxq_comp_ring.vxcr_dma.dma_paddr; txs->comp_ring = txq->vxtxq_comp_ring.vxcr_dma.dma_paddr;
txs->comp_ring_len = sc->vmx_ntxdescs; txs->comp_ring_len = txq->vxtxq_comp_ring.vxcr_ndesc;
txs->driver_data = vtophys(txq); txs->driver_data = vtophys(txq);
txs->driver_data_len = sizeof(struct vmxnet3_txqueue); txs->driver_data_len = sizeof(struct vmxnet3_txqueue);
} }
@ -1342,8 +1344,7 @@ vmxnet3_init_shared_data(struct vmxnet3_softc *sc)
rxs->cmd_ring[1] = rxq->vxrxq_cmd_ring[1].vxrxr_dma.dma_paddr; rxs->cmd_ring[1] = rxq->vxrxq_cmd_ring[1].vxrxr_dma.dma_paddr;
rxs->cmd_ring_len[1] = rxq->vxrxq_cmd_ring[1].vxrxr_ndesc; rxs->cmd_ring_len[1] = rxq->vxrxq_cmd_ring[1].vxrxr_ndesc;
rxs->comp_ring = rxq->vxrxq_comp_ring.vxcr_dma.dma_paddr; rxs->comp_ring = rxq->vxrxq_comp_ring.vxcr_dma.dma_paddr;
rxs->comp_ring_len = rxq->vxrxq_cmd_ring[0].vxrxr_ndesc + rxs->comp_ring_len = rxq->vxrxq_comp_ring.vxcr_ndesc;
rxq->vxrxq_cmd_ring[1].vxrxr_ndesc;
rxs->driver_data = vtophys(rxq); rxs->driver_data = vtophys(rxq);
rxs->driver_data_len = sizeof(struct vmxnet3_rxqueue); rxs->driver_data_len = sizeof(struct vmxnet3_rxqueue);
} }
@ -1558,6 +1559,7 @@ vmxnet3_txq_eof(struct vmxnet3_txqueue *txq)
txcd = &txc->vxcr_u.txcd[txc->vxcr_next]; txcd = &txc->vxcr_u.txcd[txc->vxcr_next];
if (txcd->gen != txc->vxcr_gen) if (txcd->gen != txc->vxcr_gen)
break; break;
vmxnet3_barrier(sc, VMXNET3_BARRIER_RD);
if (++txc->vxcr_next == txc->vxcr_ndesc) { if (++txc->vxcr_next == txc->vxcr_ndesc) {
txc->vxcr_next = 0; txc->vxcr_next = 0;
@ -1647,7 +1649,7 @@ vmxnet3_newbuf(struct vmxnet3_softc *sc, struct vmxnet3_rxring *rxr)
BUS_DMA_NOWAIT); BUS_DMA_NOWAIT);
if (error) { if (error) {
m_freem(m); m_freem(m);
sc->vmx_stats.vmst_mbuf_load_failed++;; sc->vmx_stats.vmst_mbuf_load_failed++;
return (error); return (error);
} }
KASSERT(nsegs == 1, KASSERT(nsegs == 1,
@ -2119,19 +2121,19 @@ vmxnet3_rxinit(struct vmxnet3_softc *sc, struct vmxnet3_rxqueue *rxq)
int i, populate, idx, frame_size, error; int i, populate, idx, frame_size, error;
ifp = sc->vmx_ifp; ifp = sc->vmx_ifp;
frame_size = ifp->if_mtu + sizeof(struct ether_vlan_header); frame_size = ETHER_ALIGN + sizeof(struct ether_vlan_header) +
ifp->if_mtu;
/* /*
* If the MTU causes us to exceed what a regular sized cluster * If the MTU causes us to exceed what a regular sized cluster can
* can handle, we allocate a second MJUMPAGESIZE cluster after * handle, we allocate a second MJUMPAGESIZE cluster after it in
* it in ring 0. If in use, ring 1 always contains MJUMPAGESIZE * ring 0. If in use, ring 1 always contains MJUMPAGESIZE clusters.
* clusters.
* *
* Keep rx_max_chain a divisor of the maximum Rx ring size to * Keep rx_max_chain a divisor of the maximum Rx ring size to make
* to make our life easier. We do not support changing the ring * our life easier. We do not support changing the ring size after
* size after the attach. * the attach.
*/ */
if (frame_size <= MCLBYTES - ETHER_ALIGN) if (frame_size <= MCLBYTES)
sc->vmx_rx_max_chain = 1; sc->vmx_rx_max_chain = 1;
else else
sc->vmx_rx_max_chain = 2; sc->vmx_rx_max_chain = 2;

14
sys/dev/vmware/vmxnet3/if_vmxvar.h
View File

@ -50,13 +50,6 @@ struct vmxnet3_dma_alloc {
#define VMXNET3_MAX_RX_NCOMPDESC \ #define VMXNET3_MAX_RX_NCOMPDESC \
(VMXNET3_MAX_RX_NDESC * VMXNET3_RXRINGS_PERQ) (VMXNET3_MAX_RX_NDESC * VMXNET3_RXRINGS_PERQ)
/*
* The maximum number of Rx segments we accept. When LRO is enabled,
* this allows us to receive the maximum sized frame with one MCLBYTES
* cluster followed by 16 MJUMPAGESIZE clusters.
*/
#define VMXNET3_MAX_RX_SEGS 17
struct vmxnet3_txbuf { struct vmxnet3_txbuf {
bus_dmamap_t vtxb_dmamap; bus_dmamap_t vtxb_dmamap;
struct mbuf *vtxb_m; struct mbuf *vtxb_m;
@ -271,6 +264,13 @@ struct vmxnet3_softc {
*/ */
#define VMXNET3_TX_MAXSEGSIZE (1 << 14) #define VMXNET3_TX_MAXSEGSIZE (1 << 14)
/*
* The maximum number of Rx segments we accept. When LRO is enabled,
* this allows us to receive the maximum sized frame with one MCLBYTES
* cluster followed by 16 MJUMPAGESIZE clusters.
*/
#define VMXNET3_MAX_RX_SEGS 17
/* /*
* Predetermined size of the multicast MACs filter table. If the * Predetermined size of the multicast MACs filter table. If the
* number of multicast addresses exceeds this size, then the * number of multicast addresses exceeds this size, then the