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.richard...@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 >
