diff options
author | Jens Axboe <axboe@fb.com> | 2014-11-17 18:41:57 +0100 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2014-11-17 18:41:57 +0100 |
commit | 7c7f2f2bc9a63f9605a16eabac59fc655dfe7c9a (patch) | |
tree | 5d3f60e0b52447eded55ebf8d649e19f0b4f7471 /block/blk-mq.c | |
parent | blk-mq: export blk_mq_free_request() (diff) | |
download | linux-7c7f2f2bc9a63f9605a16eabac59fc655dfe7c9a.tar.xz linux-7c7f2f2bc9a63f9605a16eabac59fc655dfe7c9a.zip |
blk-mq: add blk_mq_free_hctx_request()
It's silly to use blk_mq_free_request() which in turn maps the
request to the hardware queue, for places where we already know
what the hardware queue is. This saves us an extra mapping of a
hardware queue on request completion, if the caller knows this
information already.
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'block/blk-mq.c')
-rw-r--r-- | block/blk-mq.c | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/block/blk-mq.c b/block/blk-mq.c index fdf12152946e..4347aa2be6ae 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c @@ -269,16 +269,23 @@ static void __blk_mq_free_request(struct blk_mq_hw_ctx *hctx, blk_mq_queue_exit(q); } -void blk_mq_free_request(struct request *rq) +void blk_mq_free_hctx_request(struct blk_mq_hw_ctx *hctx, struct request *rq) { struct blk_mq_ctx *ctx = rq->mq_ctx; - struct blk_mq_hw_ctx *hctx; - struct request_queue *q = rq->q; ctx->rq_completed[rq_is_sync(rq)]++; - - hctx = q->mq_ops->map_queue(q, ctx->cpu); __blk_mq_free_request(hctx, ctx, rq); + +} +EXPORT_SYMBOL_GPL(blk_mq_free_hctx_request); + +void blk_mq_free_request(struct request *rq) +{ + struct blk_mq_hw_ctx *hctx; + struct request_queue *q = rq->q; + + hctx = q->mq_ops->map_queue(q, rq->mq_ctx->cpu); + blk_mq_free_hctx_request(hctx, rq); } EXPORT_SYMBOL_GPL(blk_mq_free_request); |