Skip to content

Commit d73acd7

Browse files
isilenceaxboe
authored andcommitted
io_uring: rename "min" arg in io_iopoll_check()
Don't name arguments "min", it shadows the namesake function. min_events is also more consistent. Signed-off-by: Pavel Begunkov <[email protected]> Link: https://lore.kernel.org/r/f52ce9d88d3bca5732a218b0da14924aa6968909.1742829388.git.asml.silence@gmail.com Signed-off-by: Jens Axboe <[email protected]>
1 parent 4c76de4 commit d73acd7

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

io_uring/io_uring.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1505,7 +1505,7 @@ static __cold void io_iopoll_try_reap_events(struct io_ring_ctx *ctx)
15051505
mutex_unlock(&ctx->uring_lock);
15061506
}
15071507

1508-
static int io_iopoll_check(struct io_ring_ctx *ctx, long min)
1508+
static int io_iopoll_check(struct io_ring_ctx *ctx, long min_events)
15091509
{
15101510
unsigned int nr_events = 0;
15111511
unsigned long check_cq;
@@ -1551,7 +1551,7 @@ static int io_iopoll_check(struct io_ring_ctx *ctx, long min)
15511551
io_task_work_pending(ctx)) {
15521552
u32 tail = ctx->cached_cq_tail;
15531553

1554-
(void) io_run_local_work_locked(ctx, min);
1554+
(void) io_run_local_work_locked(ctx, min_events);
15551555

15561556
if (task_work_pending(current) ||
15571557
wq_list_empty(&ctx->iopoll_list)) {
@@ -1564,7 +1564,7 @@ static int io_iopoll_check(struct io_ring_ctx *ctx, long min)
15641564
wq_list_empty(&ctx->iopoll_list))
15651565
break;
15661566
}
1567-
ret = io_do_iopoll(ctx, !min);
1567+
ret = io_do_iopoll(ctx, !min_events);
15681568
if (unlikely(ret < 0))
15691569
return ret;
15701570

@@ -1574,7 +1574,7 @@ static int io_iopoll_check(struct io_ring_ctx *ctx, long min)
15741574
break;
15751575

15761576
nr_events += ret;
1577-
} while (nr_events < min);
1577+
} while (nr_events < min_events);
15781578

15791579
return 0;
15801580
}

0 commit comments

Comments
 (0)