diff options
| author | Jens Axboe <axboe@kernel.dk> | 2018-11-19 00:15:35 +0100 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2018-11-26 16:26:04 +0100 |
| commit | 4ab32bf3305eedb4d31f85cac68a67becab10494 (patch) | |
| tree | d75d39772276a9be0ebadf1bc24dcf669007dd85 /block | |
| parent | blk-mq: ensure mq_ops ->poll() is entered at least once (diff) | |
| download | linux-4ab32bf3305eedb4d31f85cac68a67becab10494.tar.xz linux-4ab32bf3305eedb4d31f85cac68a67becab10494.zip | |
blk-mq: never redirect polled IO completions
It's pointless to do so, we are by definition on the CPU we want/need
to be, as that's the one waiting for a completion event.
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block')
| -rw-r--r-- | block/blk-mq.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/block/blk-mq.c b/block/blk-mq.c index ba3c7b6476b7..37674c1766a7 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c @@ -585,7 +585,12 @@ static void __blk_mq_complete_request(struct request *rq) return; } - if (!test_bit(QUEUE_FLAG_SAME_COMP, &q->queue_flags)) { + /* + * For a polled request, always complete locallly, it's pointless + * to redirect the completion. + */ + if ((rq->cmd_flags & REQ_HIPRI) || + !test_bit(QUEUE_FLAG_SAME_COMP, &q->queue_flags)) { q->mq_ops->complete(rq); return; } |