block: remove bogus check for queue_lock assignment
authorJens Axboe <axboe@kernel.dk>
Fri, 12 Oct 2018 15:24:57 +0000 (09:24 -0600)
committerJens Axboe <axboe@kernel.dk>
Sun, 14 Oct 2018 18:49:40 +0000 (12:49 -0600)
We just allocated the queue and haven't even set it up yet,
hence we know that checking if ->mq_ops is NULL is always
going to be true.

In fact we do need to assign a lock to ->queue_lock always,
as we need it for the queue flags modifications.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/blk-core.c

index c5539eed02024c865cb0c0a70363d27caf497c5c..cdfabc5646da99e3afaa60699dc33eb0b5c1fe2d 100644 (file)
@@ -1054,8 +1054,7 @@ struct request_queue *blk_alloc_queue_node(gfp_t gfp_mask, int node_id,
        mutex_init(&q->sysfs_lock);
        spin_lock_init(&q->__queue_lock);
 
-       if (!q->mq_ops)
-               q->queue_lock = lock ? : &q->__queue_lock;
+       q->queue_lock = lock ? : &q->__queue_lock;
 
        /*
         * A queue starts its life with bypass turned on to avoid