Bug#950879: bridge-utils: DSA and other VLAN aware bridges needs VLAN filtering

2020-02-09 Thread Santiago Garcia Mantinan
Hi!

> The attached patch adds the missing bridge_vlan_aware aka
> enable VLAN filtering support.

Great, thanks for the patch, I'll take a look at it and do some tests, if
everything looks ok I'll upload a new version soon.

Regards.
-- 
Manty/BestiaTester -> http://manty.net



Bug#950879: bridge-utils: DSA and other VLAN aware bridges needs VLAN filtering

2020-02-07 Thread Benedikt Spranger
Package: bridge-utils
Version: 1.6-2
Severity: wishlist
Tags: patch

Dear Maintainer,

while configuring a DSA supported switch with ifupdown I face the
problem, that VLAN filtering is not set.

Documentation/networking/dsa/configuration.rst in the linux kernel tree
describes what is nessesary to setup such a DSA supported switch.

Except for the

ip link set dev br0 type bridge vlan_filtering 1

the whole configuration can be easiely done by ifupdown like:

auto br0
iface br0 inet static
address 172.16.0.1
netmask 255.255.255.0
bridge_ports lan1 lan2 lan3 lan4 eth0.100
bridge_vlan_aware yes
bridge_maxwait 5
up bridge vlan add dev lan1 vid 100 pvid untagged
up bridge vlan add dev lan2 vid 100 pvid untagged
up bridge vlan add dev lan3 vid 100 pvid untagged
up bridge vlan add dev lan4 vid 100 pvid untagged
up bridge vlan del dev lan1 vid 1
up bridge vlan del dev lan2 vid 1
up bridge vlan del dev lan3 vid 1
up bridge vlan del dev lan4 vid 1

The attached patch adds the missing bridge_vlan_aware aka
enable VLAN filtering support.

Regards
Benedikt Spranger

-- System Information:
Debian Release: bullseye/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386, armhf

Kernel: Linux 5.2.0-2-amd64 (SMP w/4 CPU cores)
Kernel taint flags: TAINT_OOT_MODULE, TAINT_UNSIGNED_MODULE
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8), LANGUAGE=de_DE.\
UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)
LSM: AppArmor: enabled

Versions of packages bridge-utils depends on:
ii  libc6  2.29-10

bridge-utils recommends no packages.

Versions of packages bridge-utils suggests:
ii  ifupdown  0.8.35+b1

-- no debconf information