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 of 'multicast gr

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 i

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 | 23 +++ >> include/net/dsa.h

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| 16 +++ > net/dsa/slave.

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

2016-08-29 Thread Vivien Didelot
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| 16 +++ net/dsa/slave.c | 55 3 files chan