net: dummy: remove fake SR-IOV functionality

netdevsim driver seems like a better place for fake SR-IOV
functionality.  Remove the code previously added to dummy.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: Quentin Monnet <quentin.monnet@netronome.com>
Acked-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>

1 files changed, 1 insertions, 214 deletions

diff --git a/drivers/net/dummy.c b/drivers/net/dummy.c
index 58483af80bdb..30b1c8512049 100644
--- a/drivers/net/dummy.c
+++ b/drivers/net/dummy.c
@@ -42,48 +42,7 @@
 #define DRV_NAME        "dummy"
 #define DRV_VERSION     "1.0"
 
-#undef pr_fmt
-#define pr_fmt(fmt) DRV_NAME ": " fmt
-
 static int numdummies = 1;
-static int num_vfs;
-
-struct vf_data_storage {
-        u8      vf_mac[ETH_ALEN];
-        u16     pf_vlan; /* When set, guest VLAN config not allowed. */
-        u16     pf_qos;
-        __be16  vlan_proto;
-        u16     min_tx_rate;
-        u16     max_tx_rate;
-        u8      spoofchk_enabled;
-        bool    rss_query_enabled;
-        u8      trusted;
-        int     link_state;
-};
-
-struct dummy_priv {
-        struct vf_data_storage  *vfinfo;
-};
-
-static int dummy_num_vf(struct device *dev)
-{
-        return num_vfs;
-}
-
-static struct bus_type dummy_bus = {
-        .name   = "dummy",
-        .num_vf = dummy_num_vf,
-};
-
-static void release_dummy_parent(struct device *dev)
-{
-}
-
-static struct device dummy_parent = {
-        .init_name      = "dummy",
-        .bus            = &dummy_bus,
-        .release        = release_dummy_parent,
-};
 
 /* fake multicast ability */
 static void set_multicast_list(struct net_device *dev)
@@ -133,25 +92,10 @@ static netdev_tx_t dummy_xmit(struct sk_buff *skb, struct net_device *dev)
 
 static int dummy_dev_init(struct net_device *dev)
 {
-        struct dummy_priv *priv = netdev_priv(dev);
-
         dev->dstats = netdev_alloc_pcpu_stats(struct pcpu_dstats);
         if (!dev->dstats)
                 return -ENOMEM;
 
-        priv->vfinfo = NULL;
-
-        if (!num_vfs)
-                return 0;
-
-        dev->dev.parent = &dummy_parent;
-        priv->vfinfo = kcalloc(num_vfs, sizeof(struct vf_data_storage),
-                               GFP_KERNEL);
-        if (!priv->vfinfo) {
-                free_percpu(dev->dstats);
-                return -ENOMEM;
-        }
-
         return 0;
 }
 
@@ -169,117 +113,6 @@ static int dummy_change_carrier(struct net_device *dev, bool new_carrier)
         return 0;
 }
 
-static int dummy_set_vf_mac(struct net_device *dev, int vf, u8 *mac)
-{
-        struct dummy_priv *priv = netdev_priv(dev);
-
-        if (!is_valid_ether_addr(mac) || (vf >= num_vfs))
-                return -EINVAL;
-
-        memcpy(priv->vfinfo[vf].vf_mac, mac, ETH_ALEN);
-
-        return 0;
-}
-
-static int dummy_set_vf_vlan(struct net_device *dev, int vf,
-                             u16 vlan, u8 qos, __be16 vlan_proto)
-{
-        struct dummy_priv *priv = netdev_priv(dev);
-
-        if ((vf >= num_vfs) || (vlan > 4095) || (qos > 7))
-                return -EINVAL;
-
-        priv->vfinfo[vf].pf_vlan = vlan;
-        priv->vfinfo[vf].pf_qos = qos;
-        priv->vfinfo[vf].vlan_proto = vlan_proto;
-
-        return 0;
-}
-
-static int dummy_set_vf_rate(struct net_device *dev, int vf, int min, int max)
-{
-        struct dummy_priv *priv = netdev_priv(dev);
-
-        if (vf >= num_vfs)
-                return -EINVAL;
-
-        priv->vfinfo[vf].min_tx_rate = min;
-        priv->vfinfo[vf].max_tx_rate = max;
-
-        return 0;
-}
-
-static int dummy_set_vf_spoofchk(struct net_device *dev, int vf, bool val)
-{
-        struct dummy_priv *priv = netdev_priv(dev);
-
-        if (vf >= num_vfs)
-                return -EINVAL;
-
-        priv->vfinfo[vf].spoofchk_enabled = val;
-
-        return 0;
-}
-
-static int dummy_set_vf_rss_query_en(struct net_device *dev, int vf, bool val)
-{
-        struct dummy_priv *priv = netdev_priv(dev);
-
-        if (vf >= num_vfs)
-                return -EINVAL;
-
-        priv->vfinfo[vf].rss_query_enabled = val;
-
-        return 0;
-}
-
-static int dummy_set_vf_trust(struct net_device *dev, int vf, bool val)
-{
-        struct dummy_priv *priv = netdev_priv(dev);
-
-        if (vf >= num_vfs)
-                return -EINVAL;
-
-        priv->vfinfo[vf].trusted = val;
-
-        return 0;
-}
-
-static int dummy_get_vf_config(struct net_device *dev,
-                               int vf, struct ifla_vf_info *ivi)
-{
-        struct dummy_priv *priv = netdev_priv(dev);
-
-        if (vf >= num_vfs)
-                return -EINVAL;
-
-        ivi->vf = vf;
-        memcpy(&ivi->mac, priv->vfinfo[vf].vf_mac, ETH_ALEN);
-        ivi->vlan = priv->vfinfo[vf].pf_vlan;
-        ivi->qos = priv->vfinfo[vf].pf_qos;
-        ivi->spoofchk = priv->vfinfo[vf].spoofchk_enabled;
-        ivi->linkstate = priv->vfinfo[vf].link_state;
-        ivi->min_tx_rate = priv->vfinfo[vf].min_tx_rate;
-        ivi->max_tx_rate = priv->vfinfo[vf].max_tx_rate;
-        ivi->rss_query_en = priv->vfinfo[vf].rss_query_enabled;
-        ivi->trusted = priv->vfinfo[vf].trusted;
-        ivi->vlan_proto = priv->vfinfo[vf].vlan_proto;
-
-        return 0;
-}
-
-static int dummy_set_vf_link_state(struct net_device *dev, int vf, int state)
-{
-        struct dummy_priv *priv = netdev_priv(dev);
-
-        if (vf >= num_vfs)
-                return -EINVAL;
-
-        priv->vfinfo[vf].link_state = state;
-
-        return 0;
-}
-
 static const struct net_device_ops dummy_netdev_ops = {
         .ndo_init               = dummy_dev_init,
         .ndo_uninit             = dummy_dev_uninit,
@@ -289,14 +122,6 @@ static const struct net_device_ops dummy_netdev_ops = {
         .ndo_set_mac_address    = eth_mac_addr,
         .ndo_get_stats64        = dummy_get_stats64,
         .ndo_change_carrier     = dummy_change_carrier,
-        .ndo_set_vf_mac         = dummy_set_vf_mac,
-        .ndo_set_vf_vlan        = dummy_set_vf_vlan,
-        .ndo_set_vf_rate        = dummy_set_vf_rate,
-        .ndo_set_vf_spoofchk    = dummy_set_vf_spoofchk,
-        .ndo_set_vf_trust       = dummy_set_vf_trust,
-        .ndo_get_vf_config      = dummy_get_vf_config,
-        .ndo_set_vf_link_state  = dummy_set_vf_link_state,
-        .ndo_set_vf_rss_query_en = dummy_set_vf_rss_query_en,
 };
 
 static void dummy_get_drvinfo(struct net_device *dev,
@@ -323,13 +148,6 @@ static const struct ethtool_ops dummy_ethtool_ops = {
         .get_ts_info            = dummy_get_ts_info,
 };
 
-static void dummy_free_netdev(struct net_device *dev)
-{
-        struct dummy_priv *priv = netdev_priv(dev);
-
-        kfree(priv->vfinfo);
-}
-
 static void dummy_setup(struct net_device *dev)
 {
         ether_setup(dev);
@@ -338,7 +156,6 @@ static void dummy_setup(struct net_device *dev)
         dev->netdev_ops = &dummy_netdev_ops;
         dev->ethtool_ops = &dummy_ethtool_ops;
         dev->needs_free_netdev = true;
-        dev->priv_destructor = dummy_free_netdev;
 
         /* Fill in device structure with ethernet-generic values. */
         dev->flags |= IFF_NOARP;
@@ -370,7 +187,6 @@ static int dummy_validate(struct nlattr *tb[], struct nlattr *data[],
 
 static struct rtnl_link_ops dummy_link_ops __read_mostly = {
         .kind           = DRV_NAME,
-        .priv_size      = sizeof(struct dummy_priv),
         .setup          = dummy_setup,
         .validate       = dummy_validate,
 };
@@ -379,16 +195,12 @@ static struct rtnl_link_ops dummy_link_ops __read_mostly = {
 module_param(numdummies, int, 0);
 MODULE_PARM_DESC(numdummies, "Number of dummy pseudo devices");
 
-module_param(num_vfs, int, 0);
-MODULE_PARM_DESC(num_vfs, "Number of dummy VFs per dummy device");
-
 static int __init dummy_init_one(void)
 {
         struct net_device *dev_dummy;
         int err;
 
-        dev_dummy = alloc_netdev(sizeof(struct dummy_priv),
-                                 "dummy%d", NET_NAME_ENUM, dummy_setup);
+        dev_dummy = alloc_netdev(0, "dummy%d", NET_NAME_ENUM, dummy_setup);
         if (!dev_dummy)
                 return -ENOMEM;
 
@@ -407,21 +219,6 @@ static int __init dummy_init_module(void)
 {
         int i, err = 0;
 
-        if (num_vfs) {
-                err = bus_register(&dummy_bus);
-                if (err < 0) {
-                        pr_err("registering dummy bus failed\n");
-                        return err;
-                }
-
-                err = device_register(&dummy_parent);
-                if (err < 0) {
-                        pr_err("registering dummy parent device failed\n");
-                        bus_unregister(&dummy_bus);
-                        return err;
-                }
-        }
-
         rtnl_lock();
         err = __rtnl_link_register(&dummy_link_ops);
         if (err < 0)
@@ -437,22 +234,12 @@ static int __init dummy_init_module(void)
 out:
         rtnl_unlock();
 
-        if (err && num_vfs) {
-                device_unregister(&dummy_parent);
-                bus_unregister(&dummy_bus);
-        }
-
         return err;
 }
 
 static void __exit dummy_cleanup_module(void)
 {
         rtnl_link_unregister(&dummy_link_ops);
-
-        if (num_vfs) {
-                device_unregister(&dummy_parent);
-                bus_unregister(&dummy_bus);
-        }
 }
 
 module_init(dummy_init_module);