Re: [PATCH] net: dsa: vsc73xx: fix MTU configuration

From: Vladimir Oltean
Date: Wed Jun 28 2023 - 17:44:53 EST


On Wed, Jun 28, 2023 at 09:43:27PM +0200, Pawel Dembicki wrote:
> Switch in MAXLEN register stores the maximum size of a data frame.
> The MTU size is 18 bytes smaller than the frame size.
>
> The current settings are causing problems with packet forwarding.
> This patch fixes the MTU settings to proper values.
>
> Fixes: fb77ffc6ec86 ("net: dsa: vsc73xx: make the MTU configurable")
> Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> Signed-off-by: Pawel Dembicki <paweldembicki@xxxxxxxxx>
>
> ---
> In my first attempt, I sent it to net-next [0], but I was asked to resend
> it to net.
>
> [0]https://lore.kernel.org/netdev/20230625115343.1603330-7-paweldembicki@xxxxxxxxx/

Just like you had marked those earlier patches as "[PATCH net-next]",
this should have also been marked as "[PATCH v2 net]". Patchwork does
complain about that and deduced the wrong tree, but accepted it anyway
and otherwise ran its tests.
https://patchwork.kernel.org/project/netdevbpf/patch/20230628194327.1765644-1-paweldembicki@xxxxxxxxx/

I don't believe there is any reason to resend this patch.

Reviewed-by: Vladimir Oltean <olteanv@xxxxxxxxx>