RE: [Intel-wired-lan] [PATCH] i40e: little endian only valid checksums

From: G, GurucharanX
Date: Tue Mar 08 2022 - 07:55:02 EST




> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@xxxxxxxxxx> On Behalf Of
> trix@xxxxxxxxxx
> Sent: Wednesday, March 2, 2022 6:27 PM
> To: Brandeburg, Jesse <jesse.brandeburg@xxxxxxxxx>; Nguyen, Anthony L
> <anthony.l.nguyen@xxxxxxxxx>; davem@xxxxxxxxxxxxx; kuba@xxxxxxxxxx
> Cc: netdev@xxxxxxxxxxxxxxx; intel-wired-lan@xxxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; Tom Rix <trix@xxxxxxxxxx>
> Subject: [Intel-wired-lan] [PATCH] i40e: little endian only valid checksums
>
> From: Tom Rix <trix@xxxxxxxxxx>
>
> The calculation of the checksum can fail.
> So move converting the checksum to little endian
> to inside the return status check.
>
> Signed-off-by: Tom Rix <trix@xxxxxxxxxx>
> ---
> drivers/net/ethernet/intel/i40e/i40e_nvm.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>

Tested-by: Gurucharan G <gurucharanx.g@xxxxxxxxx> (A Contingent worker at Intel)