netdev - Re: [PATCH] [NET]: Add per-connection option to set max TSO frame size

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID:  <20080212212218.5eb7d121@extreme>
Date:	Tue, 12 Feb 2008 21:22:18 -0800
From:	Stephen Hemminger <shemminger@...ux-foundation.org>
To:	netdev@...r.kernel.org
Subject:  Re: [PATCH] [NET]: Add per-connection option to set max TSO frame
 size

On Tue, 12 Feb 2008 14:01:32 -0800
PJ Waskiewicz <peter.p.waskiewicz.jr@...el.com> wrote:

> This patch adds the ability for device drivers to control the size of the
> TSO frames being sent to them, per TCP connection.  By setting the
> netdevice's max_gso_frame_size value, the socket layer will set the GSO
> frame size based on that value.  This will propogate into the TCP layer,
> and send TSO's of that size to the hardware.
> 
> This can be desirable to help tune the bursty nature of TSO on a
> per-adapter basis, where one may have 1 GbE and 10 GbE devices coexisting
> in a system, one running multiqueue and the other not, etc.
> 
> This can also be desirable for devices that cannot support full 64 KB
> TSO's, but still want to benefit from some level of segmentation
> offloading.
> 
> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@...el.com>
> ---
> 
>  include/linux/netdevice.h |    6 ++++++
>  include/net/sock.h        |    2 ++
>  net/core/dev.c            |    1 +
>  net/core/sock.c           |    6 ++++--
>  net/ipv4/tcp_output.c     |    4 ++--
>  5 files changed, 15 insertions(+), 4 deletions(-)
> 
> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
> index 047d432..ed1cc32 100644
> --- a/include/linux/netdevice.h
> +++ b/include/linux/netdevice.h
> @@ -616,6 +616,7 @@ struct net_device
>  
>  	/* Partially transmitted GSO packet. */
>  	struct sk_buff		*gso_skb;
> +	int			max_gso_frame_size;

should use unsigned rather than int (yes the older code is sloppy).

Also what about IPV6? 
-- 
Stephen Hemminger <stephen.hemminger@...tta.com>


--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html