mbox series

[iproute2-next,0/2] bridge: vlan: add support for mcast_router option

Message ID 20210901103816.1163765-1-razor@blackwall.org (mailing list archive)
Headers show
Series bridge: vlan: add support for mcast_router option | expand

Message

Nikolay Aleksandrov Sept. 1, 2021, 10:38 a.m. UTC
From: Nikolay Aleksandrov <nikolay@nvidia.com>

Hi,
This set adds support for vlan port/bridge multicast router option. It is
similar to the already existing bridge-wide mcast_router control. Patch 01
moves attribute adding and parsing together for vlan option setting,
similar to global vlan option setting. It simplifies adding new options
because we can avoid reserved values and additional checks. Patch 02
adds the new mcast_router option and updates the related man page.

Example:
 # mark port ens16 as a permanent mcast router for vlan 100
 $ bridge vlan set dev ens16 vid 100 mcast_router 2
 # disable mcast router for port ens16 and vlan 200
 $ bridge vlan set dev ens16 vid 200 mcast_router 0
 $ bridge -d vlan show
 port              vlan-id
 ens16             1 PVID Egress Untagged
                     state forwarding mcast_router 1
                   100
                     state forwarding mcast_router 2
                   200
                     state forwarding mcast_router 0

Note that this set depends on the latest kernel uapi headers.

Thanks,
 Nik


Nikolay Aleksandrov (2):
  bridge: vlan: set vlan option attributes while parsing
  bridge: vlan: add support for mcast_router option

 bridge/vlan.c     | 70 +++++++++++++++++++++++++++++------------------
 man/man8/bridge.8 | 29 +++++++++++++++++++-
 2 files changed, 71 insertions(+), 28 deletions(-)

Comments

David Ahern Sept. 6, 2021, 11:04 p.m. UTC | #1
On 9/1/21 4:38 AM, Nikolay Aleksandrov wrote:
> From: Nikolay Aleksandrov <nikolay@nvidia.com>
> 
> Hi,
> This set adds support for vlan port/bridge multicast router option. It is
> similar to the already existing bridge-wide mcast_router control. Patch 01
> moves attribute adding and parsing together for vlan option setting,
> similar to global vlan option setting. It simplifies adding new options
> because we can avoid reserved values and additional checks. Patch 02
> adds the new mcast_router option and updates the related man page.
> 
> Example:
>  # mark port ens16 as a permanent mcast router for vlan 100
>  $ bridge vlan set dev ens16 vid 100 mcast_router 2
>  # disable mcast router for port ens16 and vlan 200
>  $ bridge vlan set dev ens16 vid 200 mcast_router 0
>  $ bridge -d vlan show
>  port              vlan-id
>  ens16             1 PVID Egress Untagged
>                      state forwarding mcast_router 1
>                    100
>                      state forwarding mcast_router 2
>                    200
>                      state forwarding mcast_router 0
> 
> Note that this set depends on the latest kernel uapi headers.
> 

applied to iproute2-next.