[PATCH v2 net-next 00/15] Traffic support for dsa_8021q in vlan_filtering=1 mode
From: Vladimir Oltean
Date: Mon May 11 2020 - 09:53:50 EST
From: Vladimir Oltean <vladimir.oltean@xxxxxxx>
This series is an attempt to support as much as possible in terms of
traffic I/O from the network stack with the only dsa_8021q user thus
far, sja1105.
The hardware doesn't support pushing a second VLAN tag to packets that
are already tagged, so our only option is to combine the dsa_8021q with
the user tag into a single tag and decode that on the CPU.
The assumption is that there is a type of use cases for which 7 VLANs
per port are more than sufficient, and that there's another type of use
cases where the full 4096 entries are barely enough. Those use cases are
very different from one another, so I prefer trying to give both the
best experience by creating this best_effort_vlan_filtering knob to
select the mode in which they want to operate in.
v1 was submitted here:
https://patchwork.ozlabs.org/project/netdev/cover/20200510164255.19322-1-olteanv@xxxxxxxxx/
Changes in v2:
Patch 01/15:
- Rename variable from vlan_bridge_vtu to configure_vlans_while_disabled.
Patch 03/15:
- Be much more thorough, and make sure that things like virtual links
and FDB operations still work properly.
Patch 05/15:
- Free the vlan lists on teardown.
- Simplify sja1105_classify_vlan: only look at priv->expect_dsa_8021q.
- Keep vid 1 in the list of dsa_8021q VLANs, to make sure that untagged
packets transmitted from the stack, like PTP, continue to work in
VLAN-unaware mode.
Patch 06/15:
- Adapt to vlan_bridge_vtu variable name change.
Patch 11/15:
- In sja1105_best_effort_vlan_filtering_set, get the vlan_filtering
value of each port instead of just one time for port 0. Normally this
shouldn't matter, but it avoids issues when port 0 is disabled in
device tree.
Patch 14/14:
- Only do anything in sja1105_build_subvlans and in
sja1105_build_crosschip_subvlans when operating in
SJA1105_VLAN_BEST_EFFORT state. This avoids installing VLAN retagging
rules in unaware mode, which would cost us a penalty in terms of
usable frame memory.
Russell King (1):
net: dsa: provide an option for drivers to always receive bridge VLANs
Vladimir Oltean (14):
net: dsa: tag_8021q: introduce a vid_is_dsa_8021q helper
net: dsa: sja1105: keep the VLAN awareness state in a driver variable
net: dsa: sja1105: deny alterations of dsa_8021q VLANs from the bridge
net: dsa: sja1105: save/restore VLANs using a delta commit method
net: dsa: sja1105: allow VLAN configuration from the bridge in all
states
net: dsa: sja1105: exit sja1105_vlan_filtering when called multiple
times
net: dsa: sja1105: prepare tagger for handling DSA tags and VLAN
simultaneously
net: dsa: tag_8021q: support up to 8 VLANs per port using sub-VLANs
net: dsa: tag_sja1105: implement sub-VLAN decoding
net: dsa: sja1105: add a new best_effort_vlan_filtering devlink
parameter
net: dsa: sja1105: add packing ops for the Retagging Table
net: dsa: sja1105: implement a common frame memory partitioning
function
net: dsa: sja1105: implement VLAN retagging for dsa_8021q sub-VLANs
docs: net: dsa: sja1105: document the best_effort_vlan_filtering
option
.../networking/devlink-params-sja1105.txt | 27 +
Documentation/networking/dsa/sja1105.rst | 211 ++-
drivers/net/dsa/sja1105/sja1105.h | 29 +
.../net/dsa/sja1105/sja1105_dynamic_config.c | 33 +
drivers/net/dsa/sja1105/sja1105_main.c | 1128 +++++++++++++++--
drivers/net/dsa/sja1105/sja1105_spi.c | 6 +
.../net/dsa/sja1105/sja1105_static_config.c | 62 +-
.../net/dsa/sja1105/sja1105_static_config.h | 16 +
drivers/net/dsa/sja1105/sja1105_vl.c | 44 +-
include/linux/dsa/8021q.h | 42 +-
include/linux/dsa/sja1105.h | 5 +
include/net/dsa.h | 7 +
net/dsa/slave.c | 12 +-
net/dsa/tag_8021q.c | 108 +-
net/dsa/tag_sja1105.c | 38 +-
15 files changed, 1508 insertions(+), 260 deletions(-)
create mode 100644 Documentation/networking/devlink-params-sja1105.txt
--
2.17.1