2023-01-20 09:50:45

by Alexandru Tachici

[permalink] [raw]
Subject: [net] net: ethernet: adi: adin1110: Fix multicast offloading

Driver marked broadcast/multicast frames as offloaded incorrectly.
Mark them as offloaded only when HW offloading has been enabled.
This should happen only for ADIN2111 when both ports are bridged
by the software.

Fixes: bc93e19d088b ("net: ethernet: adi: Add ADIN1110 support")
Signed-off-by: Alexandru Tachici <[email protected]>
---
drivers/net/ethernet/adi/adin1110.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/adi/adin1110.c b/drivers/net/ethernet/adi/adin1110.c
index 0805f249fff2..c26b8597945b 100644
--- a/drivers/net/ethernet/adi/adin1110.c
+++ b/drivers/net/ethernet/adi/adin1110.c
@@ -356,7 +356,7 @@ static int adin1110_read_fifo(struct adin1110_port_priv *port_priv)

if ((port_priv->flags & IFF_ALLMULTI && rxb->pkt_type == PACKET_MULTICAST) ||
(port_priv->flags & IFF_BROADCAST && rxb->pkt_type == PACKET_BROADCAST))
- rxb->offload_fwd_mark = 1;
+ rxb->offload_fwd_mark = port_priv->priv->forwarding;

netif_rx(rxb);

--
2.34.1


2023-01-24 01:45:50

by Andrew Lunn

[permalink] [raw]
Subject: Re: [net] net: ethernet: adi: adin1110: Fix multicast offloading

On Fri, Jan 20, 2023 at 11:08:46AM +0200, Alexandru Tachici wrote:
> Driver marked broadcast/multicast frames as offloaded incorrectly.
> Mark them as offloaded only when HW offloading has been enabled.
> This should happen only for ADIN2111 when both ports are bridged
> by the software.
>
> Fixes: bc93e19d088b ("net: ethernet: adi: Add ADIN1110 support")
> Signed-off-by: Alexandru Tachici <[email protected]>

Reviewed-by: Andrew Lunn <[email protected]>

Andrew

2023-01-24 05:51:17

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [net] net: ethernet: adi: adin1110: Fix multicast offloading

Hello:

This patch was applied to netdev/net.git (master)
by Jakub Kicinski <[email protected]>:

On Fri, 20 Jan 2023 11:08:46 +0200 you wrote:
> Driver marked broadcast/multicast frames as offloaded incorrectly.
> Mark them as offloaded only when HW offloading has been enabled.
> This should happen only for ADIN2111 when both ports are bridged
> by the software.
>
> Fixes: bc93e19d088b ("net: ethernet: adi: Add ADIN1110 support")
> Signed-off-by: Alexandru Tachici <[email protected]>
>
> [...]

Here is the summary with links:
- [net] net: ethernet: adi: adin1110: Fix multicast offloading
https://git.kernel.org/netdev/net/c/8a4f6d023221

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