Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1473579

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

From Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Newsgroups linux.kernel
Subject Re: [PATCH net-next 1/3] net: dsa: add MDB support
Date 2016-08-31 17:00 +0200
Message-ID <scf9D-3I8-13@gated-at.bofh.it> (permalink)
References <sbBvz-3DB-7@gated-at.bofh.it> <sbBvA-3DB-15@gated-at.bofh.it> <scdKy-2UO-25@gated-at.bofh.it> <sceGC-3xO-47@gated-at.bofh.it> <sceQi-3B0-25@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

Hi Andrew,

Andrew Lunn <andrew@lunn.ch> 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 group' for a L2 entity,
> unless it is already widespread used so in the bridge code. If the
> bridge code does consider L2 a 'multciast group', fine, lets document
> it so. But if the bridge code only uses 'multciast group' to mean L3,
> we should not use it here for L2.

I am not quite sure about that. I can see that SWITCHDEV_OBJ_ID_PORT_MDB
is only crafted in net/bridge/br_mdb.c:__br_mdb_notify().

What should we do if the bridge code consider a multicast group as L3?

Thanks,

        Vivien

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH net-next 0/3] net: dsa: add MDB support Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-08-29 22:40 +0200
  [PATCH net-next 2/3] net: dsa: mv88e6xxx: make switchdev DB ops generic Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-08-29 22:40 +0200
    Re: [PATCH net-next 2/3] net: dsa: mv88e6xxx: make switchdev DB ops  generic Andrew Lunn <andrew@lunn.ch> - 2016-08-31 15:50 +0200
      Re: [PATCH net-next 2/3] net: dsa: mv88e6xxx: make switchdev DB ops generic Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-08-31 16:40 +0200
        Re: [PATCH net-next 2/3] net: dsa: mv88e6xxx: make switchdev DB ops  generic Andrew Lunn <andrew@lunn.ch> - 2016-08-31 17:00 +0200
          Re: [PATCH net-next 2/3] net: dsa: mv88e6xxx: make switchdev DB ops generic Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-08-31 17:10 +0200
  [PATCH net-next 1/3] net: dsa: add MDB support Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-08-29 22:40 +0200
    Re: [PATCH net-next 1/3] net: dsa: add MDB support Andrew Lunn <andrew@lunn.ch> - 2016-08-31 15:30 +0200
      Re: [PATCH net-next 1/3] net: dsa: add MDB support Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-08-31 16:30 +0200
        Re: [PATCH net-next 1/3] net: dsa: add MDB support Andrew Lunn <andrew@lunn.ch> - 2016-08-31 16:40 +0200
          Re: [PATCH net-next 1/3] net: dsa: add MDB support Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2016-08-31 17:00 +0200
            Re: [PATCH net-next 1/3] net: dsa: add MDB support Andrew Lunn <andrew@lunn.ch> - 2016-08-31 17:10 +0200

csiph-web