lkml.org 
[lkml]   [2013]   [Jul]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Date
    Subject[01/85] r8169: fix offloaded tx checksum for small packets.
    3.2.49-rc1 review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Ben Hutchings <ben@decadent.org.uk>

    The workaround introduced by commit e5195c1f31f3 'r8169: fix 8168evl
    frame padding.' upstream was incorrect and was entirely replaced in
    commit b423e9ae49d7 'r8169: fix offloaded tx checksum for small
    packets.'

    On the 3.2.y branch, the first commit has effectively been applied
    twice: the first time by itself, and the second time in commit
    3cf40360f431 which squashed the two upstream commits together. That
    left us with both the incorrect and the correct workaround in place.
    Remove the incorrect one.

    Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
    Cc: Francois Romieu <romieu@fr.zoreil.com>
    --- a/drivers/net/ethernet/realtek/r8169.c
    +++ b/drivers/net/ethernet/realtek/r8169.c
    @@ -5584,14 +5584,6 @@ static netdev_tx_t rtl8169_start_xmit(st
    goto err_stop_0;
    }

    - /* 8168evl does not automatically pad to minimum length. */
    - if (unlikely(tp->mac_version == RTL_GIGA_MAC_VER_34 &&
    - skb->len < ETH_ZLEN)) {
    - if (skb_padto(skb, ETH_ZLEN))
    - goto err_update_stats;
    - skb_put(skb, ETH_ZLEN - skb->len);
    - }
    -
    if (unlikely(le32_to_cpu(txd->opts1) & DescOwn))
    goto err_stop_0;



    \
     
     \ /
      Last update: 2013-07-24 16:21    [W:4.534 / U:0.064 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site