[PATCH] as-iosched: remove state assertion in as_add_request()
authorJens Axboe <axboe@suse.de>
Mon, 21 Nov 2005 18:49:41 +0000 (19:49 +0100)
committerLinus Torvalds <torvalds@g5.osdl.org>
Mon, 21 Nov 2005 19:04:52 +0000 (11:04 -0800)
Kill the arq->state poison statement in as_add_request(), it can trigger
for perfectly valid code that just reuses a request after io completion
instead of freeing it and allocating a new one. We probably should
introduce a blk_init_request() to start from scratch, but for now just
kill it as we will be removing the as specific poisoning soon.

Signed-off-by: Jens Axboe <axboe@suse.de>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
block/as-iosched.c

index fbe0501..43fa204 100644 (file)
@@ -1371,10 +1371,6 @@ static void as_add_request(request_queue_t *q, struct request *rq)
        struct as_rq *alias;
        int data_dir;
 
-       if (arq->state != AS_RQ_PRESCHED) {
-               printk("arq->state: %d\n", arq->state);
-               WARN_ON(1);
-       }
        arq->state = AS_RQ_NEW;
 
        if (rq_data_dir(arq->request) == READ