[PATCH] um: vector: remove vp->lock

[Johannes Berg]

From: Johannes Berg <johannes.b...@intel.com>

This lock is useless, all the places that are using
it for some locking will already hold the RTNL. Just
remove it.

Signed-off-by: Johannes Berg <johannes.b...@intel.com>
---
 arch/um/drivers/vector_kern.c | 14 +-------------
 arch/um/drivers/vector_kern.h |  1 -
 2 files changed, 1 insertion(+), 14 deletions(-)

diff --git a/arch/um/drivers/vector_kern.c b/arch/um/drivers/vector_kern.c
index 4279793b11b7..a46f7f6d131e 100644
--- a/arch/um/drivers/vector_kern.c
+++ b/arch/um/drivers/vector_kern.c
@@ -1158,10 +1158,8 @@ static int vector_net_close(struct net_device *dev)
                destroy_queue(vp->tx_queue);
        kfree(vp->fds);
        vp->fds = NULL;
-       spin_lock_irqsave(&vp->lock, flags);
        vp->opened = false;
        vp->in_error = false;
-       spin_unlock_irqrestore(&vp->lock, flags);
        return 0;
 }
 
@@ -1207,13 +1205,9 @@ static int vector_net_open(struct net_device *dev)
        int err = -EINVAL;
        struct vector_device *vdevice;
 
-       spin_lock_irqsave(&vp->lock, flags);
-       if (vp->opened) {
-               spin_unlock_irqrestore(&vp->lock, flags);
+       if (vp->opened)
                return -ENXIO;
-       }
        vp->opened = true;
-       spin_unlock_irqrestore(&vp->lock, flags);
 
        vp->bpf = uml_vector_user_bpf(get_bpf_file(vp->parsed));
 
@@ -1387,8 +1381,6 @@ static int vector_net_load_bpf_flash(struct net_device 
*dev,
                return -1;
        }
 
-       spin_lock(&vp->lock);
-
        if (vp->bpf != NULL) {
                if (vp->opened)
                        uml_vector_detach_bpf(vp->fds->rx_fd, vp->bpf);
@@ -1417,15 +1409,12 @@ static int vector_net_load_bpf_flash(struct net_device 
*dev,
        if (vp->opened)
                result = uml_vector_attach_bpf(vp->fds->rx_fd, vp->bpf);
 
-       spin_unlock(&vp->lock);
-
        return result;
 
 free_buffer:
        release_firmware(fw);
 
 flash_fail:
-       spin_unlock(&vp->lock);
        if (vp->bpf != NULL)
                kfree(vp->bpf->filter);
        kfree(vp->bpf);
@@ -1631,7 +1620,6 @@ static void vector_eth_configure(
        INIT_WORK(&vp->reset_tx, vector_reset_tx);
 
        timer_setup(&vp->tl, vector_timer_expire, 0);
-       spin_lock_init(&vp->lock);
 
        /* FIXME */
        dev->netdev_ops = &vector_netdev_ops;
diff --git a/arch/um/drivers/vector_kern.h b/arch/um/drivers/vector_kern.h
index 2a1fa8e0f3e1..806df551be0b 100644
--- a/arch/um/drivers/vector_kern.h
+++ b/arch/um/drivers/vector_kern.h
@@ -71,7 +71,6 @@ struct vector_estats {
 
 struct vector_private {
        struct list_head list;
-       spinlock_t lock;
        struct net_device *dev;
        struct napi_struct              napi    ____cacheline_aligned;
 
-- 
2.45.2