Re: [16/26] r8169: fix 8168evl frame padding.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 26.06.2013 04:35, Ben Hutchings wrote:
> 3.2.48-rc1 review patch.  If anyone has any objections, please let me know.
> 
> ------------------
> 
> From: Stefan Bader <stefan.bader@xxxxxxxxxxxxx>
> 
> [ Upstream commits e5195c1f31f399289347e043d6abf3ffa80f0005 and
>   b423e9ae49d78ea3f53b131c8d5a6087aed16fd6 ]
> 
> Signed-off-by: Stefan Bader <stefan.bader@xxxxxxxxxxxxx>
> Acked-by: Francois Romieu <romieu@xxxxxxxxxxxxx>
> Cc: hayeswang <hayeswang@xxxxxxxxxxx>
> Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>
> Signed-off-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx>
> ---
>  drivers/net/ethernet/realtek/r8169.c | 33 +++++++++++++++++++++++++++------

No abjection, just make sure that

commit b423e9ae49d78ea3f53b131c8d5a6087aed16fd6
Author: françois romieu <romieu@xxxxxxxxxxxxx>
Date:   Sat May 18 01:24:46 2013 +0000

    r8169: fix offloaded tx checksum for small packets.

    8168evl offloaded checksums are wrong since commit
    e5195c1f31f399289347e043d6abf3ffa80f0005 ("r8169: fix 8168evl frame padding.
    pads small packets to 60 bytes (without ethernet checksum). Typical symptoms
    appear as UDP checksums which are wrong by the count of added bytes.

    It isn't worth compensating. Let the driver checksum.

    Due to the skb length changes, TSO code is moved before the Tx descriptor ge
    written.

    Signed-off-by: Francois Romieu <romieu@xxxxxxxxxxxxx>
    Tested-by: Holger Hoffstätte <holger.hoffstaette@xxxxxxxxxxxxxx>
    Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>

is included as well.

-Stefan

Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]