Merge branch 'ethtool-rssh-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/bwh/net-next

Ben Hutchings says:

====================
Pull request: Fixes for new ethtool RSS commands

This addresses several problems I previously identified with the new
ETHTOOL_{G,S}RSSH commands:

1. Missing validation of reserved parameters
2. Vague documentation
3. Use of unnamed magic number
4. No consolidation with existing driver operations

I don't currently have access to suitable network hardware, but have
tested these changes with a dummy driver that can support various
combinations of operations and sizes, together with (a) Debian's ethtool
3.13 (b) ethtool 3.14 with the submitted patch to use ETHTOOL_{G,S}RSSH
and minor adjustment for fixes 1 and 3.

v2: Update RSS operations in vmxnet3 too
====================

Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 8 insertions, 7 deletions

diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c
index 5203a8924edf..bd0600cf7266 100644
--- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c
+++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c
@@ -3316,7 +3316,7 @@ static u32 bnx2x_get_rxfh_indir_size(struct net_device *dev)
         return T_ETH_INDIRECTION_TABLE_SIZE;
 }
 
-static int bnx2x_get_rxfh_indir(struct net_device *dev, u32 *indir)
+static int bnx2x_get_rxfh(struct net_device *dev, u32 *indir, u8 *key)
 {
         struct bnx2x *bp = netdev_priv(dev);
         u8 ind_table[T_ETH_INDIRECTION_TABLE_SIZE] = {0};
@@ -3340,14 +3340,15 @@ static int bnx2x_get_rxfh_indir(struct net_device *dev, u32 *indir)
         return 0;
 }
 
-static int bnx2x_set_rxfh_indir(struct net_device *dev, const u32 *indir)
+static int bnx2x_set_rxfh(struct net_device *dev, const u32 *indir,
+                          const u8 *key)
 {
         struct bnx2x *bp = netdev_priv(dev);
         size_t i;
 
         for (i = 0; i < T_ETH_INDIRECTION_TABLE_SIZE; i++) {
                 /*
-                 * The same as in bnx2x_get_rxfh_indir: we can't use a memcpy()
+                 * The same as in bnx2x_get_rxfh: we can't use a memcpy()
                  * as an internal storage of an indirection table is a u8 array
                  * while indir->ring_index points to an array of u32.
                  *
@@ -3471,8 +3472,8 @@ static const struct ethtool_ops bnx2x_ethtool_ops = {
         .get_rxnfc              = bnx2x_get_rxnfc,
         .set_rxnfc              = bnx2x_set_rxnfc,
         .get_rxfh_indir_size    = bnx2x_get_rxfh_indir_size,
-        .get_rxfh_indir         = bnx2x_get_rxfh_indir,
-        .set_rxfh_indir         = bnx2x_set_rxfh_indir,
+        .get_rxfh               = bnx2x_get_rxfh,
+        .set_rxfh               = bnx2x_set_rxfh,
         .get_channels           = bnx2x_get_channels,
         .set_channels           = bnx2x_set_channels,
         .get_module_info        = bnx2x_get_module_info,
@@ -3498,8 +3499,8 @@ static const struct ethtool_ops bnx2x_vf_ethtool_ops = {
         .get_rxnfc              = bnx2x_get_rxnfc,
         .set_rxnfc              = bnx2x_set_rxnfc,
         .get_rxfh_indir_size    = bnx2x_get_rxfh_indir_size,
-        .get_rxfh_indir         = bnx2x_get_rxfh_indir,
-        .set_rxfh_indir         = bnx2x_set_rxfh_indir,
+        .get_rxfh               = bnx2x_get_rxfh,
+        .set_rxfh               = bnx2x_set_rxfh,
         .get_channels           = bnx2x_get_channels,
         .set_channels           = bnx2x_set_channels,
 };