diff options
author | Alexander Duyck <alexander.h.duyck@intel.com> | 2011-06-08 08:35:08 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-06-08 19:45:31 -0400 |
commit | bff55273f98dea0ceb78e28eb69462fe5f72ef3d (patch) | |
tree | cb62da1fe759ed2f777281d14af6b683eb4d5d44 /include/linux/netdevice.h | |
parent | c0c33addcba2ce753b4e2746db99feaae2f82a85 (diff) |
v2 ethtool: remove support for ETHTOOL_GRXNTUPLE
This change is meant to remove all support for displaying an ntuple as
strings via ETHTOOL_GRXNTUPLE. The reason for this change is due to the
fact that multiple issues have been found including:
- Multiple buffer overruns for strings being displayed.
- Incorrect filters displayed, cleared filters with ring of -2 are displayed
- Setting get_rx_ntuple displays no rules if defined.
- Endianess wrong on displayed values.
- Hard limit of 1024 filters makes display functionality extremely limited
The only driver that had supported this interface was ixgbe. Since it no
longer uses the interface and due to the issues mentioned above I am
submitting this patch to remove it.
v2:
Updated based on comments from Ben Hutchings
- Left ETH_SS_NTUPLE_FILTERS in code but commented on it being deprecated
- Removed ethtool_rx_ntuple_list and ethtool_rx_ntuple_flow_spec_container
- Left ETHTOOL_GRXNTUPLE but commented it as deprecated
Also cleaned up set_rx_ntuple since there is no flow spec container to
maintain we can drop all the code for the alloc and free of it and just
return ops->set_rx_ntuple().
Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Acked-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/netdevice.h')
-rw-r--r-- | include/linux/netdevice.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index 336288567c78..6469fa942d1d 100644 --- a/include/linux/netdevice.h +++ b/include/linux/netdevice.h | |||
@@ -1348,9 +1348,6 @@ struct net_device { | |||
1348 | /* max exchange id for FCoE LRO by ddp */ | 1348 | /* max exchange id for FCoE LRO by ddp */ |
1349 | unsigned int fcoe_ddp_xid; | 1349 | unsigned int fcoe_ddp_xid; |
1350 | #endif | 1350 | #endif |
1351 | /* n-tuple filter list attached to this device */ | ||
1352 | struct ethtool_rx_ntuple_list ethtool_ntuple_list; | ||
1353 | |||
1354 | /* phy device may attach itself for hardware timestamping */ | 1351 | /* phy device may attach itself for hardware timestamping */ |
1355 | struct phy_device *phydev; | 1352 | struct phy_device *phydev; |
1356 | 1353 | ||