On Thu, 23 Jul 2026 at 22:29, Stephen Hemminger <[email protected]> wrote: > > The per-queue stats mapping mechanism > (rte_eth_dev_set_{rx,tx}_queue_stats_mapping > and the queue_stats_mapping_set ethdev op) was a workaround for hardware that > could only expose a limited number of queue stat registers. It let an > application > fold many queues onto a small set of counters. Since 25.11 queue statistics > are > filled in directly by the ethdev layer indexed by queue id, so the mapping is > no > longer needed; the functions were deprecated then by commit 2863cbd9ee6d. > > This series finishes the job. The drivers that still implemented the op are > converted to report queue stats directly by queue id (bounded by > RTE_ETHDEV_QUEUE_STAT_CNTRS), and the op, the two public functions, their > trace > points, and the documentation are removed from ethdev. The leftover per-driver > mapping tables, accessor macros, and init code go away with them. > > Also, found some leftover bits from the earlier removal. > > Stephen Hemminger (9): > net/virtio: remove unused queue stats mapping > app/testpmd: remove unused function prototype > net/enic: remove unneeded ops initialization > net/cnxk: remove queue stats mapping > net/e1000: remove queue stats mapping > net/ixgbe: remove queue stats mapping > net/txgbe: remove queue stats mapping > net/sxe2: remove queue stats mapping > ethdev: remove support for queue stats mapping > > app/test-pmd/cmdline.c | 6 -- > app/test-pmd/testpmd.h | 2 - > doc/guides/nics/features.rst | 7 -- > doc/guides/rel_notes/deprecation.rst | 6 -- > doc/guides/rel_notes/release_26_11.rst | 5 ++ > drivers/net/cnxk/cnxk_ethdev.c | 1 - > drivers/net/cnxk/cnxk_ethdev.h | 6 -- > drivers/net/cnxk/cnxk_stats.c | 76 ++++++++------------ > drivers/net/enic/enic_ethdev.c | 11 --- > drivers/net/intel/e1000/igc_ethdev.c | 54 ++------------ > drivers/net/intel/e1000/igc_ethdev.h | 2 - > drivers/net/intel/ixgbe/ixgbe_ethdev.c | 97 ------------------------- > drivers/net/intel/ixgbe/ixgbe_ethdev.h | 9 --- > drivers/net/sxe2/sxe2_cmd_chnl.c | 99 -------------------------- > drivers/net/sxe2/sxe2_cmd_chnl.h | 10 --- > drivers/net/sxe2/sxe2_ethdev.c | 5 -- > drivers/net/sxe2/sxe2_stats.c | 76 +------------------- > drivers/net/sxe2/sxe2_stats.h | 5 -- > drivers/net/txgbe/txgbe_ethdev.c | 87 ++-------------------- > drivers/net/txgbe/txgbe_ethdev.h | 13 ---- > drivers/net/virtio/virtio_ethdev.c | 21 ------ > lib/ethdev/ethdev_driver.h | 11 --- > lib/ethdev/ethdev_trace.h | 20 ------ > lib/ethdev/ethdev_trace_points.c | 6 -- > lib/ethdev/rte_ethdev.c | 62 ---------------- > lib/ethdev/rte_ethdev.h | 45 ------------ > 26 files changed, 53 insertions(+), 689 deletions(-)
I sent two comments but the series itself lgtm. For the series: Acked-by: David Marchand <[email protected]> -- David Marchand
