diff options
author | Jens Axboe <axboe@kernel.dk> | 2017-11-11 00:05:12 -0500 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2017-11-11 00:05:12 -0500 |
commit | ff821d271415f17f1a704600420e92ebb9bfb32c (patch) | |
tree | 8db4380619fe82602bcb05e06a4ec0f0ed184a07 | |
parent | 8dc7a31fbce5e2dbbacd83d910da37105181b054 (diff) |
blk-mq: fixup some comment typos and lengths
Various typos and/or spelling errors in comments. Fixes a few > 80 char
lines as well.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
-rw-r--r-- | block/blk-mq.c | 17 |
1 files changed, 10 insertions, 7 deletions
diff --git a/block/blk-mq.c b/block/blk-mq.c index 3295859f419d..b600463791ec 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c | |||
@@ -707,7 +707,7 @@ void blk_mq_add_to_requeue_list(struct request *rq, bool at_head, | |||
707 | 707 | ||
708 | /* | 708 | /* |
709 | * We abuse this flag that is otherwise used by the I/O scheduler to | 709 | * We abuse this flag that is otherwise used by the I/O scheduler to |
710 | * request head insertation from the workqueue. | 710 | * request head insertion from the workqueue. |
711 | */ | 711 | */ |
712 | BUG_ON(rq->rq_flags & RQF_SOFTBARRIER); | 712 | BUG_ON(rq->rq_flags & RQF_SOFTBARRIER); |
713 | 713 | ||
@@ -1137,7 +1137,8 @@ bool blk_mq_dispatch_rq_list(struct request_queue *q, struct list_head *list, | |||
1137 | if (ret == BLK_STS_RESOURCE) { | 1137 | if (ret == BLK_STS_RESOURCE) { |
1138 | /* | 1138 | /* |
1139 | * If an I/O scheduler has been configured and we got a | 1139 | * If an I/O scheduler has been configured and we got a |
1140 | * driver tag for the next request already, free it again. | 1140 | * driver tag for the next request already, free it |
1141 | * again. | ||
1141 | */ | 1142 | */ |
1142 | if (!list_empty(list)) { | 1143 | if (!list_empty(list)) { |
1143 | nxt = list_first_entry(list, struct request, queuelist); | 1144 | nxt = list_first_entry(list, struct request, queuelist); |
@@ -2299,8 +2300,11 @@ static void blk_mq_add_queue_tag_set(struct blk_mq_tag_set *set, | |||
2299 | 2300 | ||
2300 | mutex_lock(&set->tag_list_lock); | 2301 | mutex_lock(&set->tag_list_lock); |
2301 | 2302 | ||
2302 | /* Check to see if we're transitioning to shared (from 1 to 2 queues). */ | 2303 | /* |
2303 | if (!list_empty(&set->tag_list) && !(set->flags & BLK_MQ_F_TAG_SHARED)) { | 2304 | * Check to see if we're transitioning to shared (from 1 to 2 queues). |
2305 | */ | ||
2306 | if (!list_empty(&set->tag_list) && | ||
2307 | !(set->flags & BLK_MQ_F_TAG_SHARED)) { | ||
2304 | set->flags |= BLK_MQ_F_TAG_SHARED; | 2308 | set->flags |= BLK_MQ_F_TAG_SHARED; |
2305 | /* update existing queue */ | 2309 | /* update existing queue */ |
2306 | blk_mq_update_tag_set_depth(set, true); | 2310 | blk_mq_update_tag_set_depth(set, true); |
@@ -2532,10 +2536,9 @@ static void blk_mq_queue_reinit(struct request_queue *q) | |||
2532 | 2536 | ||
2533 | /* | 2537 | /* |
2534 | * redo blk_mq_init_cpu_queues and blk_mq_init_hw_queues. FIXME: maybe | 2538 | * redo blk_mq_init_cpu_queues and blk_mq_init_hw_queues. FIXME: maybe |
2535 | * we should change hctx numa_node according to new topology (this | 2539 | * we should change hctx numa_node according to the new topology (this |
2536 | * involves free and re-allocate memory, worthy doing?) | 2540 | * involves freeing and re-allocating memory, worth doing?) |
2537 | */ | 2541 | */ |
2538 | |||
2539 | blk_mq_map_swqueue(q); | 2542 | blk_mq_map_swqueue(q); |
2540 | 2543 | ||
2541 | blk_mq_sysfs_register(q); | 2544 | blk_mq_sysfs_register(q); |