We return 0 for success, or -error when there's an error. Move the 'ret'
variable into the loop where we are actually using it, to make it
clearer that we don't carry this variable forward for return outside of
the loop.
While at it, also move the need_resched() break condition out of the
while check itself, keeping it with the signal pending check.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
static int io_iopoll_check(struct io_ring_ctx *ctx, long min)
{
unsigned int nr_events = 0;
- int ret = 0;
unsigned long check_cq;
if (!io_allowed_run_tw(ctx))
return 0;
do {
+ int ret = 0;
+
/*
* If a submit got punted to a workqueue, we can have the
* application entering polling for a command before it gets
break;
}
ret = io_do_iopoll(ctx, !min);
- if (ret < 0)
- break;
- nr_events += ret;
- ret = 0;
+ if (unlikely(ret < 0))
+ return ret;
if (task_sigpending(current))
return -EINTR;
- } while (nr_events < min && !need_resched());
+ if (need_resched())
+ break;
- return ret;
+ nr_events += ret;
+ } while (nr_events < min);
+
+ return 0;
}
void io_req_task_complete(struct io_kiocb *req, struct io_tw_state *ts)