net/tap: remove NO-ARP setting
authorPascal Mazon <pascal.mazon@6wind.com>
Wed, 15 Mar 2017 14:48:12 +0000 (15:48 +0100)
committerFerruh Yigit <ferruh.yigit@intel.com>
Tue, 4 Apr 2017 16:59:40 +0000 (18:59 +0200)
There is no reason not to support ARP on a tap netdevice. Remove
IFF_NOARP flags.
Focus on IFF_UP when a link status change is required.

Fixes: f457b472b1f2 ("net/tap: add link up and down operations")

Signed-off-by: Pascal Mazon <pascal.mazon@6wind.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
drivers/net/tap/rte_eth_tap.c

index a5cbb23..fcf4a1d 100644 (file)
@@ -375,7 +375,7 @@ tap_link_set_down(struct rte_eth_dev *dev)
        struct pmd_internals *pmd = dev->data->dev_private;
 
        dev->data->dev_link.link_status = ETH_LINK_DOWN;
-       return tap_link_set_flags(pmd, IFF_UP | IFF_NOARP, 0);
+       return tap_link_set_flags(pmd, IFF_UP, 0);
 }
 
 static int
@@ -384,7 +384,7 @@ tap_link_set_up(struct rte_eth_dev *dev)
        struct pmd_internals *pmd = dev->data->dev_private;
 
        dev->data->dev_link.link_status = ETH_LINK_UP;
-       return tap_link_set_flags(pmd, IFF_UP | IFF_NOARP, 1);
+       return tap_link_set_flags(pmd, IFF_UP, 1);
 }
 
 static int