Re: [PATCH net-next v2 2/7] net: bcmgenet: allow a continuation descriptor without the alignment pad

[Florian Fainelli]

On 10/5/26 15:24, Nicolai Buchwitz wrote:
> A frame longer than the packet ready threshold arrives in several
> descriptors, each with its own status block. Only the first one also
> carries the two alignment bytes. The length check assumes the pad is always
> there, so a continuation holding a single byte looks a byte too short and
> the whole frame is dropped.
>
> Account for the pad on the first descriptor only.
>
> The MTU cannot produce a frame past the threshold yet, so nothing hits this
> today. It is preparation for the larger MTU.
>
> Signed-off-by: Nicolai Buchwitz <nb@xxxxxxxxxxx>
> Tested-by: Pierre-Marin Leclercq <pierremarinleclercq88@xxxxxxxxx>

Reviewed-by: Florian Fainelli <florian.fainelli@xxxxxxxxxxxx>
--
Florian