Skip to content

Workarounds and fixes for ntuple filters

Alexander Duyck netdev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest10 patches 13 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10ethtool: prevent null pointer dereference with NTUPLE set but no set_rx_ntuplenet/core/ethtool.c+3 -04 replies
2/10ethtool: add ntuple flow specifier to network flow classifierinclude/linux/ethtool.h+20 -09 replies
3/10ixgbe: remove ntuple filteringdrivers/net/ixgbe/ixgbe_dcb_nl.c drivers/net/ixgbe/ixgbe_ethtool.c drivers/net/ixgbe/ixgbe_main.c+48 -2060 replies
4/10ethtool: remove support for ETHTOOL_GRXNTUPLEinclude/linux/ethtool.h include/linux/netdevice.h net/core/dev.c +1 more+1 -3130 replies
5/10ixgbe: add support for different Rx packet buffer sizesdrivers/net/ixgbe/ixgbe_82598.c drivers/net/ixgbe/ixgbe_82599.c drivers/net/ixgbe/ixgbe_type.h +1 more+7 -00 replies
6/10ixgbe: update perfect filter framework to support retaining filtersdrivers/net/ixgbe/ixgbe.h drivers/net/ixgbe/ixgbe_82599.c drivers/net/ixgbe/ixgbe_main.c +1 more+376 -3270 replies
7/10ixgbe: add basic support for settting and getting nfc controlsdrivers/net/ixgbe/ixgbe_ethtool.c+60 -00 replies
8/10ixgbe: add support for displaying ntuple filters via the nfc interfacedrivers/net/ixgbe/ixgbe.h drivers/net/ixgbe/ixgbe_ethtool.c+106 -00 replies
9/10ixgbe: add support for nfc addition and removal of filtersdrivers/net/ixgbe/ixgbe_ethtool.c drivers/net/ixgbe/ixgbe_main.c+277 -00 replies
10/10ixgbe: Add support for using the same fields as ntuple in nfcdrivers/net/ixgbe/ixgbe_ethtool.c+19 -00 replies

Reviews trailers per patch

1 of 10 patches has a Reviewed-by or Acked-by.

PatchAlexander Duyck
1/10
2/10A-b
3/10
4/10
5/10
6/10
7/10
8/10
9/10
10/10

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [net-next-2.6 PATCH 00/10] Workarounds and fixes for ntuple filters

This patch series is meant to address issue in the implementation of ntuple filters in both ixgbe and the kernel. The main issue being that ntuple filters didn't support the ability for drivers to retain and display filters.

This series addresses it by removing the ETHTOOL_GRXNTUPLE interface from the kernel and extends the ethtool network flow classifier interface to handle the same fields that ntuple filters provide. I have a separate set of patches I will be emailing out in the next few minutes that will contain the ethtool user-space portion of these changes.

The patches labeled with RFC are currently just for comments as I am not submitting them directly to the netdev tree and will be instead submitting them to the Jeff Kirsher's Intel network driver tree for submission.

---

Alexander Duyck (10):
[RFC] ixgbe: Add support for using the same fields as ntuple in nfc [RFC] ixgbe: add support for nfc addition and removal of filters [RFC] ixgbe: add support for displaying ntuple filters via the nfc interface [RFC] ixgbe: add basic support for settting and getting nfc controls [RFC] ixgbe: update perfect filter framework to support retaining filters [RFC] ixgbe: add support for different Rx packet buffer sizes [RFC] ethtool: remove support for ETHTOOL_GRXNTUPLE
[RFC] ixgbe: remove ntuple filtering
ethtool: add ntuple flow specifier to network flow classifier
ethtool: prevent null pointer dereference with NTUPLE set but no set_rx_ntuple

 drivers/net/ixgbe/ixgbe.h         |   29 +-
 drivers/net/ixgbe/ixgbe_82598.c   |    2 
 drivers/net/ixgbe/ixgbe_82599.c   |  661 ++++++++++++++++++++-----------------
 drivers/net/ixgbe/ixgbe_dcb_nl.c  |   47 +--
 drivers/net/ixgbe/ixgbe_ethtool.c |  504 ++++++++++++++++++++++------
 drivers/net/ixgbe/ixgbe_main.c    |   90 +++--
 drivers/net/ixgbe/ixgbe_type.h    |   25 +
 drivers/net/ixgbe/ixgbe_x540.c    |    2 
 include/linux/ethtool.h           |   27 +-
 include/linux/netdevice.h         |    3 
 net/core/dev.c                    |    5 
 net/core/ethtool.c                |  302 -----------------
 12 files changed, 884 insertions(+), 813 deletions(-)