From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B8B7BC9830E for ; Wed, 30 Sep 2026 06:04:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:Subject:References:In-Reply-To:Message-Id:Cc:To:From:Date: MIME-Version:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=D/ibpmI5jg1a9kUxODZYU60ZnHvdobAWMypDGAeOo8U=; b=thr6+E6NaJF6x86hxBmVUBH7K+ MxzkYo89zFDZ6ku5p9gGr0z/XJaEDKW+9krp+zOju6hyUh3NkCfbbhYXMMr1eWlSHoQq0t0BWCPOL 3okr1y5QScWeMfhVSvQk09uwY3xN8IYRaTUKKJlqXrxiHK0RqzR/ih3igyb+SYD8XfORtwU2y+f/L yKIIQYSdrEhN6gI+tLB2AwbfTaheeAE3jb98jk/84GySChBUVVHQABeHPxqJRJSN4GhBPx9I2mzNv 0xNjjkElezGab8XN8tpHBELsrHmRfLIHDDqjtagwIdVkLVFw0zYjWTSb53huSE9qcf0PVcQfo6B1+ s0Z6ehJw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xBnQp-000000055rj-3qIX; Wed, 30 Sep 2026 06:04:43 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xBZev-00000003w7P-1f04 for linux-nvme@lists.infradead.org; Tue, 29 Sep 2026 15:22:21 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 9D7D543D99; Tue, 29 Sep 2026 15:22:18 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 30CDC1F00893; Tue, 29 Sep 2026 15:22:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1790695338; bh=D/ibpmI5jg1a9kUxODZYU60ZnHvdobAWMypDGAeOo8U=; h=Date:From:To:Cc:In-Reply-To:References:Subject; b=anpnVCGKi06BDE9uCt53/px/V/Xclaq+d8LO4yKPPxUSJgf0wPpBbfQ0wAIHNm4vj 8Vpn5/NxGqe11Gs10voadlisgh3IYeFXu3g+2SJ+7m3jil4LXAdac5/a4Xipc/L0uU uoM6DQpX+wkJM1Xtnp6/eELm3sKLoLKt7BKykQRKJe1I4BAVkcxtL+mbfq5coubEnd AY+5VfQvWQPP+hPh1TvjrAUVXIVRD2udsWqQvO1pm+PtRRNyRs2M8dIsyxgxYbw2fe VVLle+rliswZIrczTvPrFf+OJ3iwkeEU9REFJ1NId2+NyoFakGOZ6TSqsqwJWcI7hD VZ8XSrwjo6+2w== Received: from phl-compute-10.internal (phl-compute-10.internal [10.202.2.50]) by mailfauth.phl.internal (Postfix) with ESMTP id 30CFCF40066; Tue, 29 Sep 2026 11:22:16 -0400 (EDT) Received: from phl-imap-15 ([10.202.2.104]) by phl-compute-10.internal (MEProxy); Tue, 29 Sep 2026 11:22:16 -0400 X-ME-Sender: X-ME-Proxy-Cause: dmFkZTGHwpvTWnenfdWwDpfZ0uSchfAvPs9YUs2fCTjyTe1/aaP/8o/CAtZEpuBnFlnr3l qmTkjOPiHUmeNeOJhaD8lnKlk7EMaLK7X5jKQpwMypkRiS4dvAs3YrNT/c5V6Bk4Ga8MRb xPq/javh4ijLGFfdkpos6GxlxyGT42Fwa0ltNIGopZ+Mq1Abfhj0eHx9Lz1wE2F5x+0msE VuUjojmKduIXDgawsedlXFbIOAX2g6XJhtdhhvodBfsk+7R/r5zcjPGlwzTbqow/k0fhhB KSkSmJEw1/zMzycRq5vhO53cykwQS/OPCRkkdIvcZDxPRXq49tm4EIS2v6w4wGkS37Tlih UB9WCh5UmtBTKVoM5jZdVRPrR1i0Y/KGM6FHAXdqhHo3g4SwJzfP4kS1hl+kp6sqa3oe5B 09PI362u5aRRUNoro1OofwHkHSzL/FrFq/CJzhCpmp9MwcGwLzvwm0apSMBCms1xR1uIii arKpKGXFmq7VC4eJ8+APDkopGyFyqzktnwewJR+EDrFxuv09iKewKqVtwVJeGBzDK6vRKq muINJMxJgehj51nQY/hRISHFHFnxUA4qXxqSbYIxuozvMtbIXSmX4t9k0Hmo9w1FDElKMl aRA8ntUnTlF314tfn5h0SudAXalF4N4JRAjlbU/knq2/TyidO7gi2AcylzGg X-ME-Proxy: Feedback-ID: ifa6e4810:Fastmail Received: by mailuser.phl.internal (Postfix, from userid 501) id E740A780070; Tue, 29 Sep 2026 11:22:15 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface MIME-Version: 1.0 X-ThreadId: AozynqqzZJs- Date: Tue, 29 Sep 2026 08:21:55 -0700 From: "Chuck Lever" To: "Eric Dumazet" , "David S . Miller" , "Jakub Kicinski" , "Paolo Abeni" Cc: "Simon Horman" , "Neal Cardwell" , "Kuniyuki Iwashima" , edumazet@google.com, netdev@vger.kernel.org, "Alexander Aring" , "David Teigland" , gfs2@lists.linux.dev, "John Fastabend" , "Jakub Sitnicki" , "Sabrina Dubroca" , "Jiayuan Chen" , "Matthieu Baerts" , "Mat Martineau" , "Geliang Tang" , mptcp@lists.linux.dev, "Wen Gu" , "Dust Li" , "D. Wythe" , "Jeff Layton" , NeilBrown , "Olga Kornievskaia" , "Dai Ngo" , "Tom Talpey" , "Trond Myklebust" , "Anna Schumaker" , linux-nfs@vger.kernel.org, "Allison Henderson" , rds-devel@oss.oracle.com, "Philipp Reisner" , "Lars Ellenberg" , =?UTF-8?Q?Christoph_B=C3=B6hmwalder?= , "Jens Axboe" , drbd-dev@lists.linux.dev, "Keith Busch" , "Christoph Hellwig" , "Sagi Grimberg" , "Chaitanya Kulkarni" , linux-nvme@lists.infradead.org, "Ilya Dryomov" , "Alex Markuze" , "Viacheslav Dubeyko" , ceph-devel@vger.kernel.org Message-Id: <9a68f8a0-474c-44f4-97fa-bbf05b6de529@app.fastmail.com> In-Reply-To: <20260929071743.23624-4-edumazet@kernel.org> References: <20260929071743.23624-1-edumazet@kernel.org> <20260929071743.23624-4-edumazet@kernel.org> Subject: Re: [PATCH v3 net-next 3/9] sunrpc: use sk_set_nospace() and sk_clear_nospace() Content-Type: text/plain Content-Transfer-Encoding: 7bit X-Mailman-Approved-At: Tue, 29 Sep 2026 23:04:42 -0700 X-BeenThere: linux-nvme@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-nvme" Errors-To: linux-nvme-bounces+linux-nvme=archiver.kernel.org@lists.infradead.org On Tue, Sep 29, 2026, at 12:17 AM, Eric Dumazet wrote: > Use the new helpers instead of open coding the SOCK_NOSPACE > manipulation, so that TCP can later maintain a cheaper private > copy of this bit. > > No functional change intended. > > Reviewed-by: Kuniyuki Iwashima > Signed-off-by: Eric Dumazet > --- > net/sunrpc/svcsock.c | 4 ++-- > net/sunrpc/xprtsock.c | 4 ++-- > 2 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/net/sunrpc/svcsock.c b/net/sunrpc/svcsock.c > index > 50e5e7f5b762de28c34d0f58cb0c6feb1e3ce79f..2d8cdee0798d540ba8eae589deaac5bd3ff889fa > 100644 > --- a/net/sunrpc/svcsock.c > +++ b/net/sunrpc/svcsock.c > @@ -792,11 +792,11 @@ static int svc_udp_has_wspace(struct svc_xprt > *xprt) > * Set the SOCK_NOSPACE flag before checking the available > * sock space. > */ > - set_bit(SOCK_NOSPACE, &svsk->sk_sock->flags); > + sk_set_nospace(svsk->sk_sk); > required = atomic_read(&svsk->sk_xprt.xpt_reserved) + > serv->sv_max_mesg; > if (required*2 > sock_wspace(svsk->sk_sk)) > return 0; > - clear_bit(SOCK_NOSPACE, &svsk->sk_sock->flags); > + sk_clear_nospace(svsk->sk_sk); > return 1; > } > > diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c > index > 7f60723fa64d84887e260e5e130e44cdbec6f850..b97c70c12f627630509510facfd3c3cf96ab6cb6 > 100644 > --- a/net/sunrpc/xprtsock.c > +++ b/net/sunrpc/xprtsock.c > @@ -858,7 +858,7 @@ static int xs_nospace(struct rpc_rqst *req, struct > sock_xprt *transport) > if (xprt_connected(xprt)) { > /* wait for more buffer space */ > set_bit(XPRT_SOCK_NOSPACE, &transport->sock_state); > - set_bit(SOCK_NOSPACE, &sk->sk_socket->flags); > + sk_set_nospace(sk); > sk->sk_write_pending++; > xprt_wait_for_buffer_space(xprt); > } else > @@ -1615,7 +1615,7 @@ static void xs_write_space(struct sock *sk) > > if (!sk->sk_socket) > return; > - clear_bit(SOCK_NOSPACE, &sk->sk_socket->flags); > + sk_clear_nospace(sk); > > if (unlikely(!(xprt = xprt_from_sock(sk)))) > return; > -- > 2.56.0.rc1.315.gc6ed9934b7-goog For the svcsock.c hunks of this patch: Acked-by: Chuck Lever -- Chuck Lever (Come to NFS bake-a-thon! https://nfsv4bat.org)