netvsc: add some rtnl_dereference annotations
authorstephen hemminger <stephen@networkplumber.org>
Wed, 19 Jul 2017 18:53:13 +0000 (11:53 -0700)
committerDavid S. Miller <davem@davemloft.net>
Thu, 20 Jul 2017 05:20:05 +0000 (22:20 -0700)
In a couple places RTNL is held, and the netvsc_device pointer
is acquired without annotation.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/hyperv/netvsc.c
drivers/net/hyperv/netvsc_drv.c

index 0a9167dd72fb94e50692fa70ec9bc50fd99f733e..e202ec5d6f637a52bfc882a9897ab0ebf79605b3 100644 (file)
@@ -41,7 +41,7 @@ void netvsc_switch_datapath(struct net_device *ndev, bool vf)
 {
        struct net_device_context *net_device_ctx = netdev_priv(ndev);
        struct hv_device *dev = net_device_ctx->device_ctx;
-       struct netvsc_device *nv_dev = net_device_ctx->nvdev;
+       struct netvsc_device *nv_dev = rtnl_dereference(net_device_ctx->nvdev);
        struct nvsp_message *init_pkt = &nv_dev->channel_init_pkt;
 
        memset(init_pkt, 0, sizeof(struct nvsp_message));
@@ -549,7 +549,8 @@ void netvsc_device_remove(struct hv_device *device)
 {
        struct net_device *ndev = hv_get_drvdata(device);
        struct net_device_context *net_device_ctx = netdev_priv(ndev);
-       struct netvsc_device *net_device = net_device_ctx->nvdev;
+       struct netvsc_device *net_device
+               = rtnl_dereference(net_device_ctx->nvdev);
        int i;
 
        netvsc_disconnect_vsp(device);
index 09b07ca9e69a41eae216a8869ed847b1d7aab3de..e8e82a6a4b1ae806875efbd360ddbd2d136c5e0d 100644 (file)
@@ -69,7 +69,7 @@ static void netvsc_set_multicast_list(struct net_device *net)
 static int netvsc_open(struct net_device *net)
 {
        struct net_device_context *ndev_ctx = netdev_priv(net);
-       struct netvsc_device *nvdev = ndev_ctx->nvdev;
+       struct netvsc_device *nvdev = rtnl_dereference(ndev_ctx->nvdev);
        struct rndis_device *rdev;
        int ret = 0;
 
@@ -1364,7 +1364,7 @@ static struct net_device *get_netvsc_byref(struct net_device *vf_netdev)
                        continue;       /* not a netvsc device */
 
                net_device_ctx = netdev_priv(dev);
-               if (net_device_ctx->nvdev == NULL)
+               if (!rtnl_dereference(net_device_ctx->nvdev))
                        continue;       /* device is removed */
 
                if (rtnl_dereference(net_device_ctx->vf_netdev) == vf_netdev)
@@ -1589,7 +1589,8 @@ static int netvsc_remove(struct hv_device *dev)
         * removed. Also blocks mtu and channel changes.
         */
        rtnl_lock();
-       rndis_filter_device_remove(dev, ndev_ctx->nvdev);
+       rndis_filter_device_remove(dev,
+                                  rtnl_dereference(ndev_ctx->nvdev));
        rtnl_unlock();
 
        unregister_netdev(net);