Commit b0f9ca53 authored by Fan Du's avatar Fan Du Committed by David S. Miller
Browse files

ipv4: Namespecify TCP PMTU mechanism 



Packetization Layer Path MTU Discovery works separately beside
Path MTU Discovery at IP level, different net namespace has
various requirements on which one to chose, e.g., a virutalized
container instance would require TCP PMTU to probe an usable
effective mtu for underlying tunnel, while the host would
employ classical ICMP based PMTU to function.

Hence making TCP PMTU mechanism per net namespace to decouple
two functionality. Furthermore the probe base MSS should also
be configured separately for each namespace.

Signed-off-by: default avatarFan Du <fan.du@intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent f217d6ca

include/net/netns/ipv4.h

+2 −0
Original line number Diff line number Diff line
@@ -82,6 +82,8 @@ struct netns_ipv4 { 


	int sysctl_fwmark_reflect;

	int sysctl_tcp_fwmark_accept;

	int sysctl_tcp_mtu_probing;

	int sysctl_tcp_base_mss;



	struct ping_group_range ping_group_range;

include/net/tcp.h

+0 −2
Original line number Diff line number Diff line
@@ -262,8 +262,6 @@ extern int sysctl_tcp_low_latency; 
extern int sysctl_tcp_nometrics_save;

extern int sysctl_tcp_moderate_rcvbuf;

extern int sysctl_tcp_tso_win_divisor;

extern int sysctl_tcp_mtu_probing;

extern int sysctl_tcp_base_mss;

extern int sysctl_tcp_workaround_signed_windows;

extern int sysctl_tcp_slow_start_after_idle;

extern int sysctl_tcp_thin_linear_timeouts;

net/ipv4/sysctl_net_ipv4.c

+14 −14
Original line number Diff line number Diff line
@@ -603,20 +603,6 @@ static struct ctl_table ipv4_table[] = { 
		.maxlen		= TCP_CA_NAME_MAX,

		.proc_handler	= proc_tcp_congestion_control,

	},

	{

		.procname	= "tcp_mtu_probing",

		.data		= &sysctl_tcp_mtu_probing,

		.maxlen		= sizeof(int),

		.mode		= 0644,

		.proc_handler	= proc_dointvec,

	},

	{

		.procname	= "tcp_base_mss",

		.data		= &sysctl_tcp_base_mss,

		.maxlen		= sizeof(int),

		.mode		= 0644,

		.proc_handler	= proc_dointvec,

	},

	{

		.procname	= "tcp_workaround_signed_windows",

		.data		= &sysctl_tcp_workaround_signed_windows,
@@ -883,6 +869,20 @@ static struct ctl_table ipv4_net_table[] = { 
		.mode		= 0644,

		.proc_handler	= proc_dointvec,

	},

	{

		.procname	= "tcp_mtu_probing",

		.data		= &init_net.ipv4.sysctl_tcp_mtu_probing,

		.maxlen		= sizeof(int),

		.mode		= 0644,

		.proc_handler	= proc_dointvec,

	},

	{

		.procname	= "tcp_base_mss",

		.data		= &init_net.ipv4.sysctl_tcp_base_mss,

		.maxlen		= sizeof(int),

		.mode		= 0644,

		.proc_handler	= proc_dointvec,

	},

	{ }

};

net/ipv4/tcp_ipv4.c

+1 −0
Original line number Diff line number Diff line
@@ -2459,6 +2459,7 @@ static int __net_init tcp_sk_init(struct net *net) 
		*per_cpu_ptr(net->ipv4.tcp_sk, cpu) = sk;

	}

	net->ipv4.sysctl_tcp_ecn = 2;

	net->ipv4.sysctl_tcp_base_mss = TCP_BASE_MSS;

	return 0;



fail:

net/ipv4/tcp_output.c

+3 −5
Original line number Diff line number Diff line
@@ -59,9 +59,6 @@ int sysctl_tcp_limit_output_bytes __read_mostly = 131072; 
 */

int sysctl_tcp_tso_win_divisor __read_mostly = 3;



int sysctl_tcp_mtu_probing __read_mostly = 0;

int sysctl_tcp_base_mss __read_mostly = TCP_BASE_MSS;



/* By default, RFC2861 behavior.  */

int sysctl_tcp_slow_start_after_idle __read_mostly = 1;
@@ -1350,11 +1347,12 @@ void tcp_mtup_init(struct sock *sk) 
{

	struct tcp_sock *tp = tcp_sk(sk);

	struct inet_connection_sock *icsk = inet_csk(sk);

	struct net *net = sock_net(sk);



	icsk->icsk_mtup.enabled = sysctl_tcp_mtu_probing > 1;

	icsk->icsk_mtup.enabled = net->ipv4.sysctl_tcp_mtu_probing > 1;

	icsk->icsk_mtup.search_high = tp->rx_opt.mss_clamp + sizeof(struct tcphdr) +

			       icsk->icsk_af_ops->net_header_len;

	icsk->icsk_mtup.search_low = tcp_mss_to_mtu(sk, sysctl_tcp_base_mss);

	icsk->icsk_mtup.search_low = tcp_mss_to_mtu(sk, net->ipv4.sysctl_tcp_base_mss);

	icsk->icsk_mtup.probe_size = 0;

}

EXPORT_SYMBOL(tcp_mtup_init);

CRA Git | Maintained and supported by SUSTech CRA and CCSE