Re: [PATCH v2 0/6] fix the handling of deferred start in ethdev

[Bruce Richardson <bruce.richardson@intel.com>]

On Fri, Feb 14, 2025 at 09:38:53AM -0800, Stephen Hemminger wrote:
> There is a inconsistency and lack of documentation around the deferred
> start configuration flag. The queue deferred start only makes sense if
> the driver has a way to start the queue later via ethdev ops.
> Therefore move the check into the ethdev queue configuration handling.
> Fix the driver documentation where the support of deferred start
> does not match what the driver implements.
> 
> Stephen Hemminger (6):
>   ethdev: check that device supports deferred start
>   doc: fix feature flags for queue start/stop
>   net/dpaa: remove unnecessary deferred start check
>   net/dpaa2: remove unneeded deferred start check
>   net/enetfec: remove unneeded deferred start check
>   net/virtio: remove unneeded deferred start check
> 
> v2 - rebase and cleanup wording
> 

This seems a good idea to do.

Series-acked-by: Bruce Richardson <bruce.richardson@intel.com>

>  doc/guides/nics/features.rst         |  1 +
>  doc/guides/nics/features/gve.ini     |  1 +
>  doc/guides/nics/features/mana.ini    |  1 -
>  doc/guides/nics/features/netvsc.ini  |  1 -
>  doc/guides/nics/features/virtio.ini  |  1 -
>  doc/guides/nics/features/vmxnet3.ini |  1 -
>  drivers/net/dpaa/dpaa_ethdev.c       | 10 ----------
>  drivers/net/dpaa2/dpaa2_ethdev.c     | 14 --------------
>  drivers/net/enetfec/enet_ethdev.c    | 16 ++--------------
>  drivers/net/virtio/virtio_rxtx.c     | 10 ----------
>  lib/ethdev/rte_ethdev.c              | 14 ++++++++++++++
>  11 files changed, 18 insertions(+), 52 deletions(-)
> 
> -- 
> 2.47.2
>