diff options
author | Jeff Moyer <jmoyer@redhat.com> | 2008-04-29 04:03:48 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-29 11:06:29 -0400 |
commit | 39fa00311f21318cc498b139c2cc2830dcad98ff (patch) | |
tree | 936eeb3a4046fb9b17cc4297e3b783f49c0a1dc0 /fs | |
parent | 68ab3d883a2df13f4b93a923bae3a287cbee29d3 (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>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/aio.c | 5 |
1 files changed, 1 insertions, 4 deletions
@@ -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 | */ | ||
559 | static struct kioctx *lookup_ioctx(unsigned long ctx_id) | 556 | static struct kioctx *lookup_ioctx(unsigned long ctx_id) |
560 | { | 557 | { |
561 | struct kioctx *ioctx; | 558 | struct kioctx *ioctx; |