]> git.droids-corp.org - dpdk.git/commit
ethdev: prohibit polling stopped queue
authorDmitry Kozlyuk <dkozlyuk@nvidia.com>
Sun, 10 Apr 2022 21:35:50 +0000 (00:35 +0300)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 25 May 2022 10:50:53 +0000 (12:50 +0200)
commitb77d3318d6a3ca266751031ff04520b7ad485529
tree83137723b8503e7f7632323f35e7aefa29ac4f67
parentb4f94d758f9eadf45e3fbefcc8dc5701ab73bfca
ethdev: prohibit polling stopped queue

Whether it is allowed to call Rx/Tx functions for a stopped queue
was undocumented. Some PMDs make this behavior a no-op
either by explicitly checking the queue state
or by the way how their routines are implemented or HW works.

No-op behavior may be convenient for application developers.
But it also means that pollers of stopped queues
would go all the way down to PMD Rx/Tx routines, wasting cycles.
Some PMDs would do a check for the queue state on data path,
even though it may never be needed for a particular application.
Also, use cases for stopping queues or starting them deferred
do not logically require polling stopped queues.

Use case 1: a secondary that was polling the queue has crashed,
the primary is doing a recovery to free all mbufs.
By definition the queue to be restarted is not polled.

Use case 2: deferred queue start or queue reconfiguration.
The polling thread must be synchronized anyway,
because queue start and stop are non-atomic.

Prohibit calling Rx/Tx functions on stopped queues.

Fixes: 0748be2cf9a2 ("ethdev: queue start and stop")
Cc: stable@dpdk.org
Signed-off-by: Dmitry Kozlyuk <dkozlyuk@nvidia.com>
Acked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>
lib/ethdev/rte_ethdev.h