diff options
| author | Keith Busch | 2018-11-26 17:54:30 +0100 |
|---|---|---|
| committer | Jens Axboe | 2018-11-26 18:34:27 +0100 |
| commit | af78ff7c6e66832afcdf5418f67b11c409f9e7a1 (patch) | |
| tree | 22b7a66b3c246d2793180f31ffa062f6cea74e37 /include/linux/blk-mq.h | |
| parent | scsi: Do not rely on blk-mq for double completions (diff) | |
| download | kernel-qcow2-linux-af78ff7c6e66832afcdf5418f67b11c409f9e7a1.tar.gz kernel-qcow2-linux-af78ff7c6e66832afcdf5418f67b11c409f9e7a1.tar.xz kernel-qcow2-linux-af78ff7c6e66832afcdf5418f67b11c409f9e7a1.zip | |
blk-mq: Simplify request completion state
There are no more users relying on blk-mq request states to prevent
double completions, so replace the relatively expensive cmpxchg operation
with WRITE_ONCE.
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Keith Busch <keith.busch@intel.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'include/linux/blk-mq.h')
| -rw-r--r-- | include/linux/blk-mq.h | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/include/linux/blk-mq.h b/include/linux/blk-mq.h index 6e3da356a8eb..b8de11e0603b 100644 --- a/include/linux/blk-mq.h +++ b/include/linux/blk-mq.h @@ -329,20 +329,6 @@ void blk_mq_quiesce_queue_nowait(struct request_queue *q); unsigned int blk_mq_rq_cpu(struct request *rq); -/** - * blk_mq_mark_complete() - Set request state to complete - * @rq: request to set to complete state - * - * Returns true if request state was successfully set to complete. If - * successful, the caller is responsibile for seeing this request is ended, as - * blk_mq_complete_request will not work again. - */ -static inline bool blk_mq_mark_complete(struct request *rq) -{ - return cmpxchg(&rq->state, MQ_RQ_IN_FLIGHT, MQ_RQ_COMPLETE) == - MQ_RQ_IN_FLIGHT; -} - /* * Driver command data is immediately after the request. So subtract request * size to get back to the original request, add request size to get the PDU. |