Re: [PATCH] block layer: elv_iosched_show should get elv_list_lock

From: Jens Axboe
Date: Wed Oct 11 2006 - 07:23:25 EST


On Wed, Oct 11 2006, Vasily Tarasov wrote:
> elv_iosched_show function iterates other elv_list,
> hence elv_list_lock should be got.

Indeed, that's a bug. Thanks!

> Also the question is: in elv_iosched_show, elv_iosched_store
> q->elevator->elevator_type construction is used without locking
> q->queue_lock. Is it expected?..

As long as the queue doesn't go away, we should be safe enough.

--
Jens Axboe

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/