From: Igor Russkikh <[email protected]>
Date: Tue, 27 Nov 2018 14:51:17 +0000

> From: Dmitry Bogdanov <[email protected]>
> 
> The last set of csum offload fixes had a leak:
> 
> Checksum enabled status bits from rx descriptor were incorrectly
> interpreted. Consequently all the other valid logic worked on zero bits.
> That caused rx checksum offloads never to trigger.
> 
> Tested by dumping rx descriptors and validating resulting csum_level.
> 
> Reported-by: Igor Russkikh <[email protected]>
> Signed-off-by: Dmitry Bogdanov <[email protected]>
> Signed-off-by: Igor Russkikh <[email protected]>
> Fixes: ad703c2b9127f ("net: aquantia: invalid checksumm offload 
> implementation")

Applied.

This shows that the feature added was not actually tested.

Thanks.

Reply via email to