ethtool: track custom RSS contexts in the core
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v8 Latest9 patches 1 replyIn review 8 of 9 applied
- v79 patches 0 repliesSuperseded
- v69 patches 10 repliesSuperseded
Earlier versions · 5
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v5 | 7 patches | 13 replies | Superseded | |
| v4 | 7 patches | 27 replies | Superseded | |
| v3 RFC | 7 patches | 8 replies | Superseded | |
| v2 RFC | 7 patches | 16 replies | Superseded | |
| v1 RFC | 6 patches | 15 replies | Superseded |
Changes in v8 from the cover letter
* use struct_size_t in patch 3 (Przemek)
Patches in v8 8 of 9 applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/9 | net: move ethtool-related netdev state into its own structdrivers/ drivers/ drivers/ +8 more | +28 | 0 replies | |
| 2/9 | net: ethtool: attach an XArray of custom RSS contexts to a netdeviceinclude/ net/ | +69 | 0 replies | Applied |
| 3/9 | net: ethtool: record custom RSS contexts in the XArrayinclude/ net/ | +87 | 0 replies | Applied |
| 4/9 | net: ethtool: let the core choose RSS context IDsinclude/ net/ net/ | +79 | 0 replies | Applied |
| 5/9 | net: ethtool: add an extack parameter to new rxfh_context APIsinclude/ net/ net/ | +13 | 0 replies | Applied |
| 6/9 | net: ethtool: add a mutex protecting RSS contextsinclude/ net/ net/ | +15 | 0 replies | Applied |
| 7/9 | sfc: use new rxfh_context APIdrivers/ drivers/ drivers/ +10 more | +212 | 0 replies | Applied |
| 8/9 | net: ethtool: use the tracking array for get_rxfh on custom RSS contextsnet/ | +21 | 0 replies | Applied |
| 9/9 | sfc: remove get_rxfh_context dead codedrivers/ | +2 | 0 replies | Applied |
Reviews trailers per patch
None of the 9 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.
| Patch | Przemek Kitszel |
|---|---|
| Series | R-b |
| 1/9 | |
| 2/9 | |
| 3/9 | |
| 4/9 | |
| 5/9 | |
| 6/9 | |
| 7/9 | |
| 8/9 | |
| 9/9 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v8 net-next 0/9] ethtool: track custom RSS contexts in the core
From: Edward Cree <[email protected]>
Make the core responsible for tracking the set of custom RSS contexts,
their IDs, indirection tables, hash keys, and hash functions; this lets us get rid of duplicative code in drivers, and will allow us to support netlink dumps later.
This series only moves the sfc EF10 & EF100 driver over to the new API;
other drivers (mvpp2, octeontx2, mlx5, sfc/siena, bnxt_en) can be converted afterwards and the legacy API removed.
Changes in v8:
* use struct_size_t in patch 3 (Przemek)
Changes in v7:
* ensure 'ret' is initialised in ethtool_get_rxfh (horms)
Changes in v6:
* fixed kdoc for renamed fields
* always call setter in netdev_rss_contexts_free()
* document that 'create' method should populate ctx for driver-chosen defaults
* on 'ethtool -x', get info from the tracking xarray rather than calling the driver's get_rxfh method. This makes it easier to test that the tracking is correct, in the absence of future code like netlink dumps to use it.
Changes in v5:
* Rebased on top of Ahmed Zaki's struct ethtool_rxfh_param API
* Moved rxfh_max_context_id to the ethtool ops struct
Changes in v4:
* replaced IDR with XArray
* grouped initialisations together in patch 6
* dropped RFC tags
Changes in v3:
* Added WangXun ngbe to patch #1, not sure if they've added WoL support since v2 or if I just missed it last time around
* Re-ordered struct ethtool_netdev_state to avoid hole (Andrew Lunn)
* Fixed some resource leaks in error handling paths (kuba)
* Added maintainers of other context-using drivers to CC
Edward Cree (9):
net: move ethtool-related netdev state into its own struct
net: ethtool: attach an XArray of custom RSS contexts to a netdevice
net: ethtool: record custom RSS contexts in the XArray
net: ethtool: let the core choose RSS context IDs
net: ethtool: add an extack parameter to new rxfh_context APIs
net: ethtool: add a mutex protecting RSS contexts
sfc: use new rxfh_context API
net: ethtool: use the tracking array for get_rxfh on custom RSS
contexts
sfc: remove get_rxfh_context dead code
drivers/net/ethernet/realtek/r8169_main.c | 4 +- drivers/net/ethernet/sfc/ef10.c | 2 +- drivers/net/ethernet/sfc/ef100_ethtool.c | 4 + drivers/net/ethernet/sfc/efx.c | 2 +- drivers/net/ethernet/sfc/efx.h | 2 +- drivers/net/ethernet/sfc/efx_common.c | 10 +- drivers/net/ethernet/sfc/ethtool.c | 4 + drivers/net/ethernet/sfc/ethtool_common.c | 168 ++++++++---------- drivers/net/ethernet/sfc/ethtool_common.h | 12 ++ drivers/net/ethernet/sfc/mcdi_filters.c | 135 +++++++------- drivers/net/ethernet/sfc/mcdi_filters.h | 8 +- drivers/net/ethernet/sfc/net_driver.h | 28 +-- drivers/net/ethernet/sfc/rx_common.c | 64 ++----- drivers/net/ethernet/sfc/rx_common.h | 8 +- .../net/ethernet/wangxun/ngbe/ngbe_ethtool.c | 4 +- drivers/net/ethernet/wangxun/ngbe/ngbe_main.c | 2 +- drivers/net/phy/phy.c | 2 +- drivers/net/phy/phy_device.c | 5 +- drivers/net/phy/phylink.c | 2 +- include/linux/ethtool.h | 110 ++++++++++++ include/linux/netdevice.h | 7 +- net/core/dev.c | 40 +++++ net/ethtool/ioctl.c | 136 +++++++++++++- net/ethtool/wol.c | 2 +- 24 files changed, 496 insertions(+), 265 deletions(-)