]> bbs.cooldavid.org Git - net-next-2.6.git/commit - fs/ioprio.c
[PATCH] Fix current_io_context() vs set_task_ioprio() race
authorOleg Nesterov <oleg@tv-sign.ru>
Mon, 21 Aug 2006 06:34:15 +0000 (08:34 +0200)
committerJens Axboe <axboe@suse.de>
Mon, 21 Aug 2006 06:34:15 +0000 (08:34 +0200)
commit9f83e45eb54fc7198dc59fc63255341851ba4c48
tree3874581e08052b7f37f164110b3bd44ae5eff291
parent78bd4d484f81a611ef6ff02f909e576cb9aac7f2
[PATCH] Fix current_io_context() vs set_task_ioprio() race

I know nothing about io scheduler, but I suspect set_task_ioprio() is not safe.

current_io_context() initializes "struct io_context", then sets ->io_context.
set_task_ioprio() running on another cpu may see the changes out of order, so
->set_ioprio(ioc) may use io_context which was not initialized properly.

Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Signed-off-by: Jens Axboe <axboe@suse.de>
block/ll_rw_blk.c
fs/ioprio.c