> -----Original Message----- > From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] > On Behalf Of Jo-Philipp Wich > Sent: Dienstag, 7. Juli 2020 15:35 > To: j...@mein.io; openwrt-devel@lists.openwrt.org > Subject: [RFC PATCH] Introduce UCI support for configuring DSA VLAN filter > rules > > This patch series introduces a new package "dsaconfig" which provides the > necessary logic to allow configuration of bridge vlan filter rules for DSA > switches.
Hi, thanks for taking care of that. [...] > - Consider changing the types of the UCI sections from > switch/switch_vlan/switch_port to dsa/dsa_vlan/dsa_port Using new names for DSA would have the benefit that migration would be much easier, as we always knew what the config is meant for (and we wouldn't need another migration_done=1 variable or similar.) However, it might create some code duplication in other tools/packages that alter uci network config. This might be seen as drawback (duplication) or advantage (separate code based on separate setup, as there might be differences anyway). I currently have a weak to moderate tendency for the dsa_* scheme. Best Adrian > > - Investigate potential MTU issues regarding the CPU port > > Jo-Philipp Wich (1): > dsaconfig: introduce package for UCI configuration of VLAN filter > rules > > package/network/config/dsaconfig/Makefile | 40 +++ > .../config/dsaconfig/files/dsaconfig.hotplug | 7 + > .../config/dsaconfig/files/dsaconfig.include | 11 + > .../config/dsaconfig/files/dsaconfig.sh | 296 ++++++++++++++++++ > 4 files changed, 354 insertions(+) > create mode 100644 package/network/config/dsaconfig/Makefile > create mode 100644 > package/network/config/dsaconfig/files/dsaconfig.hotplug > create mode 100755 > package/network/config/dsaconfig/files/dsaconfig.include > create mode 100755 package/network/config/dsaconfig/files/dsaconfig.sh > > -- > 2.20.1 > > > _______________________________________________ > openwrt-devel mailing list > openwrt-devel@lists.openwrt.org > https://lists.openwrt.org/mailman/listinfo/openwrt-devel
openpgp-digital-signature.asc
Description: PGP signature
_______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel
