[go] doc/go1.21: mention multipath TCP support

17 views
Skip to first unread message

Ian Lance Taylor (Gerrit)

unread,
May 26, 2023, 7:53:06 PM5/26/23
to Eli Bendersky, goph...@pubsubhelper.golang.org, Ian Lance Taylor, golang-co...@googlegroups.com

Attention is currently required from: Eli Bendersky.

Ian Lance Taylor would like Eli Bendersky to review this change.

View Change

doc/go1.21: mention multipath TCP support

For #56539
For #59166

Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
---
M doc/go1.21.html
1 file changed, 20 insertions(+), 17 deletions(-)

diff --git a/doc/go1.21.html b/doc/go1.21.html
index 3758d28..8c0d927 100644
--- a/doc/go1.21.html
+++ b/doc/go1.21.html
@@ -568,24 +568,27 @@

<dl id="net"><dt><a href="/pkg/net/">net</a></dt>
<dd>
- <p><!-- https://go.dev/issue/56539 -->
- TODO: <a href="https://go.dev/issue/56539">https://go.dev/issue/56539</a>: add support for MPTCP
+ <p>
+ <!-- https://go.dev/issue/59166, https://go.dev/issue/56539 -->
+ <!-- CL 471136, CL 471137, CL 471140 -->
+ On Linux, the <a href="/pkg/net/">net</a> package can now use
+ Multipath TCP when the kernel supports it. It is not used by
+ default. To use Multipath TCP when possible, call the
+ <a href="/pkg/net/#Dialer.SetMultipathTCP"><code>Dialer.SetMultipathTCP</code></a>
+ method before calling the
+ <a href="/pkg/net/#Dialer.Dial"><code>Dialer.Dial</code></a> or
+ <a href="/pkg/net/#Dialer.DialContext"><code>Dialer.DialContext</code></a>
+ methods. Specify the network as <code>"tcp"</code>
+ or <code>"tcp4"</code> or <code>"tcp6"</code> as usual. If
+ Multipath TCP is not supported by the kernel or the remote host,
+ the connection will silently fall back to TCP. To test whether a
+ particular connection is using Multipath TCP, use the
+ <a href="/pkg/net/#TCPConn.MultipathTCP"><code>TCPConn.MultipathTCP</code></a>
+ method.
</p>
-
- <p><!-- https://go.dev/issue/59166 -->
- TODO: <a href="https://go.dev/issue/59166">https://go.dev/issue/59166</a>: add func (*TCPConn) MultipathTCP() (bool, error)
- </p>
-
- <p><!-- CL 471136 -->
- TODO: <a href="https://go.dev/cl/471136">https://go.dev/cl/471136</a>: net: mptcp: implement dialMPTCP; modified api/next/56539.txt
- </p>
-
- <p><!-- CL 471137 -->
- TODO: <a href="https://go.dev/cl/471137">https://go.dev/cl/471137</a>: net: mptcp: implement listenMPTCP; modified api/next/56539.txt
- </p>
-
- <p><!-- CL 471140 -->
- TODO: <a href="https://go.dev/cl/471140">https://go.dev/cl/471140</a>: net: mptcp: add TCPConn&#39;s MultipathTCP checker; modified api/next/59166.txt
+ <p>
+ In the future Go 1.22 release, we may enable Multipath TCP by
+ default on systems that support it.
</p>
</dd>
</dl><!-- net -->

To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newchange
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
Gerrit-Change-Number: 498601
Gerrit-PatchSet: 1
Gerrit-Owner: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Eli Bendersky <eli...@google.com>
Gerrit-Reviewer: Ian Lance Taylor <ia...@golang.org>
Gerrit-Attention: Eli Bendersky <eli...@google.com>

Matthieu Baerts (Gerrit)

unread,
May 28, 2023, 11:54:57 AM5/28/23
to Ian Lance Taylor, goph...@pubsubhelper.golang.org, Gopher Robot, Eli Bendersky, golang-co...@googlegroups.com

Attention is currently required from: Eli Bendersky, Ian Lance Taylor.

View Change

2 comments:

    • To use Multipath TCP when possible, call the

    •       <a href="/pkg/net/#Dialer.SetMultipathTCP"><code>Dialer.SetMultipathTCP</code></a>

    •       method before calling the


    • <a href="/pkg/net/#Dialer.Dial"><code>Dialer.Dial</code></a> or

    •       <a href="/pkg/net/#Dialer.DialContext"><code>Dialer.DialContext</code></a>

    •       methods.

      Should there be a note about the listener part? (similarly by using `ListenConfig.SetMultipathTCP` method before calling `Listen`?)

To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: comment
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
Gerrit-Change-Number: 498601
Gerrit-PatchSet: 1
Gerrit-Owner: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Eli Bendersky <eli...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Ian Lance Taylor <ia...@golang.org>
Gerrit-CC: Matthieu Baerts <matthie...@tessares.net>
Gerrit-Attention: Eli Bendersky <eli...@google.com>
Gerrit-Attention: Ian Lance Taylor <ia...@golang.org>
Gerrit-Comment-Date: Sun, 28 May 2023 15:54:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Ian Lance Taylor (Gerrit)

unread,
May 29, 2023, 7:24:33 PM5/29/23
to Ian Lance Taylor, goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com

Attention is currently required from: Eli Bendersky, Ian Lance Taylor.

Ian Lance Taylor uploaded patch set #2 to this change.

View Change

The following approvals got outdated and were removed: Run-TryBot+1 by Ian Lance Taylor, TryBot-Result+1 by Gopher Robot

doc/go1.21: mention multipath TCP support

For #56539
For #59166

Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
---
M doc/go1.21.html
1 file changed, 26 insertions(+), 17 deletions(-)

To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newpatchset
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
Gerrit-Change-Number: 498601
Gerrit-PatchSet: 2

Ian Lance Taylor (Gerrit)

unread,
May 29, 2023, 7:24:45 PM5/29/23
to Ian Lance Taylor, goph...@pubsubhelper.golang.org, Matthieu Baerts, Gopher Robot, Eli Bendersky, golang-co...@googlegroups.com

Attention is currently required from: Eli Bendersky, Matthieu Baerts.

Patch set 1:Run-TryBot +1

View Change

1 comment:

  • File doc/go1.21.html:

    • Patch Set #1, Line 576:

      To use Multipath TCP when possible, call the
      <a href="/pkg/net/#Dialer.SetMultipathTCP"><code>Dialer.SetMultipathTCP</code></a>
      method before calling the
      <a href="/pkg/net/#Dialer.Dial"><code>Dialer.Dial</code></a> or
      <a href="/pkg/net/#Dialer.DialContext"><code>Dialer.DialContext</code></a>
      methods.

    • Should there be a note about the listener part? (similarly by using `ListenConfig. […]

      Done, thanks.

To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: comment
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
Gerrit-Change-Number: 498601
Gerrit-PatchSet: 1
Gerrit-Owner: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Eli Bendersky <eli...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Ian Lance Taylor <ia...@golang.org>
Gerrit-CC: Matthieu Baerts <matthie...@tessares.net>
Gerrit-Attention: Eli Bendersky <eli...@google.com>
Gerrit-Attention: Matthieu Baerts <matthie...@tessares.net>
Gerrit-Comment-Date: Mon, 29 May 2023 23:24:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Matthieu Baerts <matthie...@tessares.net>

Matthieu Baerts (Gerrit)

unread,
May 30, 2023, 5:12:35 AM5/30/23
to Ian Lance Taylor, goph...@pubsubhelper.golang.org, Gopher Robot, Eli Bendersky, golang-co...@googlegroups.com

Attention is currently required from: Eli Bendersky, Ian Lance Taylor.

Patch set 2:Code-Review +1

View Change

1 comment:

To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: comment
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
Gerrit-Change-Number: 498601
Gerrit-PatchSet: 2
Gerrit-Owner: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Eli Bendersky <eli...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Matthieu Baerts <matthie...@tessares.net>
Gerrit-Attention: Eli Bendersky <eli...@google.com>
Gerrit-Attention: Ian Lance Taylor <ia...@golang.org>
Gerrit-Comment-Date: Tue, 30 May 2023 09:12:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes

Eli Bendersky (Gerrit)

unread,
May 30, 2023, 9:05:43 AM5/30/23
to Ian Lance Taylor, goph...@pubsubhelper.golang.org, Matthieu Baerts, Gopher Robot, golang-co...@googlegroups.com

Attention is currently required from: Ian Lance Taylor.

Patch set 2:Code-Review +2

View Change

1 comment:

  • File doc/go1.21.html:

    • Patch Set #2, Line 636: In the future Go 1.22 release, we may enable Multipath TCP by

      Nit: do we typically make specific version promises like this one in release notes? Should it just say "In a future Go release, ..."?

To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: comment
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
Gerrit-Change-Number: 498601
Gerrit-PatchSet: 2
Gerrit-Owner: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Eli Bendersky <eli...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Matthieu Baerts <matthie...@tessares.net>
Gerrit-Attention: Ian Lance Taylor <ia...@golang.org>
Gerrit-Comment-Date: Tue, 30 May 2023 13:05:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes

Ian Lance Taylor (Gerrit)

unread,
May 30, 2023, 12:07:30 PM5/30/23
to Ian Lance Taylor, goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com

Attention is currently required from: Ian Lance Taylor.

Ian Lance Taylor uploaded patch set #3 to this change.

View Change

The following approvals got outdated and were removed: Run-TryBot+1 by Ian Lance Taylor, TryBot-Result+1 by Gopher Robot

doc/go1.21: mention multipath TCP support

For #56539
For #59166

Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
---
M doc/go1.21.html
1 file changed, 26 insertions(+), 17 deletions(-)

To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newpatchset
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
Gerrit-Change-Number: 498601
Gerrit-PatchSet: 3

Ian Lance Taylor (Gerrit)

unread,
May 30, 2023, 12:07:31 PM5/30/23
to Ian Lance Taylor, goph...@pubsubhelper.golang.org, Eli Bendersky, Matthieu Baerts, Gopher Robot, golang-co...@googlegroups.com

Patch set 2:Run-TryBot +1

View Change

1 comment:

  • File doc/go1.21.html:

    • Nit: do we typically make specific version promises like this one in release notes? Should it just s […]

      Done

To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: comment
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
Gerrit-Change-Number: 498601
Gerrit-PatchSet: 2
Gerrit-Owner: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Eli Bendersky <eli...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Matthieu Baerts <matthie...@tessares.net>
Gerrit-Comment-Date: Tue, 30 May 2023 16:07:28 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Eli Bendersky <eli...@google.com>

Ian Lance Taylor (Gerrit)

unread,
May 30, 2023, 12:07:43 PM5/30/23
to Ian Lance Taylor, goph...@pubsubhelper.golang.org, Eli Bendersky, Matthieu Baerts, Gopher Robot, golang-co...@googlegroups.com

Attention is currently required from: Ian Lance Taylor.

Patch set 3:Run-TryBot +1Auto-Submit +1Code-Review +1

View Change

    To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

    Gerrit-MessageType: comment
    Gerrit-Project: go
    Gerrit-Branch: master
    Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
    Gerrit-Change-Number: 498601
    Gerrit-PatchSet: 3
    Gerrit-Owner: Ian Lance Taylor <ia...@golang.org>
    Gerrit-Reviewer: Eli Bendersky <eli...@google.com>
    Gerrit-Reviewer: Gopher Robot <go...@golang.org>
    Gerrit-Reviewer: Ian Lance Taylor <ia...@golang.org>
    Gerrit-Reviewer: Ian Lance Taylor <ia...@google.com>
    Gerrit-Reviewer: Matthieu Baerts <matthie...@tessares.net>
    Gerrit-Attention: Ian Lance Taylor <ia...@golang.org>
    Gerrit-Comment-Date: Tue, 30 May 2023 16:07:39 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes

    Gopher Robot (Gerrit)

    unread,
    May 30, 2023, 12:18:44 PM5/30/23
    to Ian Lance Taylor, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Eli Bendersky, Matthieu Baerts, golang-co...@googlegroups.com

    Gopher Robot submitted this change.

    View Change



    2 is the latest approved patch-set.
    The change was submitted with unreviewed changes in the following files:

    ```
    The name of the file: doc/go1.21.html
    Insertions: 2, Deletions: 2.

    @@ -633,8 +633,8 @@
    method.
    </p>
    <p>
    - In the future Go 1.22 release, we may enable Multipath TCP by
    - default on systems that support it.
    + In a future Go release we may enable Multipath TCP by default on
    + systems that support it.

    </p>
    </dd>
    </dl><!-- net -->
    ```

    Approvals: Gopher Robot: TryBots succeeded Ian Lance Taylor: Run TryBots Eli Bendersky: Looks good to me, approved Matthieu Baerts: Looks good to me, but someone else must approve Ian Lance Taylor: Looks good to me, but someone else must approve; Run TryBots; Automatically submit change
    doc/go1.21: mention multipath TCP support

    For #56539
    For #59166

    Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
    Reviewed-on: https://go-review.googlesource.com/c/go/+/498601
    Run-TryBot: Ian Lance Taylor <ia...@google.com>
    Run-TryBot: Ian Lance Taylor <ia...@golang.org>
    Reviewed-by: Ian Lance Taylor <ia...@google.com>
    Auto-Submit: Ian Lance Taylor <ia...@google.com>
    Reviewed-by: Eli Bendersky <eli...@google.com>
    TryBot-Result: Gopher Robot <go...@golang.org>
    Reviewed-by: Matthieu Baerts <matthie...@tessares.net>

    ---
    M doc/go1.21.html
    1 file changed, 26 insertions(+), 17 deletions(-)

    
    
    diff --git a/doc/go1.21.html b/doc/go1.21.html
    index 075cee5..d3d79a2 100644
    --- a/doc/go1.21.html
    +++ b/doc/go1.21.html
    @@ -608,24 +608,33 @@


    <dl id="net"><dt><a href="/pkg/net/">net</a></dt>
    <dd>
    - <p><!-- https://go.dev/issue/56539 -->
    - TODO: <a href="https://go.dev/issue/56539">https://go.dev/issue/56539</a>: add support for MPTCP
    + <p>
    + <!-- https://go.dev/issue/59166, https://go.dev/issue/56539 -->
    + <!-- CL 471136, CL 471137, CL 471140 -->
    + On Linux, the <a href="/pkg/net/">net</a> package can now use
    + Multipath TCP when the kernel supports it. It is not used by
    +      default. To use Multipath TCP when available on a client, call
    + the

    + <a href="/pkg/net/#Dialer.SetMultipathTCP"><code>Dialer.SetMultipathTCP</code></a>
    + method before calling the
    +      <a href="/pkg/net/#Dialer.Dial"><code>Dialer.Dial</code></a> or
    + <a href="/pkg/net/#Dialer.DialContext"><code>Dialer.DialContext</code></a>
    + methods. To use Multipath TCP when available on a server, call
    + the
    + <a href="/pkg/net/#ListenConfig.SetMultipathTCP"><code>ListenConfig.SetMultipathTCP</code></a>

    + method before calling the
    +      <a href="/pkg/net/#ListenConfig.Listen"><code>ListenConfig.Listen</code></a>
    + method. Specify the network as <code>"tcp"</code> or
    + <code>"tcp4"</code> or <code>"tcp6"</code> as usual. If

    + Multipath TCP is not supported by the kernel or the remote host,
    + the connection will silently fall back to TCP. To test whether a
    + particular connection is using Multipath TCP, use the
    + <a href="/pkg/net/#TCPConn.MultipathTCP"><code>TCPConn.MultipathTCP</code></a>
    + method.
    </p>
    -
    - <p><!-- https://go.dev/issue/59166 -->
    - TODO: <a href="https://go.dev/issue/59166">https://go.dev/issue/59166</a>: add func (*TCPConn) MultipathTCP() (bool, error)
    - </p>
    -
    - <p><!-- CL 471136 -->
    - TODO: <a href="https://go.dev/cl/471136">https://go.dev/cl/471136</a>: net: mptcp: implement dialMPTCP; modified api/next/56539.txt
    - </p>
    -
    - <p><!-- CL 471137 -->
    - TODO: <a href="https://go.dev/cl/471137">https://go.dev/cl/471137</a>: net: mptcp: implement listenMPTCP; modified api/next/56539.txt
    - </p>
    -
    - <p><!-- CL 471140 -->
    - TODO: <a href="https://go.dev/cl/471140">https://go.dev/cl/471140</a>: net: mptcp: add TCPConn&#39;s MultipathTCP checker; modified api/next/59166.txt
    + <p>
    +      In a future Go release we may enable Multipath TCP by default on
    + systems that support it.

    </p>
    </dd>
    </dl><!-- net -->

    To view, visit change 498601. To unsubscribe, or for help writing mail filters, visit settings.

    Gerrit-MessageType: merged
    Gerrit-Project: go
    Gerrit-Branch: master
    Gerrit-Change-Id: Ief392464916a1a74a8fcc6c3c7bdb213e8c6ef98
    Gerrit-Change-Number: 498601
    Gerrit-PatchSet: 4
    Reply all
    Reply to author
    Forward
    0 new messages