Commit 52f7eb94 authored by Bart Van Assche's avatar Bart Van Assche Committed by Jens Axboe

blk-mq: Micro-optimize bt_get()

Remove a superfluous finish_wait() call. Convert the two bt_wait_ptr()
calls into a single call.
Signed-off-by: default avatarBart Van Assche <bvanassche@acm.org>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Robert Elliott <elliott@hp.com>
Cc: Ming Lei <ming.lei@canonical.com>
Cc: Alexander Gordeev <agordeev@redhat.com>
Signed-off-by: default avatarJens Axboe <axboe@fb.com>
parent c38d185d
...@@ -248,8 +248,8 @@ static int bt_get(struct blk_mq_alloc_data *data, ...@@ -248,8 +248,8 @@ static int bt_get(struct blk_mq_alloc_data *data,
if (!(data->gfp & __GFP_WAIT)) if (!(data->gfp & __GFP_WAIT))
return -1; return -1;
bs = bt_wait_ptr(bt, hctx);
do { do {
bs = bt_wait_ptr(bt, hctx);
prepare_to_wait(&bs->wait, &wait, TASK_UNINTERRUPTIBLE); prepare_to_wait(&bs->wait, &wait, TASK_UNINTERRUPTIBLE);
tag = __bt_get(hctx, bt, last_tag); tag = __bt_get(hctx, bt, last_tag);
...@@ -285,8 +285,6 @@ static int bt_get(struct blk_mq_alloc_data *data, ...@@ -285,8 +285,6 @@ static int bt_get(struct blk_mq_alloc_data *data,
hctx = data->hctx; hctx = data->hctx;
bt = &hctx->tags->bitmap_tags; bt = &hctx->tags->bitmap_tags;
} }
finish_wait(&bs->wait, &wait);
bs = bt_wait_ptr(bt, hctx);
} while (1); } while (1);
finish_wait(&bs->wait, &wait); finish_wait(&bs->wait, &wait);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment