* Re: [net-next 1/2] bonding: remove duplicate set of flag IFF_MULTICAST
@ 2016-03-16 12:30 张胜举
0 siblings, 0 replies; 4+ messages in thread
From: 张胜举 @ 2016-03-16 12:30 UTC (permalink / raw)
To: 'Nikolay Aleksandrov'
Cc: netdev, gospo, 'Veaceslav Falico', davem, j.vosburgh,
jiri
> 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>
Thanks, Nikolay.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [net-next 0/2] remove duplicate set of flag IFF_MULTICAST
@ 2016-03-16 9:59 Zhang Shengju
2016-03-16 9:59 ` [net-next 1/2] bonding: " Zhang Shengju
0 siblings, 1 reply; 4+ messages in thread
From: Zhang Shengju @ 2016-03-16 9:59 UTC (permalink / raw)
To: davem, j.vosburgh, jiri; +Cc: netdev
This patch series remove duplicate set of flag IFF_MULTICAST.
Zhang Shengju (2):
bonding: remove duplicate set of flag IFF_MULTICAST
team: remove duplicate set of flag IFF_MULTICAST
drivers/net/bonding/bond_main.c | 2 +-
drivers/net/team/team.c | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
--
1.8.3.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* [net-next 1/2] bonding: remove duplicate set of flag IFF_MULTICAST
2016-03-16 9:59 [net-next 0/2] " Zhang Shengju
@ 2016-03-16 9:59 ` Zhang Shengju
2016-03-16 10:06 ` Nikolay Aleksandrov
2016-03-16 14:01 ` Andy Gospodarek
0 siblings, 2 replies; 4+ messages in thread
From: Zhang Shengju @ 2016-03-16 9:59 UTC (permalink / raw)
To: davem, j.vosburgh, jiri; +Cc: netdev
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(-)
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
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [net-next 1/2] bonding: remove duplicate set of flag IFF_MULTICAST
2016-03-16 9:59 ` [net-next 1/2] bonding: " Zhang Shengju
@ 2016-03-16 10:06 ` Nikolay Aleksandrov
2016-03-16 14:01 ` Andy Gospodarek
1 sibling, 0 replies; 4+ messages in thread
From: Nikolay Aleksandrov @ 2016-03-16 10:06 UTC (permalink / raw)
To: Zhang Shengju, davem, j.vosburgh, jiri; +Cc: netdev, gospo, Veaceslav Falico
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>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [net-next 1/2] bonding: remove duplicate set of flag IFF_MULTICAST
2016-03-16 9:59 ` [net-next 1/2] bonding: " Zhang Shengju
2016-03-16 10:06 ` Nikolay Aleksandrov
@ 2016-03-16 14:01 ` Andy Gospodarek
1 sibling, 0 replies; 4+ messages in thread
From: Andy Gospodarek @ 2016-03-16 14:01 UTC (permalink / raw)
To: Zhang Shengju; +Cc: davem, j.vosburgh, jiri, netdev
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
>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-03-16 14:01 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-16 12:30 [net-next 1/2] bonding: remove duplicate set of flag IFF_MULTICAST 张胜举
-- strict thread matches above, loose matches on Subject: below --
2016-03-16 9:59 [net-next 0/2] " Zhang Shengju
2016-03-16 9:59 ` [net-next 1/2] bonding: " Zhang Shengju
2016-03-16 10:06 ` Nikolay Aleksandrov
2016-03-16 14:01 ` Andy Gospodarek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox