{"id":830982,"url":"http://patchwork.ozlabs.org/api/1.2/patches/830982/?format=json","web_url":"http://patchwork.ozlabs.org/project/netdev/patch/20171026233526.6155-2-edumazet@google.com/","project":{"id":7,"url":"http://patchwork.ozlabs.org/api/1.2/projects/7/?format=json","name":"Linux network development","link_name":"netdev","list_id":"netdev.vger.kernel.org","list_email":"netdev@vger.kernel.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20171026233526.6155-2-edumazet@google.com>","list_archive_url":null,"date":"2017-10-26T23:35:12","name":"[net-next,01/15] tcp: Namespace-ify sysctl_tcp_early_retrans","commit_ref":null,"pull_url":null,"state":"changes-requested","archived":true,"hash":"c92ea0e4bf9a251fa2733b67a2e1abe778aa07d7","submitter":{"id":13357,"url":"http://patchwork.ozlabs.org/api/1.2/people/13357/?format=json","name":"Eric Dumazet","email":"edumazet@google.com"},"delegate":{"id":34,"url":"http://patchwork.ozlabs.org/api/1.2/users/34/?format=json","username":"davem","first_name":"David","last_name":"Miller","email":"davem@davemloft.net"},"mbox":"http://patchwork.ozlabs.org/project/netdev/patch/20171026233526.6155-2-edumazet@google.com/mbox/","series":[{"id":10461,"url":"http://patchwork.ozlabs.org/api/1.2/series/10461/?format=json","web_url":"http://patchwork.ozlabs.org/project/netdev/list/?series=10461","date":"2017-10-26T23:35:11","name":"tcp: move 14 sysctls to namespaces","version":1,"mbox":"http://patchwork.ozlabs.org/series/10461/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/830982/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/830982/checks/","tags":{},"related":[],"headers":{"Return-Path":"<netdev-owner@vger.kernel.org>","X-Original-To":"patchwork-incoming@ozlabs.org","Delivered-To":"patchwork-incoming@ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netdev-owner@vger.kernel.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"Tsxqd5dF\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3yNNfn3qcfz9t2Z\n\tfor <patchwork-incoming@ozlabs.org>;\n\tFri, 27 Oct 2017 10:35:45 +1100 (AEDT)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S932402AbdJZXfh (ORCPT <rfc822;patchwork-incoming@ozlabs.org>);\n\tThu, 26 Oct 2017 19:35:37 -0400","from mail-pg0-f65.google.com ([74.125.83.65]:56022 \"EHLO\n\tmail-pg0-f65.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S932381AbdJZXfd (ORCPT\n\t<rfc822;netdev@vger.kernel.org>); Thu, 26 Oct 2017 19:35:33 -0400","by mail-pg0-f65.google.com with SMTP id 15so3872363pgc.12\n\tfor <netdev@vger.kernel.org>; Thu, 26 Oct 2017 16:35:33 -0700 (PDT)","from localhost ([2620:15c:2c4:201:c5be:583d:61a8:d070])\n\tby smtp.gmail.com with ESMTPSA id\n\tk7sm11327904pgc.4.2017.10.26.16.35.32\n\t(version=TLS1_2 cipher=AES128-SHA bits=128/128);\n\tThu, 26 Oct 2017 16:35:32 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=google.com; s=20161025;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=gVLr2cVivBnVhJMozch2lYaS0Gc6FZaUS4LuBW91aZQ=;\n\tb=Tsxqd5dFkNdz15i0EcgCO56PILuijBcOH/lXvOvdIII2c3zoROOaWlXNKntlUYkdKB\n\tYClKdFvYgHnbwHqZG7zmDhrnA1wbK3fb6ZHsAbEuO9T/IUhEfWt2Moy6VGOIxM/UNelu\n\tSqYdJyRMH6iiNwj8OUGzvybT6xB/D7WHqwQoINnSNDZHeod5/fwZviYkm21ZEJSBy71f\n\t+cDSS1gp4FQu8rXWpqHHV+dZSZCfNym6RPUnliuakT6FQ8LvdcKZBa1VHX0mZXETsHS3\n\txdwGnhl+b4foqt2IkFQ2nA+ynEtFQfNC9cZO7Nvtukpq7h9gKW67gBxGXC2ZAreyqRc2\n\tWM+g==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references;\n\tbh=gVLr2cVivBnVhJMozch2lYaS0Gc6FZaUS4LuBW91aZQ=;\n\tb=W0gUmcF1jimz9QB/pvdrIeObL++Zfxrez3t5xIlK+Cq8uduw+ZBJsySWWKGkqeY5Pe\n\tHTNo5J/QRuWJbx/WP6TTJXza45Jo/yoO0cpBlJXJHCE+z+EfHtbv81mXADJ1GQwc1Qa6\n\t31ZkjBiN0BIb+EaCYMrvD0JMsoOfh0RF0SeRZ96BXwT3uz6h6lKHHBYlfA5fPGZYKShY\n\t+1TE1T3NdFVJrAGh4IaT5XoxiyJmBWHrYRNqsXPaBd4+rf2SNSNw4x+Ww97LEXnu7g+W\n\tfGjyRQuXMIvxP7j0yvFvhD12MwY88U9OpCR6OWqcz+y0SloqfLTP8MTt47XYLuGegp4R\n\tVxBA==","X-Gm-Message-State":"AMCzsaVFoiQsIY4HXadFYEPFTs+5JiydByQC7pjvOP+xBtM7AdUCQPxI\n\tWkhyXfBImYB62G/PKntchIqSvw==","X-Google-Smtp-Source":"ABhQp+TjKersxJUPSbdVxyyle4Ehhz9tbEUNRQA1P+z98HauS6wh7B7mg4TxIfEQDZuo7Tkn9acYKA==","X-Received":"by 10.98.71.132 with SMTP id p4mr7094039pfi.274.1509060932732;\n\tThu, 26 Oct 2017 16:35:32 -0700 (PDT)","From":"Eric Dumazet <edumazet@google.com>","To":"\"David S . Miller\" <davem@davemloft.net>","Cc":"netdev <netdev@vger.kernel.org>, Eric Dumazet <edumazet@google.com>,\n\tEric Dumazet <eric.dumazet@gmail.com>","Subject":"[PATCH net-next 01/15] tcp: Namespace-ify sysctl_tcp_early_retrans","Date":"Thu, 26 Oct 2017 16:35:12 -0700","Message-Id":"<20171026233526.6155-2-edumazet@google.com>","X-Mailer":"git-send-email 2.15.0.rc2.357.g7e34df9404-goog","In-Reply-To":"<20171026233526.6155-1-edumazet@google.com>","References":"<20171026233526.6155-1-edumazet@google.com>","Sender":"netdev-owner@vger.kernel.org","Precedence":"bulk","List-ID":"<netdev.vger.kernel.org>","X-Mailing-List":"netdev@vger.kernel.org"},"content":"Signed-off-by: Eric Dumazet <edumazet@google.com>\n---\n include/net/netns/ipv4.h   |  1 +\n include/net/tcp.h          |  1 -\n net/ipv4/sysctl_net_ipv4.c | 18 +++++++++---------\n net/ipv4/tcp_input.c       |  1 -\n net/ipv4/tcp_ipv4.c        |  1 +\n net/ipv4/tcp_output.c      |  4 +++-\n 6 files changed, 14 insertions(+), 12 deletions(-)","diff":"diff --git a/include/net/netns/ipv4.h b/include/net/netns/ipv4.h\nindex 2c4222a5d1025f5928665e10edb70fad65352dba..a7f39e3ea666a835b6042e4008c86ccaadd14b46 100644\n--- a/include/net/netns/ipv4.h\n+++ b/include/net/netns/ipv4.h\n@@ -128,6 +128,7 @@ struct netns_ipv4 {\n \tint sysctl_tcp_sack;\n \tint sysctl_tcp_window_scaling;\n \tint sysctl_tcp_timestamps;\n+\tint sysctl_tcp_early_retrans;\n \tstruct inet_timewait_death_row tcp_death_row;\n \tint sysctl_max_syn_backlog;\n \tint sysctl_tcp_fastopen;\ndiff --git a/include/net/tcp.h b/include/net/tcp.h\nindex 2392f74074e71af6a6f8113f1ba9c13c7ee5c1bc..862cecd9424b30e637cfb717db519e2fe95d2c8e 100644\n--- a/include/net/tcp.h\n+++ b/include/net/tcp.h\n@@ -262,7 +262,6 @@ extern int sysctl_tcp_workaround_signed_windows;\n extern int sysctl_tcp_slow_start_after_idle;\n extern int sysctl_tcp_thin_linear_timeouts;\n extern int sysctl_tcp_thin_dupack;\n-extern int sysctl_tcp_early_retrans;\n extern int sysctl_tcp_recovery;\n #define TCP_RACK_LOSS_DETECTION  0x1 /* Use RACK to detect losses */\n \ndiff --git a/net/ipv4/sysctl_net_ipv4.c b/net/ipv4/sysctl_net_ipv4.c\nindex 81d218346cf7a0f340f964c434a21cace5c41fa0..f0f650f020afd535f41943c6c9fb1483be7cfb8d 100644\n--- a/net/ipv4/sysctl_net_ipv4.c\n+++ b/net/ipv4/sysctl_net_ipv4.c\n@@ -634,15 +634,6 @@ static struct ctl_table ipv4_table[] = {\n \t\t.mode           = 0644,\n \t\t.proc_handler   = proc_dointvec\n \t},\n-\t{\n-\t\t.procname\t= \"tcp_early_retrans\",\n-\t\t.data\t\t= &sysctl_tcp_early_retrans,\n-\t\t.maxlen\t\t= sizeof(int),\n-\t\t.mode\t\t= 0644,\n-\t\t.proc_handler\t= proc_dointvec_minmax,\n-\t\t.extra1\t\t= &zero,\n-\t\t.extra2\t\t= &four,\n-\t},\n \t{\n \t\t.procname\t= \"tcp_min_tso_segs\",\n \t\t.data\t\t= &sysctl_tcp_min_tso_segs,\n@@ -1145,6 +1136,15 @@ static struct ctl_table ipv4_net_table[] = {\n \t\t.mode\t\t= 0644,\n \t\t.proc_handler\t= proc_dointvec\n \t},\n+\t{\n+\t\t.procname\t= \"tcp_early_retrans\",\n+\t\t.data\t\t= &init_net.ipv4.sysctl_tcp_early_retrans,\n+\t\t.maxlen\t\t= sizeof(int),\n+\t\t.mode\t\t= 0644,\n+\t\t.proc_handler\t= proc_dointvec_minmax,\n+\t\t.extra1\t\t= &zero,\n+\t\t.extra2\t\t= &four,\n+\t},\n \t{ }\n };\n \ndiff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c\nindex 893286db4623773afe75f4e0cfea7ad0d24ea2fe..91b4f14f71fb6c63c97dde8ba6a5af61c5a1fc82 100644\n--- a/net/ipv4/tcp_input.c\n+++ b/net/ipv4/tcp_input.c\n@@ -93,7 +93,6 @@ int sysctl_tcp_max_orphans __read_mostly = NR_FILE;\n int sysctl_tcp_frto __read_mostly = 2;\n int sysctl_tcp_min_rtt_wlen __read_mostly = 300;\n int sysctl_tcp_moderate_rcvbuf __read_mostly = 1;\n-int sysctl_tcp_early_retrans __read_mostly = 3;\n int sysctl_tcp_invalid_ratelimit __read_mostly = HZ/2;\n \n #define FLAG_DATA\t\t0x01 /* Incoming frame contained data.\t\t*/\ndiff --git a/net/ipv4/tcp_ipv4.c b/net/ipv4/tcp_ipv4.c\nindex 23a8100af5ad399d4fa2568f4cac19192a008055..7ab313f6768e234173d78f17cfb1f664b230e958 100644\n--- a/net/ipv4/tcp_ipv4.c\n+++ b/net/ipv4/tcp_ipv4.c\n@@ -2484,6 +2484,7 @@ static int __net_init tcp_sk_init(struct net *net)\n \tnet->ipv4.sysctl_tcp_sack = 1;\n \tnet->ipv4.sysctl_tcp_window_scaling = 1;\n \tnet->ipv4.sysctl_tcp_timestamps = 1;\n+\tnet->ipv4.sysctl_tcp_early_retrans = 3;\n \n \tnet->ipv4.sysctl_tcp_fastopen = TFO_CLIENT_ENABLE;\n \tspin_lock_init(&net->ipv4.tcp_fastopen_ctx_lock);\ndiff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c\nindex 1f01f4c9c738484822aee7a6810d90857e6a72dc..dd7c22b7d23b016b1da83138d6e8047abdef97e6 100644\n--- a/net/ipv4/tcp_output.c\n+++ b/net/ipv4/tcp_output.c\n@@ -2378,6 +2378,7 @@ bool tcp_schedule_loss_probe(struct sock *sk)\n \tstruct inet_connection_sock *icsk = inet_csk(sk);\n \tstruct tcp_sock *tp = tcp_sk(sk);\n \tu32 timeout, rto_delta_us;\n+\tint early_retrans;\n \n \t/* Don't do any loss probe on a Fast Open connection before 3WHS\n \t * finishes.\n@@ -2385,10 +2386,11 @@ bool tcp_schedule_loss_probe(struct sock *sk)\n \tif (tp->fastopen_rsk)\n \t\treturn false;\n \n+\tearly_retrans = sock_net(sk)->ipv4.sysctl_tcp_early_retrans;\n \t/* Schedule a loss probe in 2*RTT for SACK capable connections\n \t * in Open state, that are either limited by cwnd or application.\n \t */\n-\tif ((sysctl_tcp_early_retrans != 3 && sysctl_tcp_early_retrans != 4) ||\n+\tif ((early_retrans != 3 && early_retrans != 4) ||\n \t    !tp->packets_out || !tcp_is_sack(tp) ||\n \t    icsk->icsk_ca_state != TCP_CA_Open)\n \t\treturn false;\n","prefixes":["net-next","01/15"]}