[PATCH v3 net-next 00/15] Traffic support for dsa_8021q in vlan_filtering=1 mode

From: Vladimir Oltean
Date: Tue May 12 2020 - 13:20:47 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.

v2 was submitted here:
https://patchwork.ozlabs.org/project/netdev/cover/20200511135338.20263-1-olteanv@xxxxxxxxx/

v1 was submitted here:
https://patchwork.ozlabs.org/project/netdev/cover/20200510164255.19322-1-olteanv@xxxxxxxxx/

Changes in v3:
Patch 01/15:
- Rename again to configure_vlan_while_not_filtering, and add a helper
function for skipping VLAN configuration.
Patch 03/15:
- Remove sja1105_can_use_vlan_as_tags from driver code.
Patch 06/15:
- Adapt sja1105 driver to the second variable name change.
Patch 08/15:
- Provide an implementation of sja1105_can_use_vlan_as_tags as part of
the tagger and not as part of the switch driver. So we have to look at
the skb only, and not at the VLAN awareness state.

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 | 1120 +++++++++++++++--
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 | 3 +
include/net/dsa.h | 7 +
net/dsa/dsa_priv.h | 1 +
net/dsa/port.c | 14 +
net/dsa/slave.c | 8 +-
net/dsa/tag_8021q.c | 108 +-
net/dsa/tag_sja1105.c | 51 +-
17 files changed, 1522 insertions(+), 260 deletions(-)
create mode 100644 Documentation/networking/devlink-params-sja1105.txt

--
2.17.1