block, bfq: remove redundant __GFP_NOWARN

qianfengrong · axboe · commit 8f3e4e87b094 · 2025-08-11T07:59:22.000-06:00
Commit 16f5dfb ("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. Reviewed-by: Yu Kuai <yukuai3@huawei.com> Signed-off-by: Qianfeng Rong <rongqianfeng@vivo.com> Link: https://lore.kernel.org/r/20250811081135.374315-1-rongqianfeng@vivo.com Signed-off-by: Jens Axboe <axboe@kernel.dk>
diff --git a/block/bfq-iosched.c b/block/bfq-iosched.c
@@ -5847,8 +5847,7 @@ static struct bfq_queue *bfq_get_queue(struct bfq_data *bfqd,
 			goto out;
 	}
 
-	bfqq = kmem_cache_alloc_node(bfq_pool,
-				     GFP_NOWAIT | __GFP_ZERO | __GFP_NOWARN,
+	bfqq = kmem_cache_alloc_node(bfq_pool, GFP_NOWAIT | __GFP_ZERO,
 				     bfqd->queue->node);
 
 	if (bfqq) {

Original file line number	Diff line number	Diff line change
`@@ -5847,8 +5847,7 @@ static struct bfq_queue bfq_get_queue(struct bfq_data bfqd,`
`5847`	`5847`	`goto out;`
`5848`	`5848`	`}`
`5849`	`5849`
`5850`		`- bfqq = kmem_cache_alloc_node(bfq_pool,`
`5851`		`- GFP_NOWAIT \| __GFP_ZERO \| __GFP_NOWARN,`
	`5850`	`+ bfqq = kmem_cache_alloc_node(bfq_pool, GFP_NOWAIT \| __GFP_ZERO,`
`5852`	`5851`	`bfqd->queue->node);`
`5853`	`5852`
`5854`	`5853`	`if (bfqq) {`