Re: [PATCH net-next 1/3] net: dsa: add MDB support

2016-08-31 Thread Andrew Lunn
> What should we do if the bridge code consider a multicast group as L3? Hi Vivien Just drop the word 'group' from the Documentation. And there appears to be one comment in the code which should be changed. Andrew

Re: [PATCH net-next 1/3] net: dsa: add MDB support

2016-08-31 Thread Vivien Didelot
Hi Andrew, Andrew Lunn writes: >> > Does the bridge code use multicast group when referring to L2? >> >> The switchdev MDB attribute is very similary to the FDB attribute. It >> contains the VLAN ID and the Ethernet MAC address of the group. > > I'm just trying to avoid the use

Re: [PATCH net-next 1/3] net: dsa: add MDB support

2016-08-31 Thread Andrew Lunn
> > Does the bridge code use multicast group when referring to L2? > > The switchdev MDB attribute is very similary to the FDB attribute. It > contains the VLAN ID and the Ethernet MAC address of the group. Hi Vivien I'm just trying to avoid the use of 'multicast group' for a L2 entity, unless

Re: [PATCH net-next 1/3] net: dsa: add MDB support

2016-08-31 Thread Vivien Didelot
Hi Andrew, Andrew Lunn writes: > On Mon, Aug 29, 2016 at 04:32:44PM -0400, Vivien Didelot wrote: >> Add SWITCHDEV_OBJ_ID_PORT_MDB support to the DSA layer. >> >> Signed-off-by: Vivien Didelot >> --- >> Documentation/networking/dsa/dsa.txt

Re: [PATCH net-next 1/3] net: dsa: add MDB support

2016-08-31 Thread Andrew Lunn
On Mon, Aug 29, 2016 at 04:32:44PM -0400, Vivien Didelot wrote: > Add SWITCHDEV_OBJ_ID_PORT_MDB support to the DSA layer. > > Signed-off-by: Vivien Didelot > --- > Documentation/networking/dsa/dsa.txt | 23 +++ > include/net/dsa.h