Re: [PATCH net-next v2 1/6] net: bridge: Publish bridge accessor functions

2018-04-27 Thread Petr Machata
Stephen Hemminger writes: > On Fri, 27 Apr 2018 18:11:06 +0300 > Ido Schimmel wrote: > >> +int br_vlan_pvid_rtnl(const struct net_device *dev, u16 *p_pvid) >> +{ >> +struct net_bridge_vlan_group *vg; >> + >> +ASSERT_RTNL(); >> +if (netif_is_bridge_master(dev)) >> +vg = br

Re: [PATCH net-next v2 1/6] net: bridge: Publish bridge accessor functions

2018-04-27 Thread Stephen Hemminger
On Fri, 27 Apr 2018 18:11:06 +0300 Ido Schimmel wrote: > +int br_vlan_pvid_rtnl(const struct net_device *dev, u16 *p_pvid) > +{ > + struct net_bridge_vlan_group *vg; > + > + ASSERT_RTNL(); > + if (netif_is_bridge_master(dev)) > + vg = br_vlan_group(netdev_priv(dev)); > +

Re: [PATCH net-next v2 1/6] net: bridge: Publish bridge accessor functions

2018-04-27 Thread Nikolay Aleksandrov
On 27/04/18 19:08, Petr Machata wrote: > Nikolay Aleksandrov writes: > >> On 27/04/18 18:11, Ido Schimmel wrote: >>> From: Petr Machata >>> >>> Add a couple new functions to allow querying FDB and vlan settings of a >>> bridge. >>> >>> Signed-off-by: Petr Machata >>> Signed-off-by: Ido Schimmel

Re: [PATCH net-next v2 1/6] net: bridge: Publish bridge accessor functions

2018-04-27 Thread Petr Machata
Nikolay Aleksandrov writes: > On 27/04/18 18:11, Ido Schimmel wrote: >> From: Petr Machata >> >> Add a couple new functions to allow querying FDB and vlan settings of a >> bridge. >> >> Signed-off-by: Petr Machata >> Signed-off-by: Ido Schimmel >> --- >> include/linux/if_bridge.h | 28 +

Re: [PATCH net-next v2 1/6] net: bridge: Publish bridge accessor functions

2018-04-27 Thread Nikolay Aleksandrov
On 27/04/18 18:11, Ido Schimmel wrote: > From: Petr Machata > > Add a couple new functions to allow querying FDB and vlan settings of a > bridge. > > Signed-off-by: Petr Machata > Signed-off-by: Ido Schimmel > --- > include/linux/if_bridge.h | 28 > net/bridge/br_

[PATCH net-next v2 1/6] net: bridge: Publish bridge accessor functions

2018-04-27 Thread Ido Schimmel
From: Petr Machata Add a couple new functions to allow querying FDB and vlan settings of a bridge. Signed-off-by: Petr Machata Signed-off-by: Ido Schimmel --- include/linux/if_bridge.h | 28 net/bridge/br_fdb.c | 22 ++ net/bridge/br_priv