const: constify remaining dev_pm_ops
[safe/jmp/linux-2.6] / net / core / ethtool.c
index d255209..d8aee58 100644 (file)
@@ -30,10 +30,17 @@ u32 ethtool_op_get_link(struct net_device *dev)
        return netif_carrier_ok(dev) ? 1 : 0;
 }
 
+u32 ethtool_op_get_rx_csum(struct net_device *dev)
+{
+       return (dev->features & NETIF_F_ALL_CSUM) != 0;
+}
+EXPORT_SYMBOL(ethtool_op_get_rx_csum);
+
 u32 ethtool_op_get_tx_csum(struct net_device *dev)
 {
        return (dev->features & NETIF_F_ALL_CSUM) != 0;
 }
+EXPORT_SYMBOL(ethtool_op_get_tx_csum);
 
 int ethtool_op_set_tx_csum(struct net_device *dev, u32 data)
 {
@@ -191,13 +198,6 @@ static int ethtool_get_drvinfo(struct net_device *dev, void __user *useraddr)
                rc = ops->get_sset_count(dev, ETH_SS_PRIV_FLAGS);
                if (rc >= 0)
                        info.n_priv_flags = rc;
-       } else {
-               /* code path for obsolete hooks */
-
-               if (ops->self_test_count)
-                       info.testinfo_len = ops->self_test_count(dev);
-               if (ops->get_stats_count)
-                       info.n_stats = ops->get_stats_count(dev);
        }
        if (ops->get_regs_len)
                info.regdump_len = ops->get_regs_len(dev);
@@ -209,6 +209,63 @@ static int ethtool_get_drvinfo(struct net_device *dev, void __user *useraddr)
        return 0;
 }
 
+static int ethtool_set_rxnfc(struct net_device *dev, void __user *useraddr)
+{
+       struct ethtool_rxnfc cmd;
+
+       if (!dev->ethtool_ops->set_rxnfc)
+               return -EOPNOTSUPP;
+
+       if (copy_from_user(&cmd, useraddr, sizeof(cmd)))
+               return -EFAULT;
+
+       return dev->ethtool_ops->set_rxnfc(dev, &cmd);
+}
+
+static int ethtool_get_rxnfc(struct net_device *dev, void __user *useraddr)
+{
+       struct ethtool_rxnfc info;
+       const struct ethtool_ops *ops = dev->ethtool_ops;
+       int ret;
+       void *rule_buf = NULL;
+
+       if (!ops->get_rxnfc)
+               return -EOPNOTSUPP;
+
+       if (copy_from_user(&info, useraddr, sizeof(info)))
+               return -EFAULT;
+
+       if (info.cmd == ETHTOOL_GRXCLSRLALL) {
+               if (info.rule_cnt > 0) {
+                       rule_buf = kmalloc(info.rule_cnt * sizeof(u32),
+                                          GFP_USER);
+                       if (!rule_buf)
+                               return -ENOMEM;
+               }
+       }
+
+       ret = ops->get_rxnfc(dev, &info, rule_buf);
+       if (ret < 0)
+               goto err_out;
+
+       ret = -EFAULT;
+       if (copy_to_user(useraddr, &info, sizeof(info)))
+               goto err_out;
+
+       if (rule_buf) {
+               useraddr += offsetof(struct ethtool_rxnfc, rule_locs);
+               if (copy_to_user(useraddr, rule_buf,
+                                info.rule_cnt * sizeof(u32)))
+                       goto err_out;
+       }
+       ret = 0;
+
+err_out:
+       kfree(rule_buf);
+
+       return ret;
+}
+
 static int ethtool_get_regs(struct net_device *dev, char __user *useraddr)
 {
        struct ethtool_regs regs;
@@ -245,59 +302,51 @@ static int ethtool_get_regs(struct net_device *dev, char __user *useraddr)
        return ret;
 }
 
-static int ethtool_get_wol(struct net_device *dev, char __user *useraddr)
+static int ethtool_reset(struct net_device *dev, char __user *useraddr)
 {
-       struct ethtool_wolinfo wol = { ETHTOOL_GWOL };
+       struct ethtool_value reset;
+       int ret;
 
-       if (!dev->ethtool_ops->get_wol)
+       if (!dev->ethtool_ops->reset)
                return -EOPNOTSUPP;
 
-       dev->ethtool_ops->get_wol(dev, &wol);
-
-       if (copy_to_user(useraddr, &wol, sizeof(wol)))
+       if (copy_from_user(&reset, useraddr, sizeof(reset)))
                return -EFAULT;
-       return 0;
-}
 
-static int ethtool_set_wol(struct net_device *dev, char __user *useraddr)
-{
-       struct ethtool_wolinfo wol;
-
-       if (!dev->ethtool_ops->set_wol)
-               return -EOPNOTSUPP;
+       ret = dev->ethtool_ops->reset(dev, &reset.data);
+       if (ret)
+               return ret;
 
-       if (copy_from_user(&wol, useraddr, sizeof(wol)))
+       if (copy_to_user(useraddr, &reset, sizeof(reset)))
                return -EFAULT;
-
-       return dev->ethtool_ops->set_wol(dev, &wol);
+       return 0;
 }
 
-static int ethtool_get_msglevel(struct net_device *dev, char __user *useraddr)
+static int ethtool_get_wol(struct net_device *dev, char __user *useraddr)
 {
-       struct ethtool_value edata = { ETHTOOL_GMSGLVL };
+       struct ethtool_wolinfo wol = { ETHTOOL_GWOL };
 
-       if (!dev->ethtool_ops->get_msglevel)
+       if (!dev->ethtool_ops->get_wol)
                return -EOPNOTSUPP;
 
-       edata.data = dev->ethtool_ops->get_msglevel(dev);
+       dev->ethtool_ops->get_wol(dev, &wol);
 
-       if (copy_to_user(useraddr, &edata, sizeof(edata)))
+       if (copy_to_user(useraddr, &wol, sizeof(wol)))
                return -EFAULT;
        return 0;
 }
 
-static int ethtool_set_msglevel(struct net_device *dev, char __user *useraddr)
+static int ethtool_set_wol(struct net_device *dev, char __user *useraddr)
 {
-       struct ethtool_value edata;
+       struct ethtool_wolinfo wol;
 
-       if (!dev->ethtool_ops->set_msglevel)
+       if (!dev->ethtool_ops->set_wol)
                return -EOPNOTSUPP;
 
-       if (copy_from_user(&edata, useraddr, sizeof(edata)))
+       if (copy_from_user(&wol, useraddr, sizeof(wol)))
                return -EFAULT;
 
-       dev->ethtool_ops->set_msglevel(dev, edata.data);
-       return 0;
+       return dev->ethtool_ops->set_wol(dev, &wol);
 }
 
 static int ethtool_nway_reset(struct net_device *dev)
@@ -308,26 +357,14 @@ static int ethtool_nway_reset(struct net_device *dev)
        return dev->ethtool_ops->nway_reset(dev);
 }
 
-static int ethtool_get_link(struct net_device *dev, void __user *useraddr)
-{
-       struct ethtool_value edata = { ETHTOOL_GLINK };
-
-       if (!dev->ethtool_ops->get_link)
-               return -EOPNOTSUPP;
-
-       edata.data = dev->ethtool_ops->get_link(dev);
-
-       if (copy_to_user(useraddr, &edata, sizeof(edata)))
-               return -EFAULT;
-       return 0;
-}
-
 static int ethtool_get_eeprom(struct net_device *dev, void __user *useraddr)
 {
        struct ethtool_eeprom eeprom;
        const struct ethtool_ops *ops = dev->ethtool_ops;
+       void __user *userbuf = useraddr + sizeof(eeprom);
+       u32 bytes_remaining;
        u8 *data;
-       int ret;
+       int ret = 0;
 
        if (!ops->get_eeprom || !ops->get_eeprom_len)
                return -EOPNOTSUPP;
@@ -343,26 +380,31 @@ static int ethtool_get_eeprom(struct net_device *dev, void __user *useraddr)
        if (eeprom.offset + eeprom.len > ops->get_eeprom_len(dev))
                return -EINVAL;
 
-       data = kmalloc(eeprom.len, GFP_USER);
+       data = kmalloc(PAGE_SIZE, GFP_USER);
        if (!data)
                return -ENOMEM;
 
-       ret = -EFAULT;
-       if (copy_from_user(data, useraddr + sizeof(eeprom), eeprom.len))
-               goto out;
+       bytes_remaining = eeprom.len;
+       while (bytes_remaining > 0) {
+               eeprom.len = min(bytes_remaining, (u32)PAGE_SIZE);
 
-       ret = ops->get_eeprom(dev, &eeprom, data);
-       if (ret)
-               goto out;
+               ret = ops->get_eeprom(dev, &eeprom, data);
+               if (ret)
+                       break;
+               if (copy_to_user(userbuf, data, eeprom.len)) {
+                       ret = -EFAULT;
+                       break;
+               }
+               userbuf += eeprom.len;
+               eeprom.offset += eeprom.len;
+               bytes_remaining -= eeprom.len;
+       }
 
-       ret = -EFAULT;
+       eeprom.len = userbuf - (useraddr + sizeof(eeprom));
+       eeprom.offset -= eeprom.len;
        if (copy_to_user(useraddr, &eeprom, sizeof(eeprom)))
-               goto out;
-       if (copy_to_user(useraddr + sizeof(eeprom), data, eeprom.len))
-               goto out;
-       ret = 0;
+               ret = -EFAULT;
 
- out:
        kfree(data);
        return ret;
 }
@@ -371,8 +413,10 @@ static int ethtool_set_eeprom(struct net_device *dev, void __user *useraddr)
 {
        struct ethtool_eeprom eeprom;
        const struct ethtool_ops *ops = dev->ethtool_ops;
+       void __user *userbuf = useraddr + sizeof(eeprom);
+       u32 bytes_remaining;
        u8 *data;
-       int ret;
+       int ret = 0;
 
        if (!ops->set_eeprom || !ops->get_eeprom_len)
                return -EOPNOTSUPP;
@@ -388,22 +432,26 @@ static int ethtool_set_eeprom(struct net_device *dev, void __user *useraddr)
        if (eeprom.offset + eeprom.len > ops->get_eeprom_len(dev))
                return -EINVAL;
 
-       data = kmalloc(eeprom.len, GFP_USER);
+       data = kmalloc(PAGE_SIZE, GFP_USER);
        if (!data)
                return -ENOMEM;
 
-       ret = -EFAULT;
-       if (copy_from_user(data, useraddr + sizeof(eeprom), eeprom.len))
-               goto out;
+       bytes_remaining = eeprom.len;
+       while (bytes_remaining > 0) {
+               eeprom.len = min(bytes_remaining, (u32)PAGE_SIZE);
 
-       ret = ops->set_eeprom(dev, &eeprom, data);
-       if (ret)
-               goto out;
-
-       if (copy_to_user(useraddr + sizeof(eeprom), data, eeprom.len))
-               ret = -EFAULT;
+               if (copy_from_user(data, userbuf, eeprom.len)) {
+                       ret = -EFAULT;
+                       break;
+               }
+               ret = ops->set_eeprom(dev, &eeprom, data);
+               if (ret)
+                       break;
+               userbuf += eeprom.len;
+               eeprom.offset += eeprom.len;
+               bytes_remaining -= eeprom.len;
+       }
 
- out:
        kfree(data);
        return ret;
 }
@@ -489,48 +537,6 @@ static int ethtool_set_pauseparam(struct net_device *dev, void __user *useraddr)
        return dev->ethtool_ops->set_pauseparam(dev, &pauseparam);
 }
 
-static int ethtool_get_rx_csum(struct net_device *dev, char __user *useraddr)
-{
-       struct ethtool_value edata = { ETHTOOL_GRXCSUM };
-
-       if (!dev->ethtool_ops->get_rx_csum)
-               return -EOPNOTSUPP;
-
-       edata.data = dev->ethtool_ops->get_rx_csum(dev);
-
-       if (copy_to_user(useraddr, &edata, sizeof(edata)))
-               return -EFAULT;
-       return 0;
-}
-
-static int ethtool_set_rx_csum(struct net_device *dev, char __user *useraddr)
-{
-       struct ethtool_value edata;
-
-       if (!dev->ethtool_ops->set_rx_csum)
-               return -EOPNOTSUPP;
-
-       if (copy_from_user(&edata, useraddr, sizeof(edata)))
-               return -EFAULT;
-
-       dev->ethtool_ops->set_rx_csum(dev, edata.data);
-       return 0;
-}
-
-static int ethtool_get_tx_csum(struct net_device *dev, char __user *useraddr)
-{
-       struct ethtool_value edata = { ETHTOOL_GTXCSUM };
-
-       if (!dev->ethtool_ops->get_tx_csum)
-               return -EOPNOTSUPP;
-
-       edata.data = dev->ethtool_ops->get_tx_csum(dev);
-
-       if (copy_to_user(useraddr, &edata, sizeof(edata)))
-               return -EFAULT;
-       return 0;
-}
-
 static int __ethtool_set_sg(struct net_device *dev, u32 data)
 {
        int err;
@@ -569,18 +575,20 @@ static int ethtool_set_tx_csum(struct net_device *dev, char __user *useraddr)
        return dev->ethtool_ops->set_tx_csum(dev, edata.data);
 }
 
-static int ethtool_get_sg(struct net_device *dev, char __user *useraddr)
+static int ethtool_set_rx_csum(struct net_device *dev, char __user *useraddr)
 {
-       struct ethtool_value edata = { ETHTOOL_GSG };
+       struct ethtool_value edata;
 
-       if (!dev->ethtool_ops->get_sg)
+       if (!dev->ethtool_ops->set_rx_csum)
                return -EOPNOTSUPP;
 
-       edata.data = dev->ethtool_ops->get_sg(dev);
-
-       if (copy_to_user(useraddr, &edata, sizeof(edata)))
+       if (copy_from_user(&edata, useraddr, sizeof(edata)))
                return -EFAULT;
-       return 0;
+
+       if (!edata.data && dev->ethtool_ops->set_sg)
+               dev->features &= ~NETIF_F_GRO;
+
+       return dev->ethtool_ops->set_rx_csum(dev, edata.data);
 }
 
 static int ethtool_set_sg(struct net_device *dev, char __user *useraddr)
@@ -600,20 +608,6 @@ static int ethtool_set_sg(struct net_device *dev, char __user *useraddr)
        return __ethtool_set_sg(dev, edata.data);
 }
 
-static int ethtool_get_tso(struct net_device *dev, char __user *useraddr)
-{
-       struct ethtool_value edata = { ETHTOOL_GTSO };
-
-       if (!dev->ethtool_ops->get_tso)
-               return -EOPNOTSUPP;
-
-       edata.data = dev->ethtool_ops->get_tso(dev);
-
-       if (copy_to_user(useraddr, &edata, sizeof(edata)))
-               return -EFAULT;
-       return 0;
-}
-
 static int ethtool_set_tso(struct net_device *dev, char __user *useraddr)
 {
        struct ethtool_value edata;
@@ -630,18 +624,6 @@ static int ethtool_set_tso(struct net_device *dev, char __user *useraddr)
        return dev->ethtool_ops->set_tso(dev, edata.data);
 }
 
-static int ethtool_get_ufo(struct net_device *dev, char __user *useraddr)
-{
-       struct ethtool_value edata = { ETHTOOL_GUFO };
-
-       if (!dev->ethtool_ops->get_ufo)
-               return -EOPNOTSUPP;
-       edata.data = dev->ethtool_ops->get_ufo(dev);
-       if (copy_to_user(useraddr, &edata, sizeof(edata)))
-                return -EFAULT;
-       return 0;
-}
-
 static int ethtool_set_ufo(struct net_device *dev, char __user *useraddr)
 {
        struct ethtool_value edata;
@@ -680,6 +662,34 @@ static int ethtool_set_gso(struct net_device *dev, char __user *useraddr)
        return 0;
 }
 
+static int ethtool_get_gro(struct net_device *dev, char __user *useraddr)
+{
+       struct ethtool_value edata = { ETHTOOL_GGRO };
+
+       edata.data = dev->features & NETIF_F_GRO;
+       if (copy_to_user(useraddr, &edata, sizeof(edata)))
+                return -EFAULT;
+       return 0;
+}
+
+static int ethtool_set_gro(struct net_device *dev, char __user *useraddr)
+{
+       struct ethtool_value edata;
+
+       if (copy_from_user(&edata, useraddr, sizeof(edata)))
+               return -EFAULT;
+
+       if (edata.data) {
+               if (!dev->ethtool_ops->get_rx_csum ||
+                   !dev->ethtool_ops->get_rx_csum(dev))
+                       return -EINVAL;
+               dev->features |= NETIF_F_GRO;
+       } else
+               dev->features &= ~NETIF_F_GRO;
+
+       return 0;
+}
+
 static int ethtool_self_test(struct net_device *dev, char __user *useraddr)
 {
        struct ethtool_test test;
@@ -687,16 +697,10 @@ static int ethtool_self_test(struct net_device *dev, char __user *useraddr)
        u64 *data;
        int ret, test_len;
 
-       if (!ops->self_test)
-               return -EOPNOTSUPP;
-       if (!ops->get_sset_count && !ops->self_test_count)
+       if (!ops->self_test || !ops->get_sset_count)
                return -EOPNOTSUPP;
 
-       if (ops->get_sset_count)
-               test_len = ops->get_sset_count(dev, ETH_SS_TEST);
-       else
-               /* code path for obsolete hook */
-               test_len = ops->self_test_count(dev);
+       test_len = ops->get_sset_count(dev, ETH_SS_TEST);
        if (test_len < 0)
                return test_len;
        WARN_ON(test_len == 0);
@@ -731,36 +735,17 @@ static int ethtool_get_strings(struct net_device *dev, void __user *useraddr)
        u8 *data;
        int ret;
 
-       if (!ops->get_strings)
+       if (!ops->get_strings || !ops->get_sset_count)
                return -EOPNOTSUPP;
 
        if (copy_from_user(&gstrings, useraddr, sizeof(gstrings)))
                return -EFAULT;
 
-       if (ops->get_sset_count) {
-               ret = ops->get_sset_count(dev, gstrings.string_set);
-               if (ret < 0)
-                       return ret;
-
-               gstrings.len = ret;
-       } else {
-               /* code path for obsolete hooks */
-
-               switch (gstrings.string_set) {
-               case ETH_SS_TEST:
-                       if (!ops->self_test_count)
-                               return -EOPNOTSUPP;
-                       gstrings.len = ops->self_test_count(dev);
-                       break;
-               case ETH_SS_STATS:
-                       if (!ops->get_stats_count)
-                               return -EOPNOTSUPP;
-                       gstrings.len = ops->get_stats_count(dev);
-                       break;
-               default:
-                       return -EINVAL;
-               }
-       }
+       ret = ops->get_sset_count(dev, gstrings.string_set);
+       if (ret < 0)
+               return ret;
+
+       gstrings.len = ret;
 
        data = kmalloc(gstrings.len * ETH_GSTRING_LEN, GFP_USER);
        if (!data)
@@ -801,16 +786,10 @@ static int ethtool_get_stats(struct net_device *dev, void __user *useraddr)
        u64 *data;
        int ret, n_stats;
 
-       if (!ops->get_ethtool_stats)
-               return -EOPNOTSUPP;
-       if (!ops->get_sset_count && !ops->get_stats_count)
+       if (!ops->get_ethtool_stats || !ops->get_sset_count)
                return -EOPNOTSUPP;
 
-       if (ops->get_sset_count)
-               n_stats = ops->get_sset_count(dev, ETH_SS_STATS);
-       else
-               /* code path for obsolete hook */
-               n_stats = ops->get_stats_count(dev);
+       n_stats = ops->get_sset_count(dev, ETH_SS_STATS);
        if (n_stats < 0)
                return n_stats;
        WARN_ON(n_stats == 0);
@@ -857,65 +836,68 @@ static int ethtool_get_perm_addr(struct net_device *dev, void __user *useraddr)
        return 0;
 }
 
-static int ethtool_get_flags(struct net_device *dev, char __user *useraddr)
+static int ethtool_get_value(struct net_device *dev, char __user *useraddr,
+                            u32 cmd, u32 (*actor)(struct net_device *))
 {
-       struct ethtool_value edata = { ETHTOOL_GFLAGS };
+       struct ethtool_value edata = { cmd };
 
-       if (!dev->ethtool_ops->get_flags)
+       if (!actor)
                return -EOPNOTSUPP;
 
-       edata.data = dev->ethtool_ops->get_flags(dev);
+       edata.data = actor(dev);
 
        if (copy_to_user(useraddr, &edata, sizeof(edata)))
                return -EFAULT;
        return 0;
 }
 
-static int ethtool_set_flags(struct net_device *dev, char __user *useraddr)
+static int ethtool_set_value_void(struct net_device *dev, char __user *useraddr,
+                            void (*actor)(struct net_device *, u32))
 {
        struct ethtool_value edata;
 
-       if (!dev->ethtool_ops->set_flags)
+       if (!actor)
                return -EOPNOTSUPP;
 
        if (copy_from_user(&edata, useraddr, sizeof(edata)))
                return -EFAULT;
 
-       return dev->ethtool_ops->set_flags(dev, edata.data);
+       actor(dev, edata.data);
+       return 0;
 }
 
-static int ethtool_get_priv_flags(struct net_device *dev, char __user *useraddr)
+static int ethtool_set_value(struct net_device *dev, char __user *useraddr,
+                            int (*actor)(struct net_device *, u32))
 {
-       struct ethtool_value edata = { ETHTOOL_GPFLAGS };
+       struct ethtool_value edata;
 
-       if (!dev->ethtool_ops->get_priv_flags)
+       if (!actor)
                return -EOPNOTSUPP;
 
-       edata.data = dev->ethtool_ops->get_priv_flags(dev);
-
-       if (copy_to_user(useraddr, &edata, sizeof(edata)))
+       if (copy_from_user(&edata, useraddr, sizeof(edata)))
                return -EFAULT;
-       return 0;
+
+       return actor(dev, edata.data);
 }
 
-static int ethtool_set_priv_flags(struct net_device *dev, char __user *useraddr)
+static int ethtool_flash_device(struct net_device *dev, char __user *useraddr)
 {
-       struct ethtool_value edata;
-
-       if (!dev->ethtool_ops->set_priv_flags)
-               return -EOPNOTSUPP;
+       struct ethtool_flash efl;
 
-       if (copy_from_user(&edata, useraddr, sizeof(edata)))
+       if (copy_from_user(&efl, useraddr, sizeof(efl)))
                return -EFAULT;
 
-       return dev->ethtool_ops->set_priv_flags(dev, edata.data);
+       if (!dev->ethtool_ops->flash_device)
+               return -EOPNOTSUPP;
+
+       return dev->ethtool_ops->flash_device(dev, &efl);
 }
 
 /* The main entry point in this file.  Called from net/core/dev.c */
 
-int dev_ethtool(struct ifreq *ifr)
+int dev_ethtool(struct net *net, struct ifreq *ifr)
 {
-       struct net_device *dev = __dev_get_by_name(ifr->ifr_name);
+       struct net_device *dev = __dev_get_by_name(net, ifr->ifr_name);
        void __user *useraddr = ifr->ifr_data;
        u32 ethcmd;
        int rc;
@@ -947,6 +929,11 @@ int dev_ethtool(struct ifreq *ifr)
        case ETHTOOL_GGSO:
        case ETHTOOL_GFLAGS:
        case ETHTOOL_GPFLAGS:
+       case ETHTOOL_GRXFH:
+       case ETHTOOL_GRXRINGS:
+       case ETHTOOL_GRXCLSRLCNT:
+       case ETHTOOL_GRXCLSRULE:
+       case ETHTOOL_GRXCLSRLALL:
                break;
        default:
                if (!capable(CAP_NET_ADMIN))
@@ -979,16 +966,19 @@ int dev_ethtool(struct ifreq *ifr)
                rc = ethtool_set_wol(dev, useraddr);
                break;
        case ETHTOOL_GMSGLVL:
-               rc = ethtool_get_msglevel(dev, useraddr);
+               rc = ethtool_get_value(dev, useraddr, ethcmd,
+                                      dev->ethtool_ops->get_msglevel);
                break;
        case ETHTOOL_SMSGLVL:
-               rc = ethtool_set_msglevel(dev, useraddr);
+               rc = ethtool_set_value_void(dev, useraddr,
+                                      dev->ethtool_ops->set_msglevel);
                break;
        case ETHTOOL_NWAY_RST:
                rc = ethtool_nway_reset(dev);
                break;
        case ETHTOOL_GLINK:
-               rc = ethtool_get_link(dev, useraddr);
+               rc = ethtool_get_value(dev, useraddr, ethcmd,
+                                      dev->ethtool_ops->get_link);
                break;
        case ETHTOOL_GEEPROM:
                rc = ethtool_get_eeprom(dev, useraddr);
@@ -1015,25 +1005,37 @@ int dev_ethtool(struct ifreq *ifr)
                rc = ethtool_set_pauseparam(dev, useraddr);
                break;
        case ETHTOOL_GRXCSUM:
-               rc = ethtool_get_rx_csum(dev, useraddr);
+               rc = ethtool_get_value(dev, useraddr, ethcmd,
+                                      (dev->ethtool_ops->get_rx_csum ?
+                                       dev->ethtool_ops->get_rx_csum :
+                                       ethtool_op_get_rx_csum));
                break;
        case ETHTOOL_SRXCSUM:
                rc = ethtool_set_rx_csum(dev, useraddr);
                break;
        case ETHTOOL_GTXCSUM:
-               rc = ethtool_get_tx_csum(dev, useraddr);
+               rc = ethtool_get_value(dev, useraddr, ethcmd,
+                                      (dev->ethtool_ops->get_tx_csum ?
+                                       dev->ethtool_ops->get_tx_csum :
+                                       ethtool_op_get_tx_csum));
                break;
        case ETHTOOL_STXCSUM:
                rc = ethtool_set_tx_csum(dev, useraddr);
                break;
        case ETHTOOL_GSG:
-               rc = ethtool_get_sg(dev, useraddr);
+               rc = ethtool_get_value(dev, useraddr, ethcmd,
+                                      (dev->ethtool_ops->get_sg ?
+                                       dev->ethtool_ops->get_sg :
+                                       ethtool_op_get_sg));
                break;
        case ETHTOOL_SSG:
                rc = ethtool_set_sg(dev, useraddr);
                break;
        case ETHTOOL_GTSO:
-               rc = ethtool_get_tso(dev, useraddr);
+               rc = ethtool_get_value(dev, useraddr, ethcmd,
+                                      (dev->ethtool_ops->get_tso ?
+                                       dev->ethtool_ops->get_tso :
+                                       ethtool_op_get_tso));
                break;
        case ETHTOOL_STSO:
                rc = ethtool_set_tso(dev, useraddr);
@@ -1054,7 +1056,10 @@ int dev_ethtool(struct ifreq *ifr)
                rc = ethtool_get_perm_addr(dev, useraddr);
                break;
        case ETHTOOL_GUFO:
-               rc = ethtool_get_ufo(dev, useraddr);
+               rc = ethtool_get_value(dev, useraddr, ethcmd,
+                                      (dev->ethtool_ops->get_ufo ?
+                                       dev->ethtool_ops->get_ufo :
+                                       ethtool_op_get_ufo));
                break;
        case ETHTOOL_SUFO:
                rc = ethtool_set_ufo(dev, useraddr);
@@ -1066,16 +1071,46 @@ int dev_ethtool(struct ifreq *ifr)
                rc = ethtool_set_gso(dev, useraddr);
                break;
        case ETHTOOL_GFLAGS:
-               rc = ethtool_get_flags(dev, useraddr);
+               rc = ethtool_get_value(dev, useraddr, ethcmd,
+                                      (dev->ethtool_ops->get_flags ?
+                                       dev->ethtool_ops->get_flags :
+                                       ethtool_op_get_flags));
                break;
        case ETHTOOL_SFLAGS:
-               rc = ethtool_set_flags(dev, useraddr);
+               rc = ethtool_set_value(dev, useraddr,
+                                      dev->ethtool_ops->set_flags);
                break;
        case ETHTOOL_GPFLAGS:
-               rc = ethtool_get_priv_flags(dev, useraddr);
+               rc = ethtool_get_value(dev, useraddr, ethcmd,
+                                      dev->ethtool_ops->get_priv_flags);
                break;
        case ETHTOOL_SPFLAGS:
-               rc = ethtool_set_priv_flags(dev, useraddr);
+               rc = ethtool_set_value(dev, useraddr,
+                                      dev->ethtool_ops->set_priv_flags);
+               break;
+       case ETHTOOL_GRXFH:
+       case ETHTOOL_GRXRINGS:
+       case ETHTOOL_GRXCLSRLCNT:
+       case ETHTOOL_GRXCLSRULE:
+       case ETHTOOL_GRXCLSRLALL:
+               rc = ethtool_get_rxnfc(dev, useraddr);
+               break;
+       case ETHTOOL_SRXFH:
+       case ETHTOOL_SRXCLSRLDEL:
+       case ETHTOOL_SRXCLSRLINS:
+               rc = ethtool_set_rxnfc(dev, useraddr);
+               break;
+       case ETHTOOL_GGRO:
+               rc = ethtool_get_gro(dev, useraddr);
+               break;
+       case ETHTOOL_SGRO:
+               rc = ethtool_set_gro(dev, useraddr);
+               break;
+       case ETHTOOL_FLASHDEV:
+               rc = ethtool_flash_device(dev, useraddr);
+               break;
+       case ETHTOOL_RESET:
+               rc = ethtool_reset(dev, useraddr);
                break;
        default:
                rc = -EOPNOTSUPP;
@@ -1093,7 +1128,6 @@ int dev_ethtool(struct ifreq *ifr)
 EXPORT_SYMBOL(ethtool_op_get_link);
 EXPORT_SYMBOL(ethtool_op_get_sg);
 EXPORT_SYMBOL(ethtool_op_get_tso);
-EXPORT_SYMBOL(ethtool_op_get_tx_csum);
 EXPORT_SYMBOL(ethtool_op_set_sg);
 EXPORT_SYMBOL(ethtool_op_set_tso);
 EXPORT_SYMBOL(ethtool_op_set_tx_csum);