[Devel] [PATCH RH8] aio: ioctl(VE_AIO_IOC_WAIT_ACTIVE) in-flight reqs counting fix
Alexander Mikhalitsyn
alexander.mikhalitsyn at virtuozzo.com
Fri Apr 23 13:57:21 MSK 2021
We have to take into account percpu part of reqs_available
counter on struct kioctx.
Fixes: f5d1279 ("ve/aio: Add a handle to checkpoint/restore AIO context")
https://jira.sw.ru/browse/PSBM-128710
Signed-off-by: Alexander Mikhalitsyn <alexander.mikhalitsyn at virtuozzo.com>
---
fs/aio.c | 12 +++++++++++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/fs/aio.c b/fs/aio.c
index 7c547247b056..4ae0cac0f3ff 100644
--- a/fs/aio.c
+++ b/fs/aio.c
@@ -1898,9 +1898,19 @@ static bool has_reqs_active(struct kioctx *ctx)
{
unsigned long flags;
unsigned nr;
+ int cpu;
+ unsigned reqs_avail_batch = 0;
spin_lock_irqsave(&ctx->completion_lock, flags);
- nr = (ctx->nr_events - 1) - atomic_read(&ctx->reqs_available);
+ /*
+ * See get_reqs_available()/put_reqs_available() about
+ * how reqs_available distributed between atomic
+ * ctx->reqs_available and percpu ctx->cpu reqs_available.
+ */
+ for_each_possible_cpu(cpu)
+ reqs_avail_batch += per_cpu_ptr(ctx->cpu, cpu)->reqs_available;
+ nr = ctx->nr_events - 1;
+ nr -= atomic_read(&ctx->reqs_available) + reqs_avail_batch;
nr -= ctx->completed_events;
spin_unlock_irqrestore(&ctx->completion_lock, flags);
--
2.28.0
More information about the Devel
mailing list