[PATCH net-next v4 6/8] net: ethtool: update set_rxfh_indir to use ethtool_get_rx_ring_count helper

[Breno Leitao]

Modify ethtool_set_rxfh() to use the new ethtool_get_rx_ring_count()
helper function for retrieving the number of RX rings instead of
directly calling get_rxnfc with ETHTOOL_GRXRINGS.

This way, we can leverage the new helper if it is available in ethtool_ops.

Signed-off-by: Breno Leitao <lei...@debian.org>
---
 net/ethtool/ioctl.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/net/ethtool/ioctl.c b/net/ethtool/ioctl.c
index d61e34751adc8..fa83ddade4f81 100644
--- a/net/ethtool/ioctl.c
+++ b/net/ethtool/ioctl.c
@@ -1350,13 +1350,12 @@ static noinline_for_stack int 
ethtool_set_rxfh_indir(struct net_device *dev,
        const struct ethtool_ops *ops = dev->ethtool_ops;
        struct ethtool_rxfh_param rxfh_dev = {};
        struct netlink_ext_ack *extack = NULL;
-       struct ethtool_rxnfc rx_rings;
+       int num_rx_rings;
        u32 user_size, i;
        int ret;
        u32 ringidx_offset = offsetof(struct ethtool_rxfh_indir, ring_index[0]);
 
-       if (!ops->get_rxfh_indir_size || !ops->set_rxfh ||
-           !ops->get_rxnfc)
+       if (!ops->get_rxfh_indir_size || !ops->set_rxfh)
                return -EOPNOTSUPP;
 
        rxfh_dev.indir_size = ops->get_rxfh_indir_size(dev);
@@ -1376,20 +1375,21 @@ static noinline_for_stack int 
ethtool_set_rxfh_indir(struct net_device *dev,
        if (!rxfh_dev.indir)
                return -ENOMEM;
 
-       rx_rings.cmd = ETHTOOL_GRXRINGS;
-       ret = ops->get_rxnfc(dev, &rx_rings, NULL);
-       if (ret)
+       num_rx_rings = ethtool_get_rx_ring_count(dev);
+       if (num_rx_rings < 0) {
+               ret = num_rx_rings;
                goto out;
+       }
 
        if (user_size == 0) {
                u32 *indir = rxfh_dev.indir;
 
                for (i = 0; i < rxfh_dev.indir_size; i++)
-                       indir[i] = ethtool_rxfh_indir_default(i, rx_rings.data);
+                       indir[i] = ethtool_rxfh_indir_default(i, num_rx_rings);
        } else {
                ret = ethtool_copy_validate_indir(rxfh_dev.indir,
                                                  useraddr + ringidx_offset,
-                                                 rx_rings.data,
+                                                 num_rx_rings,
                                                  rxfh_dev.indir_size);
                if (ret)
                        goto out;

-- 
2.47.3