aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJeff Moyer <jmoyer@redhat.com>2008-04-29 04:03:48 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2008-04-29 11:06:29 -0400
commit39fa00311f21318cc498b139c2cc2830dcad98ff (patch)
tree936eeb3a4046fb9b17cc4297e3b783f49c0a1dc0
parent68ab3d883a2df13f4b93a923bae3a287cbee29d3 (diff)
aio: fix misleading comments
The FIXME comments are inaccurate. The locking comment over lookup_ioctx() is wrong. Signed-off-by: Jeff Moyer <jmoyer@redhat.com> Signed-off-by: Zach Brown <zach.brown@oracle.com> Signed-off-by: Shen Feng <shen@cn.fujitsu.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--fs/aio.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/fs/aio.c b/fs/aio.c
index a175ad650b66..99c2352906a0 100644
--- a/fs/aio.c
+++ b/fs/aio.c
@@ -277,7 +277,7 @@ static struct kioctx *ioctx_alloc(unsigned nr_events)
277 if (ctx->max_reqs == 0) 277 if (ctx->max_reqs == 0)
278 goto out_cleanup; 278 goto out_cleanup;
279 279
280 /* now link into global list. kludge. FIXME */ 280 /* now link into global list. */
281 write_lock(&mm->ioctx_list_lock); 281 write_lock(&mm->ioctx_list_lock);
282 ctx->next = mm->ioctx_list; 282 ctx->next = mm->ioctx_list;
283 mm->ioctx_list = ctx; 283 mm->ioctx_list = ctx;
@@ -553,9 +553,6 @@ int aio_put_req(struct kiocb *req)
553 return ret; 553 return ret;
554} 554}
555 555
556/* Lookup an ioctx id. ioctx_list is lockless for reads.
557 * FIXME: this is O(n) and is only suitable for development.
558 */
559static struct kioctx *lookup_ioctx(unsigned long ctx_id) 556static struct kioctx *lookup_ioctx(unsigned long ctx_id)
560{ 557{
561 struct kioctx *ioctx; 558 struct kioctx *ioctx;