diff options
| author | John Baldwin <jhb@FreeBSD.org> | 2012-04-12 13:53:49 +0000 |
|---|---|---|
| committer | John Baldwin <jhb@FreeBSD.org> | 2012-04-12 13:53:49 +0000 |
| commit | 6937a7ac6bc401550dc6c276eae8f78801e87a92 (patch) | |
| tree | 9a1148de78e9eee7ecb9308d385a3d945021d4f1 /sys/ofed | |
| parent | 9f2d8a383484635e415dead399a78d9b2c08c949 (diff) | |
| download | src-6937a7ac6bc401550dc6c276eae8f78801e87a92.tar.gz src-6937a7ac6bc401550dc6c276eae8f78801e87a92.zip | |
Don't update if_obytes when transmitting packets. That is already done
in IFQ_HANDOFF() when the packet is passed to the start routine, so doing
it here resulted in double counting.
Reported by: Alex Tutubalin lexa lexa ru
MFC after: 1 week
Notes
Notes:
svn path=/head/; revision=234182
Diffstat (limited to 'sys/ofed')
| -rw-r--r-- | sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_cm.c | 1 | ||||
| -rw-r--r-- | sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_ib.c | 1 |
2 files changed, 0 insertions, 2 deletions
diff --git a/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_cm.c b/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_cm.c index 4aaf2163afd7..47818bfc138a 100644 --- a/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_cm.c +++ b/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_cm.c @@ -682,7 +682,6 @@ void ipoib_cm_handle_tx_wc(struct ipoib_dev_priv *priv, struct ib_wc *wc) /* FIXME: is this right? Shouldn't we only increment on success? */ ++dev->if_opackets; - dev->if_obytes += tx_req->mb->m_pkthdr.len; m_freem(tx_req->mb); diff --git a/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_ib.c b/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_ib.c index d3b68bcf49d0..1afb80e23efe 100644 --- a/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_ib.c +++ b/sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_ib.c @@ -345,7 +345,6 @@ static void ipoib_ib_handle_tx_wc(struct ipoib_dev_priv *priv, struct ib_wc *wc) ipoib_dma_unmap_tx(priv->ca, tx_req); ++dev->if_opackets; - dev->if_obytes += tx_req->mb->m_pkthdr.len; m_freem(tx_req->mb); |