* [PATCH 0/3] net: remove redundant __GFP_NOWARN
@ 2025-08-10 7:29 Qianfeng Rong
2025-08-10 7:29 ` [PATCH 3/3] SUNRPC: Remove " Qianfeng Rong
0 siblings, 1 reply; 3+ messages in thread
From: Qianfeng Rong @ 2025-08-10 7:29 UTC (permalink / raw)
To: Eric Dumazet, Neal Cardwell, Kuniyuki Iwashima, David S. Miller,
David Ahern, Jakub Kicinski, Paolo Abeni, Simon Horman,
Allison Henderson, Trond Myklebust, Anna Schumaker, Chuck Lever,
Jeff Layton, NeilBrown, Olga Kornievskaia, Dai Ngo, Tom Talpey,
Qianfeng Rong, open list:NETWORKING [TCP], open list,
open list:RDS - RELIABLE DATAGRAM SOCKETS,
moderated list:RDS - RELIABLE DATAGRAM SOCKETS,
open list:NFS, SUNRPC, AND LOCKD CLIENTS
Commit 16f5dfbc851b ("gfp: include __GFP_NOWARN in GFP_NOWAIT") made
GFP_NOWAIT implicitly include __GFP_NOWARN.
Therefore, explicit __GFP_NOWARN combined with GFP_NOWAIT (e.g.,
`GFP_NOWAIT | __GFP_NOWARN`) is now redundant. Let's clean up these
redundant flags across subsystems.
No functional changes.
Qianfeng Rong (3):
tcp: cdg: remove redundant __GFP_NOWARN
RDS: remove redundant __GFP_NOWARN
SUNRPC: Remove redundant __GFP_NOWARN
net/ipv4/tcp_cdg.c | 2 +-
net/rds/ib_recv.c | 2 +-
net/sunrpc/socklib.c | 2 +-
net/sunrpc/xprtrdma/rpc_rdma.c | 2 +-
4 files changed, 4 insertions(+), 4 deletions(-)
--
2.34.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 3/3] SUNRPC: Remove redundant __GFP_NOWARN
2025-08-10 7:29 [PATCH 0/3] net: remove redundant __GFP_NOWARN Qianfeng Rong
@ 2025-08-10 7:29 ` Qianfeng Rong
2025-08-10 15:34 ` Chuck Lever
0 siblings, 1 reply; 3+ messages in thread
From: Qianfeng Rong @ 2025-08-10 7:29 UTC (permalink / raw)
To: Chuck Lever, Jeff Layton, NeilBrown, Olga Kornievskaia, Dai Ngo,
Tom Talpey, Trond Myklebust, Anna Schumaker, David S. Miller,
Eric Dumazet, Jakub Kicinski, Paolo Abeni, Simon Horman,
Qianfeng Rong, open list:KERNEL NFSD, SUNRPC, AND LOCKD SERVERS,
open list:NETWORKING [GENERAL], open list
GFP_NOWAIT already includes __GFP_NOWARN, so let's remove the redundant
__GFP_NOWARN.
Signed-off-by: Qianfeng Rong <rongqianfeng@vivo.com>
---
net/sunrpc/socklib.c | 2 +-
net/sunrpc/xprtrdma/rpc_rdma.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/sunrpc/socklib.c b/net/sunrpc/socklib.c
index 4e92e2a50168..d8d8842c7de5 100644
--- a/net/sunrpc/socklib.c
+++ b/net/sunrpc/socklib.c
@@ -86,7 +86,7 @@ xdr_partial_copy_from_skb(struct xdr_buf *xdr, struct xdr_skb_reader *desc)
/* ACL likes to be lazy in allocating pages - ACLs
* are small by default but can get huge. */
if ((xdr->flags & XDRBUF_SPARSE_PAGES) && *ppage == NULL) {
- *ppage = alloc_page(GFP_NOWAIT | __GFP_NOWARN);
+ *ppage = alloc_page(GFP_NOWAIT);
if (unlikely(*ppage == NULL)) {
if (copied == 0)
return -ENOMEM;
diff --git a/net/sunrpc/xprtrdma/rpc_rdma.c b/net/sunrpc/xprtrdma/rpc_rdma.c
index 1478c41c7e9d..3aac1456e23e 100644
--- a/net/sunrpc/xprtrdma/rpc_rdma.c
+++ b/net/sunrpc/xprtrdma/rpc_rdma.c
@@ -190,7 +190,7 @@ rpcrdma_alloc_sparse_pages(struct xdr_buf *buf)
ppages = buf->pages + (buf->page_base >> PAGE_SHIFT);
while (len > 0) {
if (!*ppages)
- *ppages = alloc_page(GFP_NOWAIT | __GFP_NOWARN);
+ *ppages = alloc_page(GFP_NOWAIT);
if (!*ppages)
return -ENOBUFS;
ppages++;
--
2.34.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 3/3] SUNRPC: Remove redundant __GFP_NOWARN
2025-08-10 7:29 ` [PATCH 3/3] SUNRPC: Remove " Qianfeng Rong
@ 2025-08-10 15:34 ` Chuck Lever
0 siblings, 0 replies; 3+ messages in thread
From: Chuck Lever @ 2025-08-10 15:34 UTC (permalink / raw)
To: Qianfeng Rong, Jeff Layton, NeilBrown, Olga Kornievskaia, Dai Ngo,
Tom Talpey, Trond Myklebust, Anna Schumaker, David S. Miller,
Eric Dumazet, Jakub Kicinski, Paolo Abeni, Simon Horman,
open list:KERNEL NFSD, SUNRPC, AND LOCKD SERVERS,
open list:NETWORKING [GENERAL], open list
On 8/10/25 3:29 AM, Qianfeng Rong wrote:
> GFP_NOWAIT already includes __GFP_NOWARN, so let's remove the redundant
> __GFP_NOWARN.
>
> Signed-off-by: Qianfeng Rong <rongqianfeng@vivo.com>
> ---
> net/sunrpc/socklib.c | 2 +-
> net/sunrpc/xprtrdma/rpc_rdma.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/net/sunrpc/socklib.c b/net/sunrpc/socklib.c
> index 4e92e2a50168..d8d8842c7de5 100644
> --- a/net/sunrpc/socklib.c
> +++ b/net/sunrpc/socklib.c
> @@ -86,7 +86,7 @@ xdr_partial_copy_from_skb(struct xdr_buf *xdr, struct xdr_skb_reader *desc)
> /* ACL likes to be lazy in allocating pages - ACLs
> * are small by default but can get huge. */
> if ((xdr->flags & XDRBUF_SPARSE_PAGES) && *ppage == NULL) {
> - *ppage = alloc_page(GFP_NOWAIT | __GFP_NOWARN);
> + *ppage = alloc_page(GFP_NOWAIT);
> if (unlikely(*ppage == NULL)) {
> if (copied == 0)
> return -ENOMEM;
> diff --git a/net/sunrpc/xprtrdma/rpc_rdma.c b/net/sunrpc/xprtrdma/rpc_rdma.c
> index 1478c41c7e9d..3aac1456e23e 100644
> --- a/net/sunrpc/xprtrdma/rpc_rdma.c
> +++ b/net/sunrpc/xprtrdma/rpc_rdma.c
> @@ -190,7 +190,7 @@ rpcrdma_alloc_sparse_pages(struct xdr_buf *buf)
> ppages = buf->pages + (buf->page_base >> PAGE_SHIFT);
> while (len > 0) {
> if (!*ppages)
> - *ppages = alloc_page(GFP_NOWAIT | __GFP_NOWARN);
> + *ppages = alloc_page(GFP_NOWAIT);
> if (!*ppages)
> return -ENOBUFS;
> ppages++;
Acked-by: Chuck Lever <chuck.lever@oracle.com>
--
Chuck Lever
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-08-10 15:35 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-10 7:29 [PATCH 0/3] net: remove redundant __GFP_NOWARN Qianfeng Rong
2025-08-10 7:29 ` [PATCH 3/3] SUNRPC: Remove " Qianfeng Rong
2025-08-10 15:34 ` Chuck Lever
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).