[PATCH v2 0/9] Complete removal of queue stats mapping

[David Marchand]

On Thu, 23 Jul 2026 at 22:29, Stephen Hemminger
<stephen at networkplumber.org> 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 <david.marchand at redhat.com>


-- 
David Marchand