vrf: remove unnecessary duplicate check
authorNikolay Aleksandrov <nikolay@cumulusnetworks.com>
Wed, 19 Aug 2015 03:27:08 +0000 (06:27 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 20 Aug 2015 20:01:57 +0000 (13:01 -0700)
The upper/lower functions already check for duplicate slaves so no need
to do it again.

Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/vrf.c

index 01dc91562a889f4e5ec6996a91b29dd76112a304..9907550ff640519fa65eccb109e0e7bae8c0b427 100644 (file)
@@ -352,7 +352,6 @@ static int do_vrf_add_slave(struct net_device *dev, struct net_device *port_dev)
 {
        struct net_vrf_dev *vrf_ptr = kmalloc(sizeof(*vrf_ptr), GFP_KERNEL);
        struct slave *slave = kzalloc(sizeof(*slave), GFP_KERNEL);
-       struct slave *duplicate_slave;
        struct net_vrf *vrf = netdev_priv(dev);
        struct slave_queue *queue = &vrf->queue;
        int ret = -ENOMEM;
@@ -361,16 +360,9 @@ static int do_vrf_add_slave(struct net_device *dev, struct net_device *port_dev)
                goto out_fail;
 
        slave->dev = port_dev;
-
        vrf_ptr->ifindex = dev->ifindex;
        vrf_ptr->tb_id = vrf->tb_id;
 
-       duplicate_slave = __vrf_find_slave_dev(queue, port_dev);
-       if (duplicate_slave) {
-               ret = -EBUSY;
-               goto out_fail;
-       }
-
        __vrf_insert_slave(queue, slave);
 
        /* register the packet handler for slave ports */