[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <8628b57e-3087-4fb5-9e8a-03a1b4b8c31a@intel.com>
Date: Thu, 7 Mar 2024 12:41:36 +0100
From: Wojciech Drewek <wojciech.drewek@...el.com>
To: Lukasz Majewski <lukma@...x.de>, <netdev@...r.kernel.org>
CC: Andrew Lunn <andrew@...n.ch>, Eric Dumazet <edumazet@...gle.com>, "Florian
Fainelli" <f.fainelli@...il.com>, Vladimir Oltean <olteanv@...il.com>, "David
S. Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>, "Oleksij
Rempel" <o.rempel@...gutronix.de>, <Tristram.Ha@...rochip.com>, "Sebastian
Andrzej Siewior" <bigeasy@...utronix.de>, Paolo Abeni <pabeni@...hat.com>,
Ravi Gunasekaran <r-gunasekaran@...com>, Simon Horman <horms@...nel.org>,
Nikita Zhandarovich <n.zhandarovich@...tech.ru>, Murali Karicheri
<m-karicheri2@...com>, Dan Carpenter <dan.carpenter@...aro.org>, Ziyang Xuan
<william.xuanziyang@...wei.com>, Kristian Overskeid <koverskeid@...il.com>,
Matthieu Baerts <matttbe@...nel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] net: hsr: Provide RedBox support
On 07.03.2024 12:33, Lukasz Majewski wrote:
> Introduce RedBox support (HSR-SAN to be more precise) for HSR networks.
> The 'Mode U' has been implemented without optimizations for traffic
> reduction - i.e. only sending HSR supervisory frames to Port C and not
> forwarding to HSR ring frames addressed to Port C.
>
> The corresponding patch to modify iptable2 sources has already been sent:
> https://lore.kernel.org/netdev/20240226124110.37892211@hermes.local/T/
>
> Testing procedure:
> ------------------
> The EVB-KSZ9477 has been used for testing on net-next branch
> (SHA1: 709776ea8562).
>
> Ports 4/5 were used for SW managed HSR (hsr1) as first hsr0 for ports 1/2
> (with HW offloading for ksz9477) was created. Port 3 has been used as
> interlink port (single USB-ETH dongle).
>
> Configuration - RedBox (EVB-KSZ9477):
> ifconfig lan1 down;ifconfig lan2 down
> ip link add name hsr0 type hsr slave1 lan1 slave2 lan2 supervision 45 version 1
> ip link add name hsr1 type hsr slave1 lan4 slave2 lan5 interlink lan3 supervision 45 version 1
> ifconfig lan4 up;ifconfig lan5 up
> ifconfig lan3 up
> ifconfig hsr1 192.168.0.11 up
>
> Configuration - DAN-H (EVB-KSZ9477):
>
> ifconfig lan1 down;ifconfig lan2 down
> ip link add name hsr0 type hsr slave1 lan1 slave2 lan2 supervision 45 version 1
> ip link add name hsr1 type hsr slave1 lan4 slave2 lan5 supervision 45 version 1
> ifconfig lan4 up;ifconfig lan5 up
> ifconfig hsr1 192.168.0.12 up
>
> This approach uses only SW based HSR devices (hsr1).
>
> -------------- ----------------- ------------
> DAN-H Port5 | <------> | Port5 | |
> Port4 | <------> | Port4 Port3 | <---> | PC
> | | (RedBox) | | (USB-ETH)
> EVB-KSZ9477 | | EVB-KSZ9477 | |
> -------------- ----------------- ------------
>
> Signed-off-by: Lukasz Majewski <lukma@...x.de>
Reviewed-by: Wojciech Drewek <wojciech.drewek@...el.com>
>
> ---
> Changes for v2:
>
> - Add deleting of HSR_PT_INTERLINK node to hsr_del_ports()
> - Rewrite handle_std_frame() to awoid code duplication
> - Fix reverse christmas tree in hsr_prune_proxy_nodes() as well as
> remove stale node indication as interlink doesn't need this detection
> (only check if node is inactive for more than HSR_PROXY_NODE_FORGET_TIME
> is required)
> - Rewrite commit message
> ---
> include/uapi/linux/if_link.h | 1 +
> net/hsr/hsr_device.c | 36 ++++++++++++++++++++--
> net/hsr/hsr_device.h | 4 +--
> net/hsr/hsr_forward.c | 58 ++++++++++++++++++++++++++++++------
> net/hsr/hsr_framereg.c | 52 ++++++++++++++++++++++++++++++++
> net/hsr/hsr_framereg.h | 6 ++++
> net/hsr/hsr_main.h | 7 +++++
> net/hsr/hsr_netlink.c | 29 ++++++++++++++++--
> net/hsr/hsr_slave.c | 1 +
> 9 files changed, 178 insertions(+), 16 deletions(-)
>
<...>
Powered by blists - more mailing lists