net/ixgbe: fix Tx descriptor status api
authorOlivier Matz <olivier.matz@6wind.com>
Mon, 25 Jun 2018 13:34:50 +0000 (15:34 +0200)
committerOlivier Matz <olivier.matz@6wind.com>
Mon, 25 Jun 2018 13:38:17 +0000 (15:38 +0200)
commitba6b53716407104772daff10df7e4cc90931bf28
tree063ba59eace6ec6cb7ed65deeea0ff3a4623988c
parentd506c50015296a1dacc83ed1e9edf10718a4caa3
net/ixgbe: 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: a2919e13d95e ("net/ixgbe: implement descriptor status API")
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Signed-off-by: Didier Pallard <didier.pallard@6wind.com>
drivers/net/ixgbe/ixgbe_rxtx.c
drivers/net/ixgbe/ixgbe_rxtx.h