On 8/18/22 15:23, Ferruh Yigit wrote:
On 8/18/2022 10:37 AM, Morten Brørup wrote:
Applications may use rte_eth_rx_queue_count() in the RX stage of the
dataplane, so only check the function parameters if built with
RTE_ETHDEV_DEBUG_RX.

Signed-off-by: Morten Brørup <[email protected]>

I don't remember if those checks done by default intentionally, but I don't think so,

And since other relevant functions do the same thing:

Acked-by: Ferruh Yigit <[email protected]>


The patch itself looks good to me.
Summary should be fixed to be not a statement and should not
mention function name. Other than that:

Reviewed-by: Andrew Rybchenko <[email protected]>

Reply via email to