net: ethtool: get rid of get/set_rxfh_context functions
authorAhmed Zaki <ahmed.zaki@intel.com>
Wed, 13 Dec 2023 00:33:15 +0000 (17:33 -0700)
committerJakub Kicinski <kuba@kernel.org>
Thu, 14 Dec 2023 06:07:16 +0000 (22:07 -0800)
Add the RSS context parameters to struct ethtool_rxfh_param and use the
get/set_rxfh to handle the RSS contexts as well.

This is part 2/2 of the fix suggested in [1]:

 - Add a rss_context member to the argument struct and a capability
   like cap_link_lanes_supported to indicate whether driver supports
   rss contexts, then you can remove *et_rxfh_context functions,
   and instead call *et_rxfh() with a non-zero rss_context.

Link: https://lore.kernel.org/netdev/20231121152906.2dd5f487@kernel.org/
CC: Jesse Brandeburg <jesse.brandeburg@intel.com>
CC: Tony Nguyen <anthony.l.nguyen@intel.com>
CC: Marcin Wojtas <mw@semihalf.com>
CC: Russell King <linux@armlinux.org.uk>
CC: Sunil Goutham <sgoutham@marvell.com>
CC: Geetha sowjanya <gakula@marvell.com>
CC: Subbaraya Sundeep <sbhatta@marvell.com>
CC: hariprasad <hkelam@marvell.com>
CC: Saeed Mahameed <saeedm@nvidia.com>
CC: Leon Romanovsky <leon@kernel.org>
CC: Edward Cree <ecree.xilinx@gmail.com>
CC: Martin Habets <habetsm.xilinx@gmail.com>
Suggested-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Ahmed Zaki <ahmed.zaki@intel.com>
Link: https://lore.kernel.org/r/20231213003321.605376-3-ahmed.zaki@intel.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
14 files changed:
drivers/net/ethernet/intel/ice/ice_ethtool.c
drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c
drivers/net/ethernet/marvell/octeontx2/nic/otx2_ethtool.c
drivers/net/ethernet/mellanox/mlx5/core/en_ethtool.c
drivers/net/ethernet/sfc/ef100_ethtool.c
drivers/net/ethernet/sfc/ethtool.c
drivers/net/ethernet/sfc/ethtool_common.c
drivers/net/ethernet/sfc/ethtool_common.h
drivers/net/ethernet/sfc/siena/ethtool.c
drivers/net/ethernet/sfc/siena/ethtool_common.c
drivers/net/ethernet/sfc/siena/ethtool_common.h
include/linux/ethtool.h
net/ethtool/ioctl.c
net/ethtool/rss.c

index 70cbd8092aeae553b0e026bbc730ca610c1dae08..c77605b428338af348736c77ca40520d4b355cf5 100644 (file)
@@ -3195,11 +3195,18 @@ static u32 ice_get_rxfh_indir_size(struct net_device *netdev)
        return np->vsi->rss_table_size;
 }
 
+/**
+ * ice_get_rxfh - get the Rx flow hash indirection table
+ * @netdev: network interface device structure
+ * @rxfh: pointer to param struct (indir, key, hfunc)
+ *
+ * Reads the indirection table directly from the hardware.
+ */
 static int
-ice_get_rxfh_context(struct net_device *netdev,
-                    struct ethtool_rxfh_param *rxfh, u32 rss_context)
+ice_get_rxfh(struct net_device *netdev, struct ethtool_rxfh_param *rxfh)
 {
        struct ice_netdev_priv *np = netdev_priv(netdev);
+       u32 rss_context = rxfh->rss_context;
        struct ice_vsi *vsi = np->vsi;
        struct ice_pf *pf = vsi->back;
        u16 qcount, offset;
@@ -3261,19 +3268,6 @@ out:
        return err;
 }
 
-/**
- * ice_get_rxfh - get the Rx flow hash indirection table
- * @netdev: network interface device structure
- * @rxfh: pointer to param struct (indir, key, hfunc)
- *
- * Reads the indirection table directly from the hardware.
- */
-static int
-ice_get_rxfh(struct net_device *netdev, struct ethtool_rxfh_param *rxfh)
-{
-       return ice_get_rxfh_context(netdev, rxfh, 0);
-}
-
 /**
  * ice_set_rxfh - set the Rx flow hash indirection table
  * @netdev: network interface device structure
@@ -3298,6 +3292,9 @@ ice_set_rxfh(struct net_device *netdev, struct ethtool_rxfh_param *rxfh,
            rxfh->hfunc != ETH_RSS_HASH_TOP)
                return -EOPNOTSUPP;
 
+       if (rxfh->rss_context)
+               return -EOPNOTSUPP;
+
        if (!test_bit(ICE_FLAG_RSS_ENA, pf->flags)) {
                /* RSS not supported return error here */
                netdev_warn(netdev, "RSS is not configured on this VSI!\n");
@@ -4215,6 +4212,7 @@ ice_get_module_eeprom(struct net_device *netdev,
 }
 
 static const struct ethtool_ops ice_ethtool_ops = {
+       .cap_rss_ctx_supported  = true,
        .supported_coalesce_params = ETHTOOL_COALESCE_USECS |
                                     ETHTOOL_COALESCE_USE_ADAPTIVE |
                                     ETHTOOL_COALESCE_RX_USECS_HIGH,
@@ -4248,7 +4246,6 @@ static const struct ethtool_ops ice_ethtool_ops = {
        .set_pauseparam         = ice_set_pauseparam,
        .get_rxfh_key_size      = ice_get_rxfh_key_size,
        .get_rxfh_indir_size    = ice_get_rxfh_indir_size,
-       .get_rxfh_context       = ice_get_rxfh_context,
        .get_rxfh               = ice_get_rxfh,
        .set_rxfh               = ice_set_rxfh,
        .get_channels           = ice_get_channels,
index ceef48ddd26ef78a58b909c02afc9a77be84370f..9193cf61de26393eb6551484d3592206218cbfa1 100644 (file)
@@ -5638,47 +5638,7 @@ static int mvpp2_ethtool_get_rxfh(struct net_device *dev,
                                  struct ethtool_rxfh_param *rxfh)
 {
        struct mvpp2_port *port = netdev_priv(dev);
-       int ret = 0;
-
-       if (!mvpp22_rss_is_supported(port))
-               return -EOPNOTSUPP;
-
-       if (rxfh->indir)
-               ret = mvpp22_port_rss_ctx_indir_get(port, 0, rxfh->indir);
-
-       rxfh->hfunc = ETH_RSS_HASH_CRC32;
-
-       return ret;
-}
-
-static int mvpp2_ethtool_set_rxfh(struct net_device *dev,
-                                 struct ethtool_rxfh_param *rxfh,
-                                 struct netlink_ext_ack *extack)
-{
-       struct mvpp2_port *port = netdev_priv(dev);
-       int ret = 0;
-
-       if (!mvpp22_rss_is_supported(port))
-               return -EOPNOTSUPP;
-
-       if (rxfh->hfunc != ETH_RSS_HASH_NO_CHANGE &&
-           rxfh->hfunc != ETH_RSS_HASH_CRC32)
-               return -EOPNOTSUPP;
-
-       if (rxfh->key)
-               return -EOPNOTSUPP;
-
-       if (rxfh->indir)
-               ret = mvpp22_port_rss_ctx_indir_set(port, 0, rxfh->indir);
-
-       return ret;
-}
-
-static int mvpp2_ethtool_get_rxfh_context(struct net_device *dev,
-                                         struct ethtool_rxfh_param *rxfh,
-                                         u32 rss_context)
-{
-       struct mvpp2_port *port = netdev_priv(dev);
+       u32 rss_context = rxfh->rss_context;
        int ret = 0;
 
        if (!mvpp22_rss_is_supported(port))
@@ -5695,12 +5655,13 @@ static int mvpp2_ethtool_get_rxfh_context(struct net_device *dev,
        return ret;
 }
 
-static int mvpp2_ethtool_set_rxfh_context(struct net_device *dev,
-                                         struct ethtool_rxfh_param *rxfh,
-                                         u32 *rss_context, bool delete)
+static int mvpp2_ethtool_set_rxfh(struct net_device *dev,
+                                 struct ethtool_rxfh_param *rxfh,
+                                 struct netlink_ext_ack *extack)
 {
        struct mvpp2_port *port = netdev_priv(dev);
-       int ret;
+       u32 *rss_context = &rxfh->rss_context;
+       int ret = 0;
 
        if (!mvpp22_rss_is_supported(port))
                return -EOPNOTSUPP;
@@ -5712,7 +5673,7 @@ static int mvpp2_ethtool_set_rxfh_context(struct net_device *dev,
        if (rxfh->key)
                return -EOPNOTSUPP;
 
-       if (delete)
+       if (*rss_context && rxfh->rss_delete)
                return mvpp22_port_rss_ctx_delete(port, *rss_context);
 
        if (*rss_context == ETH_RXFH_CONTEXT_ALLOC) {
@@ -5721,8 +5682,13 @@ static int mvpp2_ethtool_set_rxfh_context(struct net_device *dev,
                        return ret;
        }
 
-       return mvpp22_port_rss_ctx_indir_set(port, *rss_context, rxfh->indir);
+       if (rxfh->indir)
+               ret = mvpp22_port_rss_ctx_indir_set(port, *rss_context,
+                                                   rxfh->indir);
+
+       return ret;
 }
+
 /* Device ops */
 
 static const struct net_device_ops mvpp2_netdev_ops = {
@@ -5742,6 +5708,7 @@ static const struct net_device_ops mvpp2_netdev_ops = {
 };
 
 static const struct ethtool_ops mvpp2_eth_tool_ops = {
+       .cap_rss_ctx_supported  = true,
        .supported_coalesce_params = ETHTOOL_COALESCE_USECS |
                                     ETHTOOL_COALESCE_MAX_FRAMES,
        .nway_reset             = mvpp2_ethtool_nway_reset,
@@ -5764,8 +5731,6 @@ static const struct ethtool_ops mvpp2_eth_tool_ops = {
        .get_rxfh_indir_size    = mvpp2_ethtool_get_rxfh_indir_size,
        .get_rxfh               = mvpp2_ethtool_get_rxfh,
        .set_rxfh               = mvpp2_ethtool_set_rxfh,
-       .get_rxfh_context       = mvpp2_ethtool_get_rxfh_context,
-       .set_rxfh_context       = mvpp2_ethtool_set_rxfh_context,
 };
 
 /* Used for PPv2.1, or PPv2.2 with the old Device Tree binding that
index a2c182d7bfc4118c817dbbccd8dcb6a977babf3d..2928898c7f8df89c45092c209f9a3dd25b43ee21 100644 (file)
@@ -835,11 +835,12 @@ static int otx2_rss_ctx_create(struct otx2_nic *pfvf,
        return 0;
 }
 
-/* RSS context configuration */
-static int otx2_set_rxfh_context(struct net_device *dev,
-                                struct ethtool_rxfh_param *rxfh,
-                                u32 *rss_context, bool delete)
+/* Configure RSS table and hash key */
+static int otx2_set_rxfh(struct net_device *dev,
+                        struct ethtool_rxfh_param *rxfh,
+                        struct netlink_ext_ack *extack)
 {
+       u32 rss_context = DEFAULT_RSS_CONTEXT_GROUP;
        struct otx2_nic *pfvf = netdev_priv(dev);
        struct otx2_rss_ctx *rss_ctx;
        struct otx2_rss_info *rss;
@@ -849,8 +850,11 @@ static int otx2_set_rxfh_context(struct net_device *dev,
            rxfh->hfunc != ETH_RSS_HASH_TOP)
                return -EOPNOTSUPP;
 
-       if (*rss_context != ETH_RXFH_CONTEXT_ALLOC &&
-           *rss_context >= MAX_RSS_GROUPS)
+       if (rxfh->rss_context)
+               rss_context = rxfh->rss_context;
+
+       if (rss_context != ETH_RXFH_CONTEXT_ALLOC &&
+           rss_context >= MAX_RSS_GROUPS)
                return -EINVAL;
 
        rss = &pfvf->hw.rss_info;
@@ -864,28 +868,30 @@ static int otx2_set_rxfh_context(struct net_device *dev,
                memcpy(rss->key, rxfh->key, sizeof(rss->key));
                otx2_set_rss_key(pfvf);
        }
-       if (delete)
-               return otx2_rss_ctx_delete(pfvf, *rss_context);
+       if (rxfh->rss_delete)
+               return otx2_rss_ctx_delete(pfvf, rss_context);
 
-       if (*rss_context == ETH_RXFH_CONTEXT_ALLOC) {
-               ret = otx2_rss_ctx_create(pfvf, rss_context);
+       if (rss_context == ETH_RXFH_CONTEXT_ALLOC) {
+               ret = otx2_rss_ctx_create(pfvf, &rss_context);
+               rxfh->rss_context = rss_context;
                if (ret)
                        return ret;
        }
        if (rxfh->indir) {
-               rss_ctx = rss->rss_ctx[*rss_context];
+               rss_ctx = rss->rss_ctx[rss_context];
                for (idx = 0; idx < rss->rss_size; idx++)
                        rss_ctx->ind_tbl[idx] = rxfh->indir[idx];
        }
-       otx2_set_rss_table(pfvf, *rss_context);
+       otx2_set_rss_table(pfvf, rss_context);
 
        return 0;
 }
 
-static int otx2_get_rxfh_context(struct net_device *dev,
-                                struct ethtool_rxfh_param *rxfh,
-                                u32 rss_context)
+/* Get RSS configuration */
+static int otx2_get_rxfh(struct net_device *dev,
+                        struct ethtool_rxfh_param *rxfh)
 {
+       u32 rss_context = DEFAULT_RSS_CONTEXT_GROUP;
        struct otx2_nic *pfvf = netdev_priv(dev);
        struct otx2_rss_ctx *rss_ctx;
        struct otx2_rss_info *rss;
@@ -895,6 +901,8 @@ static int otx2_get_rxfh_context(struct net_device *dev,
        rss = &pfvf->hw.rss_info;
 
        rxfh->hfunc = ETH_RSS_HASH_TOP;
+       if (rxfh->rss_context)
+               rss_context = rxfh->rss_context;
 
        if (!indir)
                return 0;
@@ -922,25 +930,6 @@ static int otx2_get_rxfh_context(struct net_device *dev,
        return 0;
 }
 
-/* Get RSS configuration */
-static int otx2_get_rxfh(struct net_device *dev,
-                        struct ethtool_rxfh_param *rxfh)
-{
-       return otx2_get_rxfh_context(dev, rxfh,
-                                    DEFAULT_RSS_CONTEXT_GROUP);
-}
-
-/* Configure RSS table and hash key */
-static int otx2_set_rxfh(struct net_device *dev,
-                        struct ethtool_rxfh_param *rxfh,
-                        struct netlink_ext_ack *extack)
-{
-
-       u32 rss_context = DEFAULT_RSS_CONTEXT_GROUP;
-
-       return otx2_set_rxfh_context(dev, rxfh, &rss_context, 0);
-}
-
 static u32 otx2_get_msglevel(struct net_device *netdev)
 {
        struct otx2_nic *pfvf = netdev_priv(netdev);
@@ -1321,6 +1310,7 @@ static void otx2_get_fec_stats(struct net_device *netdev,
 }
 
 static const struct ethtool_ops otx2_ethtool_ops = {
+       .cap_rss_ctx_supported  = true,
        .supported_coalesce_params = ETHTOOL_COALESCE_USECS |
                                     ETHTOOL_COALESCE_MAX_FRAMES |
                                     ETHTOOL_COALESCE_USE_ADAPTIVE,
@@ -1343,8 +1333,6 @@ static const struct ethtool_ops otx2_ethtool_ops = {
        .get_rxfh_indir_size    = otx2_get_rxfh_indir_size,
        .get_rxfh               = otx2_get_rxfh,
        .set_rxfh               = otx2_set_rxfh,
-       .get_rxfh_context       = otx2_get_rxfh_context,
-       .set_rxfh_context       = otx2_set_rxfh_context,
        .get_msglevel           = otx2_get_msglevel,
        .set_msglevel           = otx2_set_msglevel,
        .get_pauseparam         = otx2_get_pauseparam,
@@ -1444,6 +1432,7 @@ static int otx2vf_get_link_ksettings(struct net_device *netdev,
 }
 
 static const struct ethtool_ops otx2vf_ethtool_ops = {
+       .cap_rss_ctx_supported  = true,
        .supported_coalesce_params = ETHTOOL_COALESCE_USECS |
                                     ETHTOOL_COALESCE_MAX_FRAMES |
                                     ETHTOOL_COALESCE_USE_ADAPTIVE,
@@ -1462,8 +1451,6 @@ static const struct ethtool_ops otx2vf_ethtool_ops = {
        .get_rxfh_indir_size    = otx2_get_rxfh_indir_size,
        .get_rxfh               = otx2_get_rxfh,
        .set_rxfh               = otx2_set_rxfh,
-       .get_rxfh_context       = otx2_get_rxfh_context,
-       .set_rxfh_context       = otx2_set_rxfh_context,
        .get_ringparam          = otx2_get_ringparam,
        .set_ringparam          = otx2_set_ringparam,
        .get_coalesce           = otx2_get_coalesce,
index 110cb7973eaeb10e2fdeed95a04b243349578aab..0fe7ea88d56735b8c21b8cb2f35e111d60375ed2 100644 (file)
@@ -1262,11 +1262,10 @@ static u32 mlx5e_get_rxfh_indir_size(struct net_device *netdev)
        return mlx5e_ethtool_get_rxfh_indir_size(priv);
 }
 
-static int mlx5e_get_rxfh_context(struct net_device *dev,
-                                 struct ethtool_rxfh_param *rxfh,
-                                 u32 rss_context)
+int mlx5e_get_rxfh(struct net_device *netdev, struct ethtool_rxfh_param *rxfh)
 {
-       struct mlx5e_priv *priv = netdev_priv(dev);
+       struct mlx5e_priv *priv = netdev_priv(netdev);
+       u32 rss_context = rxfh->rss_context;
        int err;
 
        mutex_lock(&priv->state_lock);
@@ -1276,16 +1275,16 @@ static int mlx5e_get_rxfh_context(struct net_device *dev,
        return err;
 }
 
-static int mlx5e_set_rxfh_context(struct net_device *dev,
-                                 struct ethtool_rxfh_param *rxfh,
-                                 u32 *rss_context, bool delete)
+int mlx5e_set_rxfh(struct net_device *dev, struct ethtool_rxfh_param *rxfh,
+                  struct netlink_ext_ack *extack)
 {
        struct mlx5e_priv *priv = netdev_priv(dev);
+       u32 *rss_context = &rxfh->rss_context;
        u8 hfunc = rxfh->hfunc;
        int err;
 
        mutex_lock(&priv->state_lock);
-       if (delete) {
+       if (*rss_context && rxfh->rss_delete) {
                err = mlx5e_rx_res_rss_destroy(priv->rx_res, *rss_context);
                goto unlock;
        }
@@ -1307,25 +1306,6 @@ unlock:
        return err;
 }
 
-int mlx5e_get_rxfh(struct net_device *netdev, struct ethtool_rxfh_param *rxfh)
-{
-       return mlx5e_get_rxfh_context(netdev, rxfh, 0);
-}
-
-int mlx5e_set_rxfh(struct net_device *dev, struct ethtool_rxfh_param *rxfh,
-                  struct netlink_ext_ack *extack)
-{
-       struct mlx5e_priv *priv = netdev_priv(dev);
-       u8 hfunc = rxfh->hfunc;
-       int err;
-
-       mutex_lock(&priv->state_lock);
-       err = mlx5e_rx_res_rss_set_rxfh(priv->rx_res, 0, rxfh->indir, rxfh->key,
-                                       hfunc == ETH_RSS_HASH_NO_CHANGE ? NULL : &hfunc);
-       mutex_unlock(&priv->state_lock);
-       return err;
-}
-
 #define MLX5E_PFC_PREVEN_AUTO_TOUT_MSEC                100
 #define MLX5E_PFC_PREVEN_TOUT_MAX_MSEC         8000
 #define MLX5E_PFC_PREVEN_MINOR_PRECENT         85
@@ -2402,6 +2382,7 @@ static void mlx5e_get_rmon_stats(struct net_device *netdev,
 }
 
 const struct ethtool_ops mlx5e_ethtool_ops = {
+       .cap_rss_ctx_supported  = true,
        .supported_coalesce_params = ETHTOOL_COALESCE_USECS |
                                     ETHTOOL_COALESCE_MAX_FRAMES |
                                     ETHTOOL_COALESCE_USE_ADAPTIVE |
@@ -2424,8 +2405,6 @@ const struct ethtool_ops mlx5e_ethtool_ops = {
        .get_rxfh_indir_size = mlx5e_get_rxfh_indir_size,
        .get_rxfh          = mlx5e_get_rxfh,
        .set_rxfh          = mlx5e_set_rxfh,
-       .get_rxfh_context  = mlx5e_get_rxfh_context,
-       .set_rxfh_context  = mlx5e_set_rxfh_context,
        .get_rxnfc         = mlx5e_get_rxnfc,
        .set_rxnfc         = mlx5e_set_rxnfc,
        .get_tunable       = mlx5e_get_tunable,
index 702abbe59b7600551241becfe105f809d0f01df3..cf55202b3a7bc7a652f43ceab8be0cb4179ec98b 100644 (file)
@@ -37,6 +37,7 @@ ef100_ethtool_get_ringparam(struct net_device *net_dev,
 /*     Ethtool options available
  */
 const struct ethtool_ops ef100_ethtool_ops = {
+       .cap_rss_ctx_supported  = true,
        .get_drvinfo            = efx_ethtool_get_drvinfo,
        .get_msglevel           = efx_ethtool_get_msglevel,
        .set_msglevel           = efx_ethtool_set_msglevel,
@@ -60,8 +61,6 @@ const struct ethtool_ops ef100_ethtool_ops = {
        .get_rxfh_key_size      = efx_ethtool_get_rxfh_key_size,
        .get_rxfh               = efx_ethtool_get_rxfh,
        .set_rxfh               = efx_ethtool_set_rxfh,
-       .get_rxfh_context       = efx_ethtool_get_rxfh_context,
-       .set_rxfh_context       = efx_ethtool_set_rxfh_context,
 
        .get_module_info        = efx_ethtool_get_module_info,
        .get_module_eeprom      = efx_ethtool_get_module_eeprom,
index 364323599f7b8229aa82530f8eb30c3d960e361e..37c69c8d90b1166a44c006a4cd38b7a055873df5 100644 (file)
@@ -240,6 +240,7 @@ static int efx_ethtool_get_ts_info(struct net_device *net_dev,
 }
 
 const struct ethtool_ops efx_ethtool_ops = {
+       .cap_rss_ctx_supported  = true,
        .supported_coalesce_params = ETHTOOL_COALESCE_USECS |
                                     ETHTOOL_COALESCE_USECS_IRQ |
                                     ETHTOOL_COALESCE_USE_ADAPTIVE_RX,
@@ -269,8 +270,6 @@ const struct ethtool_ops efx_ethtool_ops = {
        .get_rxfh_key_size      = efx_ethtool_get_rxfh_key_size,
        .get_rxfh               = efx_ethtool_get_rxfh,
        .set_rxfh               = efx_ethtool_set_rxfh,
-       .get_rxfh_context       = efx_ethtool_get_rxfh_context,
-       .set_rxfh_context       = efx_ethtool_set_rxfh_context,
        .get_ts_info            = efx_ethtool_get_ts_info,
        .get_module_info        = efx_ethtool_get_module_info,
        .get_module_eeprom      = efx_ethtool_get_module_eeprom,
index e32ae9038d9e49dd4bf22343af3d25936d0d2ac1..7d5e5db4eac5d5bc6e6f27a20b0df71bf9fc8760 100644 (file)
@@ -1163,52 +1163,8 @@ u32 efx_ethtool_get_rxfh_key_size(struct net_device *net_dev)
        return efx->type->rx_hash_key_size;
 }
 
-int efx_ethtool_get_rxfh(struct net_device *net_dev,
-                        struct ethtool_rxfh_param *rxfh)
-{
-       struct efx_nic *efx = efx_netdev_priv(net_dev);
-       int rc;
-
-       rc = efx->type->rx_pull_rss_config(efx);
-       if (rc)
-               return rc;
-
-       rxfh->hfunc = ETH_RSS_HASH_TOP;
-       if (rxfh->indir)
-               memcpy(rxfh->indir, efx->rss_context.rx_indir_table,
-                      sizeof(efx->rss_context.rx_indir_table));
-       if (rxfh->key)
-               memcpy(rxfh->key, efx->rss_context.rx_hash_key,
-                      efx->type->rx_hash_key_size);
-       return 0;
-}
-
-int efx_ethtool_set_rxfh(struct net_device *net_dev,
-                        struct ethtool_rxfh_param *rxfh,
-                        struct netlink_ext_ack *extack)
-{
-       struct efx_nic *efx = efx_netdev_priv(net_dev);
-       u32 *indir = rxfh->indir;
-       u8 *key = rxfh->key;
-
-       /* Hash function is Toeplitz, cannot be changed */
-       if (rxfh->hfunc != ETH_RSS_HASH_NO_CHANGE &&
-           rxfh->hfunc != ETH_RSS_HASH_TOP)
-               return -EOPNOTSUPP;
-       if (!indir && !key)
-               return 0;
-
-       if (!key)
-               key = efx->rss_context.rx_hash_key;
-       if (!indir)
-               indir = efx->rss_context.rx_indir_table;
-
-       return efx->type->rx_push_rss_config(efx, true, indir, key);
-}
-
-int efx_ethtool_get_rxfh_context(struct net_device *net_dev,
-                                struct ethtool_rxfh_param *rxfh,
-                                u32 rss_context)
+static int efx_ethtool_get_rxfh_context(struct net_device *net_dev,
+                                       struct ethtool_rxfh_param *rxfh)
 {
        struct efx_nic *efx = efx_netdev_priv(net_dev);
        struct efx_rss_context *ctx;
@@ -1218,7 +1174,7 @@ int efx_ethtool_get_rxfh_context(struct net_device *net_dev,
                return -EOPNOTSUPP;
 
        mutex_lock(&efx->rss_lock);
-       ctx = efx_find_rss_context_entry(efx, rss_context);
+       ctx = efx_find_rss_context_entry(efx, rxfh->rss_context);
        if (!ctx) {
                rc = -ENOENT;
                goto out_unlock;
@@ -1239,11 +1195,35 @@ out_unlock:
        return rc;
 }
 
-int efx_ethtool_set_rxfh_context(struct net_device *net_dev,
-                                struct ethtool_rxfh_param *rxfh,
-                                u32 *rss_context, bool delete)
+int efx_ethtool_get_rxfh(struct net_device *net_dev,
+                        struct ethtool_rxfh_param *rxfh)
 {
        struct efx_nic *efx = efx_netdev_priv(net_dev);
+       int rc;
+
+       if (rxfh->rss_context)
+               return efx_ethtool_get_rxfh_context(net_dev, rxfh);
+
+       rc = efx->type->rx_pull_rss_config(efx);
+       if (rc)
+               return rc;
+
+       rxfh->hfunc = ETH_RSS_HASH_TOP;
+       if (rxfh->indir)
+               memcpy(rxfh->indir, efx->rss_context.rx_indir_table,
+                      sizeof(efx->rss_context.rx_indir_table));
+       if (rxfh->key)
+               memcpy(rxfh->key, efx->rss_context.rx_hash_key,
+                      efx->type->rx_hash_key_size);
+       return 0;
+}
+
+static int efx_ethtool_set_rxfh_context(struct net_device *net_dev,
+                                       struct ethtool_rxfh_param *rxfh,
+                                       struct netlink_ext_ack *extack)
+{
+       struct efx_nic *efx = efx_netdev_priv(net_dev);
+       u32 *rss_context = &rxfh->rss_context;
        struct efx_rss_context *ctx;
        u32 *indir = rxfh->indir;
        bool allocated = false;
@@ -1252,15 +1232,11 @@ int efx_ethtool_set_rxfh_context(struct net_device *net_dev,
 
        if (!efx->type->rx_push_rss_context_config)
                return -EOPNOTSUPP;
-       /* Hash function is Toeplitz, cannot be changed */
-       if (rxfh->hfunc != ETH_RSS_HASH_NO_CHANGE &&
-           rxfh->hfunc != ETH_RSS_HASH_TOP)
-               return -EOPNOTSUPP;
 
        mutex_lock(&efx->rss_lock);
 
        if (*rss_context == ETH_RXFH_CONTEXT_ALLOC) {
-               if (delete) {
+               if (rxfh->rss_delete) {
                        /* alloc + delete == Nothing to do */
                        rc = -EINVAL;
                        goto out_unlock;
@@ -1283,7 +1259,7 @@ int efx_ethtool_set_rxfh_context(struct net_device *net_dev,
                }
        }
 
-       if (delete) {
+       if (rxfh->rss_delete) {
                /* delete this context */
                rc = efx->type->rx_push_rss_context_config(efx, ctx, NULL, NULL);
                if (!rc)
@@ -1306,6 +1282,33 @@ out_unlock:
        return rc;
 }
 
+int efx_ethtool_set_rxfh(struct net_device *net_dev,
+                        struct ethtool_rxfh_param *rxfh,
+                        struct netlink_ext_ack *extack)
+{
+       struct efx_nic *efx = efx_netdev_priv(net_dev);
+       u32 *indir = rxfh->indir;
+       u8 *key = rxfh->key;
+
+       /* Hash function is Toeplitz, cannot be changed */
+       if (rxfh->hfunc != ETH_RSS_HASH_NO_CHANGE &&
+           rxfh->hfunc != ETH_RSS_HASH_TOP)
+               return -EOPNOTSUPP;
+
+       if (rxfh->rss_context)
+               return efx_ethtool_set_rxfh_context(net_dev, rxfh, extack);
+
+       if (!indir && !key)
+               return 0;
+
+       if (!key)
+               key = efx->rss_context.rx_hash_key;
+       if (!indir)
+               indir = efx->rss_context.rx_indir_table;
+
+       return efx->type->rx_push_rss_config(efx, true, indir, key);
+}
+
 int efx_ethtool_reset(struct net_device *net_dev, u32 *flags)
 {
        struct efx_nic *efx = efx_netdev_priv(net_dev);
index 384318ee11277414dab2cee17278aa9280e8cc83..a680e59802138c7f0c9ea87c89bc79500eb0af08 100644 (file)
@@ -49,12 +49,6 @@ int efx_ethtool_get_rxfh(struct net_device *net_dev,
 int efx_ethtool_set_rxfh(struct net_device *net_dev,
                         struct ethtool_rxfh_param *rxfh,
                         struct netlink_ext_ack *extack);
-int efx_ethtool_get_rxfh_context(struct net_device *net_dev,
-                                struct ethtool_rxfh_param *rxfh,
-                                u32 rss_context);
-int efx_ethtool_set_rxfh_context(struct net_device *net_dev,
-                                struct ethtool_rxfh_param *rxfh,
-                                u32 *rss_context, bool delete);
 int efx_ethtool_reset(struct net_device *net_dev, u32 *flags);
 int efx_ethtool_get_module_eeprom(struct net_device *net_dev,
                                  struct ethtool_eeprom *ee,
index e4ec589216c1b926a2733277c362123008c52b82..14dd3893bdefad0f38edb62899130d4211fc6577 100644 (file)
@@ -240,6 +240,7 @@ static int efx_ethtool_get_ts_info(struct net_device *net_dev,
 }
 
 const struct ethtool_ops efx_siena_ethtool_ops = {
+       .cap_rss_ctx_supported  = true,
        .supported_coalesce_params = ETHTOOL_COALESCE_USECS |
                                     ETHTOOL_COALESCE_USECS_IRQ |
                                     ETHTOOL_COALESCE_USE_ADAPTIVE_RX,
@@ -269,8 +270,6 @@ const struct ethtool_ops efx_siena_ethtool_ops = {
        .get_rxfh_key_size      = efx_siena_ethtool_get_rxfh_key_size,
        .get_rxfh               = efx_siena_ethtool_get_rxfh,
        .set_rxfh               = efx_siena_ethtool_set_rxfh,
-       .get_rxfh_context       = efx_siena_ethtool_get_rxfh_context,
-       .set_rxfh_context       = efx_siena_ethtool_set_rxfh_context,
        .get_ts_info            = efx_ethtool_get_ts_info,
        .get_module_info        = efx_siena_ethtool_get_module_info,
        .get_module_eeprom      = efx_siena_ethtool_get_module_eeprom,
index 20b64e3521cb0b9b7c71c6c155b033afbc8855cf..5f0a8127e967f7e1780412f2d0813eb39eeaef2a 100644 (file)
@@ -1164,52 +1164,8 @@ u32 efx_siena_ethtool_get_rxfh_key_size(struct net_device *net_dev)
        return efx->type->rx_hash_key_size;
 }
 
-int efx_siena_ethtool_get_rxfh(struct net_device *net_dev,
-                              struct ethtool_rxfh_param *rxfh)
-{
-       struct efx_nic *efx = netdev_priv(net_dev);
-       int rc;
-
-       rc = efx->type->rx_pull_rss_config(efx);
-       if (rc)
-               return rc;
-
-       rxfh->hfunc = ETH_RSS_HASH_TOP;
-       if (rxfh->indir)
-               memcpy(rxfh->indir, efx->rss_context.rx_indir_table,
-                      sizeof(efx->rss_context.rx_indir_table));
-       if (rxfh->key)
-               memcpy(rxfh->key, efx->rss_context.rx_hash_key,
-                      efx->type->rx_hash_key_size);
-       return 0;
-}
-
-int efx_siena_ethtool_set_rxfh(struct net_device *net_dev,
-                              struct ethtool_rxfh_param *rxfh,
-                              struct netlink_ext_ack *extack)
-{
-       struct efx_nic *efx = netdev_priv(net_dev);
-       u32 *indir = rxfh->indir;
-       u8 *key = rxfh->key;
-
-       /* Hash function is Toeplitz, cannot be changed */
-       if (rxfh->hfunc != ETH_RSS_HASH_NO_CHANGE &&
-           rxfh->hfunc != ETH_RSS_HASH_TOP)
-               return -EOPNOTSUPP;
-       if (!indir && !key)
-               return 0;
-
-       if (!key)
-               key = efx->rss_context.rx_hash_key;
-       if (!indir)
-               indir = efx->rss_context.rx_indir_table;
-
-       return efx->type->rx_push_rss_config(efx, true, indir, key);
-}
-
-int efx_siena_ethtool_get_rxfh_context(struct net_device *net_dev,
-                                      struct ethtool_rxfh_param *rxfh,
-                                      u32 rss_context)
+static int efx_siena_ethtool_get_rxfh_context(struct net_device *net_dev,
+                                             struct ethtool_rxfh_param *rxfh)
 {
        struct efx_nic *efx = netdev_priv(net_dev);
        struct efx_rss_context *ctx;
@@ -1219,7 +1175,7 @@ int efx_siena_ethtool_get_rxfh_context(struct net_device *net_dev,
                return -EOPNOTSUPP;
 
        mutex_lock(&efx->rss_lock);
-       ctx = efx_siena_find_rss_context_entry(efx, rss_context);
+       ctx = efx_siena_find_rss_context_entry(efx, rxfh->rss_context);
        if (!ctx) {
                rc = -ENOENT;
                goto out_unlock;
@@ -1240,11 +1196,35 @@ out_unlock:
        return rc;
 }
 
-int efx_siena_ethtool_set_rxfh_context(struct net_device *net_dev,
-                                      struct ethtool_rxfh_param *rxfh,
-                                      u32 *rss_context, bool delete)
+int efx_siena_ethtool_get_rxfh(struct net_device *net_dev,
+                              struct ethtool_rxfh_param *rxfh)
 {
        struct efx_nic *efx = netdev_priv(net_dev);
+       int rc;
+
+       if (rxfh->rss_context)
+               return efx_siena_ethtool_get_rxfh_context(net_dev, rxfh);
+
+       rc = efx->type->rx_pull_rss_config(efx);
+       if (rc)
+               return rc;
+
+       rxfh->hfunc = ETH_RSS_HASH_TOP;
+       if (rxfh->indir)
+               memcpy(rxfh->indir, efx->rss_context.rx_indir_table,
+                      sizeof(efx->rss_context.rx_indir_table));
+       if (rxfh->key)
+               memcpy(rxfh->key, efx->rss_context.rx_hash_key,
+                      efx->type->rx_hash_key_size);
+       return 0;
+}
+
+static int efx_siena_ethtool_set_rxfh_context(struct net_device *net_dev,
+                                             struct ethtool_rxfh_param *rxfh,
+                                             struct netlink_ext_ack *extack)
+{
+       struct efx_nic *efx = netdev_priv(net_dev);
+       u32 *rss_context = &rxfh->rss_context;
        struct efx_rss_context *ctx;
        u32 *indir = rxfh->indir;
        bool allocated = false;
@@ -1253,15 +1233,11 @@ int efx_siena_ethtool_set_rxfh_context(struct net_device *net_dev,
 
        if (!efx->type->rx_push_rss_context_config)
                return -EOPNOTSUPP;
-       /* Hash function is Toeplitz, cannot be changed */
-       if (rxfh->hfunc != ETH_RSS_HASH_NO_CHANGE &&
-           rxfh->hfunc != ETH_RSS_HASH_TOP)
-               return -EOPNOTSUPP;
 
        mutex_lock(&efx->rss_lock);
 
        if (*rss_context == ETH_RXFH_CONTEXT_ALLOC) {
-               if (delete) {
+               if (rxfh->rss_delete) {
                        /* alloc + delete == Nothing to do */
                        rc = -EINVAL;
                        goto out_unlock;
@@ -1284,7 +1260,7 @@ int efx_siena_ethtool_set_rxfh_context(struct net_device *net_dev,
                }
        }
 
-       if (delete) {
+       if (rxfh->rss_delete) {
                /* delete this context */
                rc = efx->type->rx_push_rss_context_config(efx, ctx, NULL, NULL);
                if (!rc)
@@ -1307,6 +1283,33 @@ out_unlock:
        return rc;
 }
 
+int efx_siena_ethtool_set_rxfh(struct net_device *net_dev,
+                              struct ethtool_rxfh_param *rxfh,
+                              struct netlink_ext_ack *extack)
+{
+       struct efx_nic *efx = netdev_priv(net_dev);
+       u32 *indir = rxfh->indir;
+       u8 *key = rxfh->key;
+
+       /* Hash function is Toeplitz, cannot be changed */
+       if (rxfh->hfunc != ETH_RSS_HASH_NO_CHANGE &&
+           rxfh->hfunc != ETH_RSS_HASH_TOP)
+               return -EOPNOTSUPP;
+
+       if (rxfh->rss_context)
+               efx_siena_ethtool_set_rxfh_context(net_dev, rxfh, extack);
+
+       if (!indir && !key)
+               return 0;
+
+       if (!key)
+               key = efx->rss_context.rx_hash_key;
+       if (!indir)
+               indir = efx->rss_context.rx_indir_table;
+
+       return efx->type->rx_push_rss_config(efx, true, indir, key);
+}
+
 int efx_siena_ethtool_reset(struct net_device *net_dev, u32 *flags)
 {
        struct efx_nic *efx = netdev_priv(net_dev);
index 2f892f8eff70cc40d7fdd67c49efb521f7f2e4bb..d674bab0f65b17a3f22676fa48f8132c235c4c90 100644 (file)
@@ -46,12 +46,6 @@ int efx_siena_ethtool_get_rxfh(struct net_device *net_dev,
 int efx_siena_ethtool_set_rxfh(struct net_device *net_dev,
                               struct ethtool_rxfh_param *rxfh,
                               struct netlink_ext_ack *extack);
-int efx_siena_ethtool_get_rxfh_context(struct net_device *net_dev,
-                                      struct ethtool_rxfh_param *rxfh,
-                                      u32 rss_context);
-int efx_siena_ethtool_set_rxfh_context(struct net_device *net_dev,
-                                      struct ethtool_rxfh_param *rxfh,
-                                      u32 *rss_context, bool delete);
 int efx_siena_ethtool_reset(struct net_device *net_dev, u32 *flags);
 int efx_siena_ethtool_get_module_eeprom(struct net_device *net_dev,
                                        struct ethtool_eeprom *ee,
index 3ab2b6a90419cd6a9c7ebfcfb9bbe056cc3f77e9..66fe254c3e516d3a64c6215d07b630e6d93f2308 100644 (file)
@@ -609,6 +609,12 @@ struct ethtool_mm_stats {
  *     which may be zero.  On GET (read from the driver), the size of the
  *     hardware hash key.
  * @key: The hash key of size @key_size bytes.
+ * @rss_context: RSS context identifier.  Context 0 is the default for normal
+ *     traffic; other contexts can be referenced as the destination for RX flow
+ *     classification rules.  On SET, %ETH_RXFH_CONTEXT_ALLOC is used
+ *     to allocate a new RSS context; on return this field will
+ *     contain the ID of the newly allocated context.
+ * @rss_delete: Set to non-ZERO to remove the @rss_context context.
  */
 struct ethtool_rxfh_param {
        u8      hfunc;
@@ -616,12 +622,16 @@ struct ethtool_rxfh_param {
        u32     *indir;
        u32     key_size;
        u8      *key;
+       u32     rss_context;
+       u8      rss_delete;
 };
 
 /**
  * struct ethtool_ops - optional netdev operations
  * @cap_link_lanes_supported: indicates if the driver supports lanes
  *     parameter.
+ * @cap_rss_ctx_supported: indicates if the driver supports RSS
+ *     contexts.
  * @supported_coalesce_params: supported types of interrupt coalescing.
  * @supported_ring_params: supported ring params.
  * @get_drvinfo: Report driver/device information. Modern drivers no
@@ -718,15 +728,6 @@ struct ethtool_rxfh_param {
  *     will remain unchanged.
  *     Returns a negative error code or zero. An error code must be returned
  *     if at least one unsupported change was requested.
- * @get_rxfh_context: Get the contents of the RX flow hash indirection table,
- *     hash key, and/or hash function assiciated to the given rss context.
- *     Returns a negative error code or zero.
- * @set_rxfh_context: Create, remove and configure RSS contexts. Allows setting
- *     the contents of the RX flow hash indirection table, hash key, and/or
- *     hash function associated to the given context. Arguments which are set
- *     to %NULL or zero will remain unchanged.
- *     Returns a negative error code or zero. An error code must be returned
- *     if at least one unsupported change was requested.
  * @get_channels: Get number of channels.
  * @set_channels: Set number of channels.  Returns a negative error code or
  *     zero.
@@ -809,6 +810,7 @@ struct ethtool_rxfh_param {
  */
 struct ethtool_ops {
        u32     cap_link_lanes_supported:1;
+       u32     cap_rss_ctx_supported:1;
        u32     supported_coalesce_params;
        u32     supported_ring_params;
        void    (*get_drvinfo)(struct net_device *, struct ethtool_drvinfo *);
@@ -871,12 +873,6 @@ struct ethtool_ops {
        int     (*get_rxfh)(struct net_device *, struct ethtool_rxfh_param *);
        int     (*set_rxfh)(struct net_device *, struct ethtool_rxfh_param *,
                            struct netlink_ext_ack *extack);
-       int     (*get_rxfh_context)(struct net_device *,
-                                   struct ethtool_rxfh_param *,
-                                   u32 rss_context);
-       int     (*set_rxfh_context)(struct net_device *,
-                                   struct ethtool_rxfh_param *,
-                                   u32 *rss_context, bool delete);
        void    (*get_channels)(struct net_device *, struct ethtool_channels *);
        int     (*set_channels)(struct net_device *, struct ethtool_channels *);
        int     (*get_dump_flag)(struct net_device *, struct ethtool_dump *);
index 5f49a105fc732285fd3323046b27ca54874b54a9..86e5fc64b7113ff5ae0b667de58bfc6e309ffc45 100644 (file)
@@ -1201,7 +1201,7 @@ static noinline_for_stack int ethtool_get_rxfh(struct net_device *dev,
        if (rxfh.rsvd8[0] || rxfh.rsvd8[1] || rxfh.rsvd8[2] || rxfh.rsvd32)
                return -EINVAL;
        /* Most drivers don't handle rss_context, check it's 0 as well */
-       if (rxfh.rss_context && !ops->get_rxfh_context)
+       if (rxfh.rss_context && !ops->cap_rss_ctx_supported)
                return -EOPNOTSUPP;
 
        rxfh.indir_size = rxfh_dev.indir_size;
@@ -1225,11 +1225,9 @@ static noinline_for_stack int ethtool_get_rxfh(struct net_device *dev,
        if (user_key_size)
                rxfh_dev.key = rss_config + indir_bytes;
 
-       if (rxfh.rss_context)
-               ret = dev->ethtool_ops->get_rxfh_context(dev, &rxfh_dev,
-                                                        rxfh.rss_context);
-       else
-               ret = dev->ethtool_ops->get_rxfh(dev, &rxfh_dev);
+       rxfh_dev.rss_context = rxfh.rss_context;
+
+       ret = dev->ethtool_ops->get_rxfh(dev, &rxfh_dev);
        if (ret)
                goto out;
 
@@ -1257,7 +1255,6 @@ static noinline_for_stack int ethtool_set_rxfh(struct net_device *dev,
        struct netlink_ext_ack *extack = NULL;
        struct ethtool_rxnfc rx_rings;
        struct ethtool_rxfh rxfh;
-       bool delete = false;
        u32 indir_bytes = 0;
        u8 *rss_config;
        int ret;
@@ -1277,7 +1274,7 @@ static noinline_for_stack int ethtool_set_rxfh(struct net_device *dev,
        if (rxfh.rsvd8[0] || rxfh.rsvd8[1] || rxfh.rsvd8[2] || rxfh.rsvd32)
                return -EINVAL;
        /* Most drivers don't handle rss_context, check it's 0 as well */
-       if (rxfh.rss_context && !ops->set_rxfh_context)
+       if (rxfh.rss_context && !ops->cap_rss_ctx_supported)
                return -EOPNOTSUPP;
 
        /* If either indir, hash key or function is valid, proceed further.
@@ -1327,7 +1324,7 @@ static noinline_for_stack int ethtool_set_rxfh(struct net_device *dev,
                        for (i = 0; i < dev_indir_size; i++)
                                indir[i] = ethtool_rxfh_indir_default(i, rx_rings.data);
                } else {
-                       delete = true;
+                       rxfh_dev.rss_delete = true;
                }
        }
 
@@ -1343,21 +1340,17 @@ static noinline_for_stack int ethtool_set_rxfh(struct net_device *dev,
        }
 
        rxfh_dev.hfunc = rxfh.hfunc;
+       rxfh_dev.rss_context = rxfh.rss_context;
 
-       if (rxfh.rss_context)
-               ret = ops->set_rxfh_context(dev, &rxfh_dev,
-                                           &rxfh.rss_context, delete);
-       else
-               ret = ops->set_rxfh(dev, &rxfh_dev, extack);
-
+       ret = ops->set_rxfh(dev, &rxfh_dev, extack);
        if (ret)
                goto out;
 
        if (copy_to_user(useraddr + offsetof(struct ethtool_rxfh, rss_context),
-                        &rxfh.rss_context, sizeof(rxfh.rss_context)))
+                        &rxfh_dev.rss_context, sizeof(rxfh_dev.rss_context)))
                ret = -EFAULT;
 
-       if (!rxfh.rss_context) {
+       if (!rxfh_dev.rss_context) {
                /* indicate whether rxfh was set to default */
                if (rxfh.indir_size == 0)
                        dev->priv_flags &= ~IFF_RXFH_CONFIGURED;
index 56fae7d5c0f77754d2853851c832ff376e1eac6f..efc9f4409e40bf9e200245c09018cffdb0f268d0 100644 (file)
@@ -59,7 +59,7 @@ rss_prepare_data(const struct ethnl_req_info *req_base,
                return -EOPNOTSUPP;
 
        /* Some drivers don't handle rss_context */
-       if (request->rss_context && !ops->get_rxfh_context)
+       if (request->rss_context && !ops->cap_rss_ctx_supported)
                return -EOPNOTSUPP;
 
        ret = ethnl_ops_begin(dev);
@@ -90,12 +90,9 @@ rss_prepare_data(const struct ethnl_req_info *req_base,
        rxfh.indir = data->indir_table;
        rxfh.key_size = data->hkey_size;
        rxfh.key = data->hkey;
+       rxfh.rss_context = request->rss_context;
 
-       if (request->rss_context)
-               ret = ops->get_rxfh_context(dev, &rxfh, request->rss_context);
-       else
-               ret = ops->get_rxfh(dev, &rxfh);
-
+       ret = ops->get_rxfh(dev, &rxfh);
        if (ret)
                goto out_ops;