[lkml]   [2012]   [Apr]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[PATCH v3] sky2: copy received packets on inefficient unaligned architecture
    Modified from original patch from Chris.

    The sky2 driver has to have 8 byte alignment of receive buffer
    on some chip versions. On architectures which don't support efficient
    unaligned access this doesn't work very well. The solution is to
    just copy all received packets which is what the driver already
    does for small packets.

    This allows the driver to be used on the Tilera TILEmpower-Gx, since
    the tile architecture doesn't currently handle kernel unaligned accesses,
    just userspace.

    Signed-off-by: Stephen Hemminger <>
    Acked-by: Chris Metcalf <>

    --- a/drivers/net/ethernet/marvell/sky2.c 2012-04-04 08:49:05.954853108 -0700
    +++ b/drivers/net/ethernet/marvell/sky2.c 2012-04-04 15:03:41.725705876 -0700
    @@ -2468,6 +2468,17 @@ static int sky2_change_mtu(struct net_de
    return err;

    +static inline bool needs_copy(const struct rx_ring_info *re,
    + unsigned length)
    + /* Some architectures need the IP header to be aligned */
    + if (!IS_ALIGNED(re->data_addr + ETH_HLEN, sizeof(u32)))
    + return true;
    + return length < copybreak;
    /* For small just reuse existing skb for next receive */
    static struct sk_buff *receive_copy(struct sky2_port *sky2,
    const struct rx_ring_info *re,
    @@ -2605,7 +2616,7 @@ static struct sk_buff *sky2_receive(stru
    goto error;

    - if (length < copybreak)
    + if (needs_copy(re, length))
    skb = receive_copy(sky2, re, length);
    skb = receive_new(sky2, re, length);

     \ /
      Last update: 2012-04-05 00:13    [W:0.021 / U:8.316 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site