[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250209101716.112774-1-tariqt@nvidia.com>
Date: Sun, 9 Feb 2025 12:17:01 +0200
From: Tariq Toukan <tariqt@...dia.com>
To: "David S. Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>, Eric Dumazet <edumazet@...gle.com>, "Andrew
Lunn" <andrew+netdev@...n.ch>
CC: <netdev@...r.kernel.org>, Saeed Mahameed <saeedm@...dia.com>, Gal Pressman
<gal@...dia.com>, Leon Romanovsky <leonro@...dia.com>, Simon Horman
<horms@...nel.org>, Donald Hunter <donald.hunter@...il.com>, Jiri Pirko
<jiri@...nulli.us>, Jonathan Corbet <corbet@....net>, Leon Romanovsky
<leon@...nel.org>, Tariq Toukan <tariqt@...dia.com>, Alexei Starovoitov
<ast@...nel.org>, Daniel Borkmann <daniel@...earbox.net>, "Jesper Dangaard
Brouer" <hawk@...nel.org>, John Fastabend <john.fastabend@...il.com>,
"Richard Cochran" <richardcochran@...il.com>, <linux-kernel@...r.kernel.org>,
<linux-doc@...r.kernel.org>, <linux-rdma@...r.kernel.org>,
<bpf@...r.kernel.org>
Subject: [PATCH net-next 00/15] Rate management on traffic classes + misc
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Hi,
This patchset consists of multiple features from the team to the mlx5
core and Eth drivers.
The first 5 patches by Carolina are V7 of the feature that adds rate
management support on traffic classes in devlink and mlx5, more details
below [1].
Patches 6-8 by William reduce the memory consumption for representors to
achieve better scalability.
Patches 9-10 by Akiva expose ICM memory consumption per function.
Patches 11-13 expose helpful information on RSS resources in devlink RX
reporter diagnose.
Patches 14-15 are simple enhancements by Alex Lazar.
Regards,
Tariq
[1]
This is V7 of the feature. Find V6 here:
https://lore.kernel.org/all/20241209210950.290129-1-tariqt@nvidia.com/
This feature extends the devlink-rate API to support traffic class (TC)
bandwidth management, enabling more granular control over traffic
shaping and rate limiting across multiple TCs. The API now allows users
to specify bandwidth proportions for different traffic classes in a
single command. This is particularly useful for managing Enhanced
Transmission Selection (ETS) for groups of Virtual Functions (VFs),
allowing precise bandwidth allocation across traffic classes.
Additionally, it refines the QoS handling in net/mlx5 to support TC
arbitration and bandwidth management on vports and rate nodes.
Discussions on traffic class shaping in net-shapers began in V5 [2],
where we discussed with maintainers whether net-shapers should support
traffic classes and how this could be implemented.
Later, after further conversations with Paolo Abeni and Simon Horman,
Cosmin provided an update [3], confirming that net-shapers' tree-based
hierarchy aligns well with traffic classes when treated as distinct
subsets of netdev queues. Since mlx5 enforces a 1:1 mapping between TX
queues and traffic classes, this approach seems feasible, though some
open questions remain regarding queue reconfiguration and certain mlx5
scheduling behaviors.
[2]
https://lore.kernel.org/netdev/20241204220931.254964-1-tariqt@nvidia.com/
[3]
https://lore.kernel.org/netdev/67df1a562614b553dcab043f347a0d7c5393ff83.camel@nvidia.com/
V7:
- Fixed disabling tc-bw on leaf nodes that did not have tc-bw
configured.
- Fixed an issue where tc-bw was disabled on a node with assigned
vports, ensuring that vport->qos.sched_node->parent is correctly
updated with the cloned node.
- Declared a constant for the maximum allowed Traffic Class index in
devlink rate.
- Added a range check to validate rate-tc-index.
- Added documentation for the tc-bw argument.
- Add a validation check to ensure that the total bandwidth assigned to
all traffic classes sums to 100.
Akiva Goldberger (2):
net/mlx5: Rename and move mlx5_esw_query_vport_vhca_id
net/mlx5: Expose ICM consumption per function
Alexei Lazar (2):
net/mlx5: Extend Ethtool loopback selftest to support non-linear SKB
net/mlx5: XDP, Enable TX side XDP multi-buffer support
Amir Tzin (3):
net/mlx5e: Move RQs diagnose to a dedicated function
net/mlx5e: Add direct TIRs to devlink rx reporter diagnose
net/mlx5e: Expose RSS via devlink rx reporter diagnose
Carolina Jubran (5):
devlink: Extend devlink rate API with traffic classes bandwidth
management
net/mlx5: Add no-op implementation for setting tc-bw on rate objects
net/mlx5: Add support for setting tc-bw on nodes
net/mlx5: Add traffic class scheduling support for vport QoS
net/mlx5: Manage TC arbiter nodes and implement full support for tc-bw
William Tu (3):
net/mlx5e: reduce the max log mpwrq sz for ECPF and reps
net/mlx5e: reduce rep rxq depth to 256 for ECPF
net/mlx5e: set the tx_queue_len for pfifo_fast
Documentation/netlink/specs/devlink.yaml | 36 +-
.../networking/devlink/devlink-port.rst | 7 +
Documentation/networking/devlink/mlx5.rst | 4 +
.../net/ethernet/mellanox/mlx5/core/devlink.c | 2 +
.../mellanox/mlx5/core/diag/reporter_vnic.c | 46 +
drivers/net/ethernet/mellanox/mlx5/core/en.h | 3 -
.../ethernet/mellanox/mlx5/core/en/params.c | 16 +-
.../ethernet/mellanox/mlx5/core/en/params.h | 1 -
.../mellanox/mlx5/core/en/reporter_rx.c | 119 ++-
.../mellanox/mlx5/core/en/reporter_tx.c | 1 -
.../net/ethernet/mellanox/mlx5/core/en/rss.c | 15 +
.../net/ethernet/mellanox/mlx5/core/en/rss.h | 3 +
.../ethernet/mellanox/mlx5/core/en/rx_res.c | 9 +-
.../ethernet/mellanox/mlx5/core/en/rx_res.h | 5 +
.../net/ethernet/mellanox/mlx5/core/en/xdp.c | 49 +-
.../net/ethernet/mellanox/mlx5/core/en_main.c | 29 -
.../net/ethernet/mellanox/mlx5/core/en_rep.c | 5 +
.../ethernet/mellanox/mlx5/core/en_selftest.c | 3 +
.../net/ethernet/mellanox/mlx5/core/esw/qos.c | 815 +++++++++++++++++-
.../net/ethernet/mellanox/mlx5/core/esw/qos.h | 4 +
.../net/ethernet/mellanox/mlx5/core/eswitch.h | 13 +-
.../mellanox/mlx5/core/eswitch_offloads.c | 29 +-
.../ethernet/mellanox/mlx5/core/lib/fs_ttc.c | 19 +
.../ethernet/mellanox/mlx5/core/lib/fs_ttc.h | 1 +
.../ethernet/mellanox/mlx5/core/mlx5_core.h | 2 +
.../net/ethernet/mellanox/mlx5/core/vport.c | 25 +
include/net/devlink.h | 9 +
include/uapi/linux/devlink.h | 4 +
net/devlink/netlink_gen.c | 16 +-
net/devlink/netlink_gen.h | 2 +
net/devlink/rate.c | 127 +++
31 files changed, 1274 insertions(+), 145 deletions(-)
base-commit: acdefab0dcbc3833b5a734ab80d792bb778517a0
--
2.45.0
Powered by blists - more mailing lists