mbox series

[RESEND,v3,net-next,00/12] LAG offload for Ocelot DSA switches

Message ID 20210205220221.255646-1-olteanv@gmail.com
Headers show
Series LAG offload for Ocelot DSA switches | expand

Message

Vladimir Oltean Feb. 5, 2021, 10:02 p.m. UTC
From: Vladimir Oltean <vladimir.oltean@nxp.com>

This patch series reworks the ocelot switchdev driver such that it could
share the same implementation for LAG offload as the felix DSA driver.

Testing has been done in the following topology:

         +----------------------------------+
         | Board 1         br0              |
         |             +---------+          |
         |            /           \         |
         |            |           |         |
         |            |         bond0       |
         |            |        +-----+      |
         |            |       /       \     |
         |  eno0     swp0    swp1    swp2   |
         +---|--------|-------|-------|-----+
             |        |       |       |
             +--------+       |       |
               Cable          |       |
                         Cable|       |Cable
               Cable          |       |
             +--------+       |       |
             |        |       |       |
         +---|--------|-------|-------|-----+
         |  eno0     swp0    swp1    swp2   |
         |            |       \       /     |
         |            |        +-----+      |
         |            |         bond0       |
         |            |           |         |
         |            \           /         |
         |             +---------+          |
         | Board 2         br0              |
         +----------------------------------+

The same script can be run on both Board 1 and Board 2 to set this up:

ip link del bond0
ip link add bond0 type bond mode balance-xor miimon 1
OR
ip link add bond0 type bond mode 802.3ad
ip link set swp1 down && ip link set swp1 master bond0 && ip link set swp1 up
ip link set swp2 down && ip link set swp2 master bond0 && ip link set swp2 up
ip link del br0
ip link add br0 type bridge
ip link set bond0 master br0
ip link set swp0 master br0

Then traffic can be tested between eno0 of Board 1 and eno0 of Board 2.

Vladimir Oltean (12):
  net: mscc: ocelot: rename ocelot_netdevice_port_event to
    ocelot_netdevice_changeupper
  net: mscc: ocelot: use a switch-case statement in
    ocelot_netdevice_event
  net: mscc: ocelot: don't refuse bonding interfaces we can't offload
  net: mscc: ocelot: use ipv6 in the aggregation code
  net: mscc: ocelot: set up the bonding mask in a way that avoids a
    net_device
  net: mscc: ocelot: avoid unneeded "lp" variable in LAG join
  net: mscc: ocelot: set up logical port IDs centrally
  net: mscc: ocelot: drop the use of the "lags" array
  net: mscc: ocelot: rename aggr_count to num_ports_in_lag
  net: mscc: ocelot: rebalance LAGs on link up/down events
  net: dsa: make assisted_learning_on_cpu_port bypass offloaded LAG
    interfaces
  net: dsa: felix: propagate the LAG offload ops towards the ocelot lib

 drivers/net/dsa/ocelot/felix.c         |  32 ++++
 drivers/net/ethernet/mscc/ocelot.c     | 206 ++++++++++++++-----------
 drivers/net/ethernet/mscc/ocelot.h     |   4 -
 drivers/net/ethernet/mscc/ocelot_net.c | 131 ++++++++++------
 include/soc/mscc/ocelot.h              |  11 +-
 net/dsa/dsa_priv.h                     |  13 ++
 net/dsa/slave.c                        |   8 +
 7 files changed, 262 insertions(+), 143 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Feb. 6, 2021, 11 p.m. UTC | #1
Hello:

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

On Sat,  6 Feb 2021 00:02:09 +0200 you wrote:
> From: Vladimir Oltean <vladimir.oltean@nxp.com>
> 
> This patch series reworks the ocelot switchdev driver such that it could
> share the same implementation for LAG offload as the felix DSA driver.
> 
> Testing has been done in the following topology:
> 
> [...]

Here is the summary with links:
  - [RESEND,v3,net-next,01/12] net: mscc: ocelot: rename ocelot_netdevice_port_event to ocelot_netdevice_changeupper
    https://git.kernel.org/netdev/net-next/c/662981bbda29
  - [RESEND,v3,net-next,02/12] net: mscc: ocelot: use a switch-case statement in ocelot_netdevice_event
    https://git.kernel.org/netdev/net-next/c/41e66fa28fef
  - [RESEND,v3,net-next,03/12] net: mscc: ocelot: don't refuse bonding interfaces we can't offload
    https://git.kernel.org/netdev/net-next/c/583cbbe3eed9
  - [RESEND,v3,net-next,04/12] net: mscc: ocelot: use ipv6 in the aggregation code
    https://git.kernel.org/netdev/net-next/c/f79c20c81723
  - [RESEND,v3,net-next,05/12] net: mscc: ocelot: set up the bonding mask in a way that avoids a net_device
    https://git.kernel.org/netdev/net-next/c/b80af659699d
  - [RESEND,v3,net-next,06/12] net: mscc: ocelot: avoid unneeded "lp" variable in LAG join
    https://git.kernel.org/netdev/net-next/c/2e9f4afadc70
  - [RESEND,v3,net-next,07/12] net: mscc: ocelot: set up logical port IDs centrally
    https://git.kernel.org/netdev/net-next/c/2527f2e88fba
  - [RESEND,v3,net-next,08/12] net: mscc: ocelot: drop the use of the "lags" array
    https://git.kernel.org/netdev/net-next/c/528d3f190c98
  - [RESEND,v3,net-next,09/12] net: mscc: ocelot: rename aggr_count to num_ports_in_lag
    https://git.kernel.org/netdev/net-next/c/21357b614d3f
  - [RESEND,v3,net-next,10/12] net: mscc: ocelot: rebalance LAGs on link up/down events
    https://git.kernel.org/netdev/net-next/c/23ca3b727ee6
  - [RESEND,v3,net-next,11/12] net: dsa: make assisted_learning_on_cpu_port bypass offloaded LAG interfaces
    https://git.kernel.org/netdev/net-next/c/a324d3d48fb3
  - [RESEND,v3,net-next,12/12] net: dsa: felix: propagate the LAG offload ops towards the ocelot lib
    https://git.kernel.org/netdev/net-next/c/8fe6832e96ac

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