]> git.droids-corp.org - dpdk.git/commit
net/e1000: fix Tx descriptor status api
authorOlivier Matz <olivier.matz@6wind.com>
Mon, 25 Jun 2018 13:34:49 +0000 (15:34 +0200)
committerOlivier Matz <olivier.matz@6wind.com>
Mon, 25 Jun 2018 13:38:07 +0000 (15:38 +0200)
commitd506c50015296a1dacc83ed1e9edf10718a4caa3
tree6a5a046fe5532a8ef17d0eb33864aed4b2fed698
parentde0ae4440ad84edc4eb6892eb2a618712d32fa78
net/e1000: fix Tx descriptor status api

The Tx descriptor status api was not behaving as expected. This API is
used to inspect the content of the descriptors in the Tx ring to
determine the length of the Tx queue.

Since the software advances the tail pointer and the hardware advances
the head pointer, the Tx queue is located before txq->tx_tail in the
ring. Therefore, a call to rte_eth_tx_descriptor_status(..., offset=20)
should inspect the 20th descriptor before the tail, not after.

As before, we still need to take care about only checking descriptors
that have the RS bit.

Additionally, we can avoid an access to the ring if offset is greater or
equal to nb_tx_desc - nb_tx_free.

Fixes: 8cd01eb049fb ("net/e1000: implement descriptor status API")
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Signed-off-by: Didier Pallard <didier.pallard@6wind.com>
drivers/net/e1000/em_rxtx.c