lkml.org 
[lkml]   [2020]   [May]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.4 23/83] rxrpc: Fix DATA Tx to disable nofrag for UDP on AF_INET6 socket
    Date
    From: David Howells <dhowells@redhat.com>

    commit 0e631eee17dcea576ab922fa70e4fdbd596ee452 upstream.

    Fix the DATA packet transmission to disable nofrag for UDPv4 on an AF_INET6
    socket as well as UDPv6 when trying to transmit fragmentably.

    Without this, packets filled to the normal size used by the kernel AFS
    client of 1412 bytes be rejected by udp_sendmsg() with EMSGSIZE
    immediately. The ->sk_error_report() notification hook is called, but
    rxrpc doesn't generate a trace for it.

    This is a temporary fix; a more permanent solution needs to involve
    changing the size of the packets being filled in accordance with the MTU,
    which isn't currently done in AF_RXRPC. The reason for not doing so was
    that, barring the last packet in an rx jumbo packet, jumbos can only be
    assembled out of 1412-byte packets - and the plan was to construct jumbos
    on the fly at transmission time.

    Also, there's no point turning on IPV6_MTU_DISCOVER, since IPv6 has to
    engage in this anyway since fragmentation is only done by the sender. We
    can then condense the switch-statement in rxrpc_send_data_packet().

    Fixes: 75b54cb57ca3 ("rxrpc: Add IPv6 support")
    Signed-off-by: David Howells <dhowells@redhat.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    net/rxrpc/local_object.c | 9 ---------
    net/rxrpc/output.c | 42 +++++++++++-------------------------------
    2 files changed, 11 insertions(+), 40 deletions(-)

    --- a/net/rxrpc/local_object.c
    +++ b/net/rxrpc/local_object.c
    @@ -165,15 +165,6 @@ static int rxrpc_open_socket(struct rxrp
    goto error;
    }

    - /* we want to set the don't fragment bit */
    - opt = IPV6_PMTUDISC_DO;
    - ret = kernel_setsockopt(local->socket, SOL_IPV6, IPV6_MTU_DISCOVER,
    - (char *) &opt, sizeof(opt));
    - if (ret < 0) {
    - _debug("setsockopt failed");
    - goto error;
    - }
    -
    /* Fall through and set IPv4 options too otherwise we don't get
    * errors from IPv4 packets sent through the IPv6 socket.
    */
    --- a/net/rxrpc/output.c
    +++ b/net/rxrpc/output.c
    @@ -474,41 +474,21 @@ send_fragmentable:
    skb->tstamp = ktime_get_real();

    switch (conn->params.local->srx.transport.family) {
    + case AF_INET6:
    case AF_INET:
    opt = IP_PMTUDISC_DONT;
    - ret = kernel_setsockopt(conn->params.local->socket,
    - SOL_IP, IP_MTU_DISCOVER,
    - (char *)&opt, sizeof(opt));
    - if (ret == 0) {
    - ret = kernel_sendmsg(conn->params.local->socket, &msg,
    - iov, 2, len);
    - conn->params.peer->last_tx_at = ktime_get_seconds();
    -
    - opt = IP_PMTUDISC_DO;
    - kernel_setsockopt(conn->params.local->socket, SOL_IP,
    - IP_MTU_DISCOVER,
    - (char *)&opt, sizeof(opt));
    - }
    - break;
    -
    -#ifdef CONFIG_AF_RXRPC_IPV6
    - case AF_INET6:
    - opt = IPV6_PMTUDISC_DONT;
    - ret = kernel_setsockopt(conn->params.local->socket,
    - SOL_IPV6, IPV6_MTU_DISCOVER,
    - (char *)&opt, sizeof(opt));
    - if (ret == 0) {
    - ret = kernel_sendmsg(conn->params.local->socket, &msg,
    - iov, 2, len);
    - conn->params.peer->last_tx_at = ktime_get_seconds();
    + kernel_setsockopt(conn->params.local->socket,
    + SOL_IP, IP_MTU_DISCOVER,
    + (char *)&opt, sizeof(opt));
    + ret = kernel_sendmsg(conn->params.local->socket, &msg,
    + iov, 2, len);
    + conn->params.peer->last_tx_at = ktime_get_seconds();

    - opt = IPV6_PMTUDISC_DO;
    - kernel_setsockopt(conn->params.local->socket,
    - SOL_IPV6, IPV6_MTU_DISCOVER,
    - (char *)&opt, sizeof(opt));
    - }
    + opt = IP_PMTUDISC_DO;
    + kernel_setsockopt(conn->params.local->socket,
    + SOL_IP, IP_MTU_DISCOVER,
    + (char *)&opt, sizeof(opt));
    break;
    -#endif

    default:
    BUG();

    \
     
     \ /
      Last update: 2020-05-01 15:52    [W:4.073 / U:0.104 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site