From: Paolo Abeni <pabeni@redhat.com>
To: netdev@vger.kernel.org
Cc: Jakub Kicinski <kuba@kernel.org>, Jiri Pirko <jiri@resnulli.us>,
Madhu Chittim <madhu.chittim@intel.com>,
Sridhar Samudrala <sridhar.samudrala@intel.com>,
Simon Horman <horms@kernel.org>,
John Fastabend <john.fastabend@gmail.com>,
Sunil Kovvuri Goutham <sgoutham@marvell.com>,
Jamal Hadi Salim <jhs@mojatatu.com>
Subject: [PATCH RFC v2 00/11] net: introduce TX shaping H/W offload API
Date: Wed, 24 Jul 2024 22:24:46 +0200 [thread overview]
Message-ID: <cover.1721851988.git.pabeni@redhat.com> (raw)
We have a plurality of shaping-related drivers API, but none flexible
enough to meet existing demand from vendors[1].
This series introduces new device APIs to configure in a flexible way
TX shaping H/W offload. The new functionality is exposed via a newly
defined generic netlink interface and includes introspection
capabilities. Some self-tests are included, on top of a dummy
netdevsim implementation, and a basic implementation for the iavf
driver.
Some usage examples:
* Configure shaping on a given queue:
./tools/net/ynl/cli.py --spec Documentation/netlink/specs/shaper.yaml \
--do set --json '{"ifindex":'$IFINDEX',
"shaper": {"handle":
{"scope": "queue", "id":'$QUEUEID' },
"bw-max": 2000000 }}'
* Container B/W sharing
The orchestration infrastructure wants to group the
container-related queues under a RR scheduling and limit the aggregate
bandwidth:
./tools/net/ynl/cli.py --spec Documentation/netlink/specs/shaper.yaml \
--do group --json '{"ifindex":'$IFINDEX',
"inputs": [
{"handle": {"scope": "queue", "id":'$QID1' },
"weight": '$W1'},
{"handle": {"scope": "queue", "id":'$QID2' },
"weight": '$W2'}],
{"handle": {"scope": "queue", "id":'$QID3' },
"weight": '$W3'}],
"output": { "handle": {"scope":"netdev"},
"output": { "handle": {"scope":"netdev"},
"bw-max": 10000000}}'
{'handle': {'id': 0, 'scope': 'netdev'}}
* Delegation
A container wants to set a B/W limit on 2 of its own queues:
./tools/net/ynl/cli.py --spec Documentation/netlink/specs/shaper.yaml \
--do group --json '{"ifindex":'$IFINDEX',
"inputs": [
{"handle": {"scope": "queue", "id":'$QID1' },
"weight": '$W1'},
{"handle": {"scope": "queue", "id":'$QID2' },
"weight": '$W2'}],
"output": { "handle": {"scope":"detached"},
"bw-max": 5000000}}'
{'handle': {'id': 0, 'scope': 'detached'}}
* Cleanup:
Deleting a single queue shaper:
./tools/net/ynl/cli.py --spec Documentation/netlink/specs/shaper.yaml \
--do delete --json \
'{"ifindex":'$IFINDEX',
"handle": {"scope": "queue", "id":'$QID1' }}'
Deleting the last shaper under a group deletes the group, too:
./tools/net/ynl/cli.py --spec Documentation/netlink/specs/shaper.yaml \
--do delete --json \
'{"ifindex":'$IFINDEX',
"handle": {"scope": "queue", "id":'$QID2' }}'
./tools/net/ynl/cli.py --spec Documentation/netlink/specs/shaper.yaml \
--do get --json '{"ifindex":'$IF',
"handle": { "scope": "detached", "id": 0}}'
Netlink error: Invalid argument
nl_len = 80 (64) nl_flags = 0x300 nl_type = 2
error: -22
extack: {'msg': "Can't find shaper for handle 10000000"}
Changes from RFC v1:
- set() and delete() ops operate on a single shaper
- added group() op to allow grouping and nesting
- split the NL implementation into multiple patches to help reviewing
RFC v1: https://lore.kernel.org/netdev/cover.1719518113.git.pabeni@redhat.com/
[1] https://lore.kernel.org/netdev/20240405102313.GA310894@kernel.org/
Paolo Abeni (7):
netlink: spec: add shaper YAML spec
net-shapers: implement NL get operation
net-shapers: implement NL set and delete operations
net-shapers: implement NL group operation
netlink: spec: add shaper introspection support
net: shaper: implement introspection support
testing: net-drv: add basic shaper test
Sudheer Mogilappagari (2):
iavf: Add net_shaper_ops support
iavf: add support to exchange qos capabilities
Wenjun Wu (2):
virtchnl: support queue rate limit and quanta size configuration
ice: Support VF queue rate limit and quanta size configuration
Documentation/netlink/specs/shaper.yaml | 337 ++++++
Documentation/networking/kapi.rst | 3 +
MAINTAINERS | 1 +
drivers/net/Kconfig | 1 +
drivers/net/ethernet/intel/Kconfig | 1 +
drivers/net/ethernet/intel/iavf/iavf.h | 13 +
drivers/net/ethernet/intel/iavf/iavf_main.c | 215 +++-
drivers/net/ethernet/intel/iavf/iavf_txrx.h | 2 +
.../net/ethernet/intel/iavf/iavf_virtchnl.c | 157 ++-
drivers/net/ethernet/intel/ice/ice.h | 2 +
drivers/net/ethernet/intel/ice/ice_base.c | 2 +
drivers/net/ethernet/intel/ice/ice_common.c | 21 +
.../net/ethernet/intel/ice/ice_hw_autogen.h | 8 +
drivers/net/ethernet/intel/ice/ice_txrx.h | 1 +
drivers/net/ethernet/intel/ice/ice_type.h | 1 +
drivers/net/ethernet/intel/ice/ice_vf_lib.h | 8 +
drivers/net/ethernet/intel/ice/ice_virtchnl.c | 333 ++++++
drivers/net/ethernet/intel/ice/ice_virtchnl.h | 11 +
.../intel/ice/ice_virtchnl_allowlist.c | 6 +
drivers/net/netdevsim/netdev.c | 37 +
include/linux/avf/virtchnl.h | 119 +++
include/linux/netdevice.h | 17 +
include/net/net_shaper.h | 169 +++
include/uapi/linux/net_shaper.h | 91 ++
net/Kconfig | 3 +
net/Makefile | 1 +
net/core/dev.c | 2 +
net/core/dev.h | 6 +
net/shaper/Makefile | 9 +
net/shaper/shaper.c | 962 ++++++++++++++++++
net/shaper/shaper_nl_gen.c | 142 +++
net/shaper/shaper_nl_gen.h | 30 +
tools/testing/selftests/drivers/net/Makefile | 1 +
tools/testing/selftests/drivers/net/shaper.py | 267 +++++
.../testing/selftests/net/lib/py/__init__.py | 1 +
tools/testing/selftests/net/lib/py/ynl.py | 5 +
36 files changed, 2983 insertions(+), 2 deletions(-)
create mode 100644 Documentation/netlink/specs/shaper.yaml
create mode 100644 include/net/net_shaper.h
create mode 100644 include/uapi/linux/net_shaper.h
create mode 100644 net/shaper/Makefile
create mode 100644 net/shaper/shaper.c
create mode 100644 net/shaper/shaper_nl_gen.c
create mode 100644 net/shaper/shaper_nl_gen.h
create mode 100755 tools/testing/selftests/drivers/net/shaper.py
--
2.45.2
next reply other threads:[~2024-07-24 20:25 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-24 20:24 Paolo Abeni [this message]
2024-07-24 20:24 ` [PATCH RFC v2 01/11] netlink: spec: add shaper YAML spec Paolo Abeni
2024-07-25 11:42 ` Paolo Abeni
2024-07-24 20:24 ` [PATCH RFC v2 02/11] net-shapers: implement NL get operation Paolo Abeni
2024-07-24 20:24 ` [PATCH RFC v2 03/11] net-shapers: implement NL set and delete operations Paolo Abeni
2024-08-02 16:01 ` Jiri Pirko
2024-07-24 20:24 ` [PATCH RFC v2 04/11] net-shapers: implement NL group operation Paolo Abeni
2024-07-24 20:24 ` [PATCH RFC v2 05/11] netlink: spec: add shaper introspection support Paolo Abeni
2024-07-24 20:24 ` [PATCH RFC v2 06/11] net: shaper: implement " Paolo Abeni
2024-07-24 20:24 ` [PATCH RFC v2 07/11] testing: net-drv: add basic shaper test Paolo Abeni
2024-07-24 20:24 ` [PATCH RFC v2 08/11] virtchnl: support queue rate limit and quanta size configuration Paolo Abeni
2024-07-24 20:24 ` [PATCH RFC v2 09/11] ice: Support VF " Paolo Abeni
2024-07-24 20:24 ` [PATCH RFC v2 10/11] iavf: Add net_shaper_ops support Paolo Abeni
2024-08-02 13:44 ` Jiri Pirko
2024-07-24 20:24 ` [PATCH RFC v2 11/11] iavf: add support to exchange qos capabilities Paolo Abeni
2024-07-29 6:30 ` [PATCH RFC v2 00/11] net: introduce TX shaping H/W offload API Jiri Pirko
2024-07-29 14:42 ` Paolo Abeni
2024-07-29 15:13 ` Jiri Pirko
2024-07-30 16:01 ` Paolo Abeni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1721851988.git.pabeni@redhat.com \
--to=pabeni@redhat.com \
--cc=horms@kernel.org \
--cc=jhs@mojatatu.com \
--cc=jiri@resnulli.us \
--cc=john.fastabend@gmail.com \
--cc=kuba@kernel.org \
--cc=madhu.chittim@intel.com \
--cc=netdev@vger.kernel.org \
--cc=sgoutham@marvell.com \
--cc=sridhar.samudrala@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.