mbox series

[net,0/6] FDB fixes for NXP SJA1105

Message ID 20210730171815.1773287-1-vladimir.oltean@nxp.com
Headers show
Series FDB fixes for NXP SJA1105 | expand

Message

Vladimir Oltean July 30, 2021, 5:18 p.m. UTC
I have some upcoming patches that make heavy use of statically installed
FDB entries, and when testing them on SJA1105P/Q/R/S and SJA1110, it
became clear that these switches do not behave reliably at all.

- On SJA1110, a static FDB entry cannot be installed at all
- On SJA1105P/Q/R/S, it is very picky about the inner/outer VLAN type
- Dynamically learned entries will make us not install static ones, or
  even if we do, they might not take effect

Patch 5/6 has a conflict with net-next (sorry), the commit message of
that patch describes how to deal with it. Thanks.

Vladimir Oltean (6):
  net: dsa: sja1105: fix static FDB writes for SJA1110
  net: dsa: sja1105: overwrite dynamic FDB entries with static ones in
    .port_fdb_add
  net: dsa: sja1105: invalidate dynamic FDB entries learned concurrently
    with statically added ones
  net: dsa: sja1105: ignore the FDB entry for unknown multicast when
    adding a new address
  net: dsa: sja1105: be stateless with FDB entries on
    SJA1105P/Q/R/S/SJA1110 too
  net: dsa: sja1105: match FDB entries regardless of inner/outer VLAN
    tag

 .../net/dsa/sja1105/sja1105_dynamic_config.c  | 27 +++---
 drivers/net/dsa/sja1105/sja1105_main.c        | 94 ++++++++++++++-----
 2 files changed, 84 insertions(+), 37 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Aug. 2, 2021, 1:30 p.m. UTC | #1
Hello:

This series was applied to netdev/net.git (refs/heads/master):

On Fri, 30 Jul 2021 20:18:09 +0300 you wrote:
> I have some upcoming patches that make heavy use of statically installed

> FDB entries, and when testing them on SJA1105P/Q/R/S and SJA1110, it

> became clear that these switches do not behave reliably at all.

> 

> - On SJA1110, a static FDB entry cannot be installed at all

> - On SJA1105P/Q/R/S, it is very picky about the inner/outer VLAN type

> - Dynamically learned entries will make us not install static ones, or

>   even if we do, they might not take effect

> 

> [...]


Here is the summary with links:
  - [net,1/6] net: dsa: sja1105: fix static FDB writes for SJA1110
    https://git.kernel.org/netdev/net/c/cb81698fddbc
  - [net,2/6] net: dsa: sja1105: overwrite dynamic FDB entries with static ones in .port_fdb_add
    https://git.kernel.org/netdev/net/c/e11e865bf84e
  - [net,3/6] net: dsa: sja1105: invalidate dynamic FDB entries learned concurrently with statically added ones
    https://git.kernel.org/netdev/net/c/6c5fc159e092
  - [net,4/6] net: dsa: sja1105: ignore the FDB entry for unknown multicast when adding a new address
    https://git.kernel.org/netdev/net/c/728db843df88
  - [net,5/6] net: dsa: sja1105: be stateless with FDB entries on SJA1105P/Q/R/S/SJA1110 too
    https://git.kernel.org/netdev/net/c/589918df9322
  - [net,6/6] net: dsa: sja1105: match FDB entries regardless of inner/outer VLAN tag
    https://git.kernel.org/netdev/net/c/47c2c0c23121

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html