Re: [PATCH] usbnet: do not pretend to support SG/TSO

From: David Miller
Date: Fri Jul 26 2013 - 16:49:06 EST


From: Eric Dumazet <eric.dumazet@xxxxxxxxx>
Date: Tue, 23 Jul 2013 17:15:54 -0700

> From: Eric Dumazet <edumazet@xxxxxxxxxx>
>
> usbnet doesn't support yet SG, so drivers should not advertise SG or TSO
> capabilities, as they allow TCP stack to build large TSO packets that
> need to be linearized and might use order-5 pages.
>
> This adds an extra copy overhead and possible allocation failures.
>
> Current code ignore skb_linearize() return code so crashes are even
> possible.
>
> Best is to not pretend SG/TSO is supported, and add this again when/if
> usbnet really supports SG for devices who could get a performance gain.
>
> Based on a prior patch from Freddy Xin <freddy@xxxxxxxxxxx>
>
> Signed-off-by: Eric Dumazet <edumazet@xxxxxxxxxx>

Applied, thanks Eric.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/