summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSébastien Dugu <sebastien.dugue@bull.net>2005-06-28 20:44:59 -0700
committerLinus Torvalds <torvalds@ppc970.osdl.org>2005-06-28 21:20:32 -0700
commitc016e2257acd00a7ffd87fa1eec896138563d1aa (patch)
treeaa22df355afe2be5b5476c1c6d26da99e47836fc
parent334a13ec3d01a1a4b4f2249735b793105cb4a519 (diff)
downloadop-kernel-dev-c016e2257acd00a7ffd87fa1eec896138563d1aa.zip
op-kernel-dev-c016e2257acd00a7ffd87fa1eec896138563d1aa.tar.gz
[PATCH] aio-retry-fix: fix aio retry work queueing
In the case of buffered AIO, in the aio retry path (aio_run_iocb), when the retry method returns EIOCBRETRY the kicked iocb is added to the context run list but is never queued onto the work queue. The request therefore is never completed. This patch fixes that by adding the appropriate call to aio_queue_work in aio_run_aiocb so that subsequent retries will be handled by the aio worker thread. Signed-off-by: Sébastien Dugué <sebastien.dugue@bull.net> Acked-by: Benjamin LaHaise <benjamin.c.lahaise@intel.com> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r--fs/aio.c9
1 files changed, 9 insertions, 0 deletions
diff --git a/fs/aio.c b/fs/aio.c
index 7afa222..06d7d43 100644
--- a/fs/aio.c
+++ b/fs/aio.c
@@ -58,6 +58,7 @@ static DEFINE_SPINLOCK(fput_lock);
static LIST_HEAD(fput_head);
static void aio_kick_handler(void *);
+static void aio_queue_work(struct kioctx *);
/* aio_setup
* Creates the slab caches used by the aio routines, panic on
@@ -747,6 +748,14 @@ out:
* has already been kicked */
if (kiocbIsKicked(iocb)) {
__queue_kicked_iocb(iocb);
+
+ /*
+ * __queue_kicked_iocb will always return 1 here, because
+ * iocb->ki_run_list is empty at this point so it should
+ * be safe to unconditionally queue the context into the
+ * work queue.
+ */
+ aio_queue_work(ctx);
}
}
return ret;
OpenPOWER on IntegriCloud