netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Shall we add some note info for tcp_min_snd_mss?
@ 2019-07-03  8:35 ZhangXiao
  2019-07-03  9:34 ` Eric Dumazet
  0 siblings, 1 reply; 2+ messages in thread
From: ZhangXiao @ 2019-07-03  8:35 UTC (permalink / raw)
  To: edumazet, davem; +Cc: netdev

Hi David & Eric,

Commit 5f3e2bf0 (tcp: add tcp_min_snd_mss sysctl) add a new interface to
adjust network. While if this variable been set too large, for example
larger then (MTU - 40), the net link maybe damaged. So, how about adding
some warning messages for the operator/administrator? In document, or in
source code.

Thanks
Xiao

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Shall we add some note info for tcp_min_snd_mss?
  2019-07-03  8:35 Shall we add some note info for tcp_min_snd_mss? ZhangXiao
@ 2019-07-03  9:34 ` Eric Dumazet
  0 siblings, 0 replies; 2+ messages in thread
From: Eric Dumazet @ 2019-07-03  9:34 UTC (permalink / raw)
  To: ZhangXiao, edumazet, davem; +Cc: netdev



On 7/3/19 1:35 AM, ZhangXiao wrote:
> Hi David & Eric,
> 
> Commit 5f3e2bf0 (tcp: add tcp_min_snd_mss sysctl) add a new interface to
> adjust network. While if this variable been set too large, for example
> larger then (MTU - 40), the net link maybe damaged. So, how about adding
> some warning messages for the operator/administrator? In document, or in
> source code.

What kind of warning message do you 
envision exactly ?

The sysctl is global, devices MTU can be quite arbitrary.

Like almost all sysctls, sysadmins are supposed to know what they are doing.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-07-03  9:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-07-03  8:35 Shall we add some note info for tcp_min_snd_mss? ZhangXiao
2019-07-03  9:34 ` Eric Dumazet

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).