block: clarify blk_add_timer() use case for blk-mq
authorJens Axboe <axboe@fb.com>
Tue, 24 Nov 2015 22:58:53 +0000 (15:58 -0700)
committerJens Axboe <axboe@fb.com>
Tue, 24 Nov 2015 22:58:53 +0000 (15:58 -0700)
Just a comment update on not needing queue_lock, and that we aren't
really adding the request to a timeout list for !mq.

Signed-off-by: Jens Axboe <axboe@fb.com>
block/blk-timeout.c

index aa40aa93381b661b73d5aa0f531e247e148c56a0..3610af5617488a9dbfec02511972c8f52d75f20f 100644 (file)
@@ -186,6 +186,7 @@ unsigned long blk_rq_timeout(unsigned long timeout)
  * Notes:
  *    Each request has its own timer, and as it is added to the queue, we
  *    set up the timer. When the request completes, we cancel the timer.
+ *    Queue lock must be held for the non-mq case, mq case doesn't care.
  */
 void blk_add_timer(struct request *req)
 {
@@ -209,6 +210,11 @@ void blk_add_timer(struct request *req)
                req->timeout = q->rq_timeout;
 
        req->deadline = jiffies + req->timeout;
+
+       /*
+        * Only the non-mq case needs to add the request to a protected list.
+        * For the mq case we simply scan the tag map.
+        */
        if (!q->mq_ops)
                list_add_tail(&req->timeout_list, &req->q->timeout_list);