Message ID | 1458122356-25021-2-git-send-email-zhangshengju@cmss.chinamobile.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
On 03/16/2016 10:59 AM, Zhang Shengju wrote: > Remove unnecessary set of flag IFF_MULTICAST, since ether_setup > already does this. > > Signed-off-by: Zhang Shengju <zhangshengju@cmss.chinamobile.com> > --- > drivers/net/bonding/bond_main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > There're a few more bonding maintainers, I've added them to the CC list. Reviewed-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
On Wed, Mar 16, 2016 at 09:59:15AM +0000, Zhang Shengju wrote: > Remove unnecessary set of flag IFF_MULTICAST, since ether_setup > already does this. > > Signed-off-by: Zhang Shengju <zhangshengju@cmss.chinamobile.com> Signed-off-by: Andy Gospodarek <gospo@cumulusnetworks.com> > --- > drivers/net/bonding/bond_main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c > index b6236ff..270b39c 100644 > --- a/drivers/net/bonding/bond_main.c > +++ b/drivers/net/bonding/bond_main.c > @@ -4175,7 +4175,7 @@ void bond_setup(struct net_device *bond_dev) > SET_NETDEV_DEVTYPE(bond_dev, &bond_type); > > /* Initialize the device options */ > - bond_dev->flags |= IFF_MASTER|IFF_MULTICAST; > + bond_dev->flags |= IFF_MASTER; > bond_dev->priv_flags |= IFF_BONDING | IFF_UNICAST_FLT | IFF_NO_QUEUE; > bond_dev->priv_flags &= ~(IFF_XMIT_DST_RELEASE | IFF_TX_SKB_SHARING); > > -- > 1.8.3.1 > > >
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index b6236ff..270b39c 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c @@ -4175,7 +4175,7 @@ void bond_setup(struct net_device *bond_dev) SET_NETDEV_DEVTYPE(bond_dev, &bond_type); /* Initialize the device options */ - bond_dev->flags |= IFF_MASTER|IFF_MULTICAST; + bond_dev->flags |= IFF_MASTER; bond_dev->priv_flags |= IFF_BONDING | IFF_UNICAST_FLT | IFF_NO_QUEUE; bond_dev->priv_flags &= ~(IFF_XMIT_DST_RELEASE | IFF_TX_SKB_SHARING);
Remove unnecessary set of flag IFF_MULTICAST, since ether_setup already does this. Signed-off-by: Zhang Shengju <zhangshengju@cmss.chinamobile.com> --- drivers/net/bonding/bond_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)