Change igb_set_rxfh() to accept and save a userspace-provided
RSS key. When a key is provided, store it in the adapter and write the
E1000 registers accordingly.

This can be tested using `ethtool -X <dev> hkey <key>`.

Reviewed-by: Simon Horman <[email protected]>
Signed-off-by: Takashi Kozu <[email protected]>
---
 drivers/net/ethernet/intel/igb/igb_ethtool.c | 48 +++++++++++---------
 1 file changed, 26 insertions(+), 22 deletions(-)

diff --git a/drivers/net/ethernet/intel/igb/igb_ethtool.c 
b/drivers/net/ethernet/intel/igb/igb_ethtool.c
index b387121f0ea7..b4eef14c09d5 100644
--- a/drivers/net/ethernet/intel/igb/igb_ethtool.c
+++ b/drivers/net/ethernet/intel/igb/igb_ethtool.c
@@ -3357,35 +3357,39 @@ static int igb_set_rxfh(struct net_device *netdev,
        u32 num_queues;
 
        /* We do not allow change in unsupported parameters */
-       if (rxfh->key ||
-           (rxfh->hfunc != ETH_RSS_HASH_NO_CHANGE &&
-            rxfh->hfunc != ETH_RSS_HASH_TOP))
+       if (rxfh->hfunc != ETH_RSS_HASH_NO_CHANGE &&
+           rxfh->hfunc != ETH_RSS_HASH_TOP)
                return -EOPNOTSUPP;
-       if (!rxfh->indir)
-               return 0;
 
-       num_queues = adapter->rss_queues;
+       if (rxfh->indir) {
+               num_queues = adapter->rss_queues;
 
-       switch (hw->mac.type) {
-       case e1000_82576:
-               /* 82576 supports 2 RSS queues for SR-IOV */
-               if (adapter->vfs_allocated_count)
-                       num_queues = 2;
-               break;
-       default:
-               break;
-       }
+               switch (hw->mac.type) {
+               case e1000_82576:
+                       /* 82576 supports 2 RSS queues for SR-IOV */
+                       if (adapter->vfs_allocated_count)
+                               num_queues = 2;
+                       break;
+               default:
+                       break;
+               }
 
-       /* Verify user input. */
-       for (i = 0; i < IGB_RETA_SIZE; i++)
-               if (rxfh->indir[i] >= num_queues)
-                       return -EINVAL;
+               /* Verify user input. */
+               for (i = 0; i < IGB_RETA_SIZE; i++)
+                       if (rxfh->indir[i] >= num_queues)
+                               return -EINVAL;
 
 
-       for (i = 0; i < IGB_RETA_SIZE; i++)
-               adapter->rss_indir_tbl[i] = rxfh->indir[i];
+               for (i = 0; i < IGB_RETA_SIZE; i++)
+                       adapter->rss_indir_tbl[i] = rxfh->indir[i];
+
+               igb_write_rss_indir_tbl(adapter);
+       }
 
-       igb_write_rss_indir_tbl(adapter);
+       if (rxfh->key) {
+               memcpy(adapter->rss_key, rxfh->key, sizeof(adapter->rss_key));
+               igb_write_rss_key(adapter);
+       }
 
        return 0;
 }
-- 
2.52.0

Reply via email to