d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

ixgbe: fix offloading bits when Rx bulk alloc is used

Browse Source 
This is a fix for the ixgbe hardware offload flags not being set
when bulk alloc RX is used. The issue was caused by masking off
the bits that store the hardware offload values in the status_error
field to retrieve the done bit for the descriptor.

Commit 7431041062 in DPDK-1.3.0
introduced bulk dequeue, which included the bug.

Signed-off-by: Bryan Benson <bmbenson@amazon.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
This commit is contained in:
Bryan Benson 2013-11-08 19:47:22 +00:00 committed by David Marchand
parent 27399312ef
commit 2c90eb650c
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
lib/librte_pmd_ixgbe/ixgbe_rxtx.c
View File

@ -1040,7 +1040,8 @@ ixgbe_rx_scan_hw_ring(struct igb_rx_queue *rxq)
struct igb_rx_entry *rxep;
struct rte_mbuf *mb;
uint16_t pkt_len;
int s[LOOK_AHEAD], nb_dd;
uint32_t s[LOOK_AHEAD];
int nb_dd;
int i, j, nb_rx = 0;
@ -1063,12 +1064,12 @@ ixgbe_rx_scan_hw_ring(struct igb_rx_queue *rxq)
for (j = LOOK_AHEAD-1; j >= 0; --j)
s[j] = rxdp[j].wb.upper.status_error;
/* Clear everything but the status bits (LSB) */
for (j = 0; j < LOOK_AHEAD; ++j)
s[j] &= IXGBE_RXDADV_STAT_DD;
nb_dd = 0;
/* add to nd_dd when the status bit is set (LSB) */
for (j = 0; j < LOOK_AHEAD; ++j) {
nb_dd += s[j] & IXGBE_RXDADV_STAT_DD;
}
/* Compute how many status bits were set */
nb_dd = s[0]+s[1]+s[2]+s[3]+s[4]+s[5]+s[6]+s[7];
nb_rx += nb_dd;
/* Translate descriptor info to mbuf format */