From: Jens Axboe <axboe@fb.com>
Date: Thu, 9 Apr 2015 21:54:05 +0000 (-0600)
Subject: blk-mq: cleanup blk_mq_rq_to_pdu()
X-Git-Tag: v4.1-rc1~106^2~2
X-Git-Url: http://git.cascardo.eti.br/?a=commitdiff_plain;h=2963e3f7e8e3465895897a175560210120b932ac;p=cascardo%2Flinux.git

blk-mq: cleanup blk_mq_rq_to_pdu()

Casting to void and adding the size of the request is "shit code" and
only a "crazy monkey on crack" would write that. So lets clean it up.

Signed-off-by: Jens Axboe <axboe@fb.com>
---

diff --git a/include/linux/blk-mq.h b/include/linux/blk-mq.h
index ebfe707cf722..8210e8797c12 100644
--- a/include/linux/blk-mq.h
+++ b/include/linux/blk-mq.h
@@ -230,7 +230,7 @@ void blk_mq_freeze_queue_start(struct request_queue *q);
 
 /*
  * Driver command data is immediately after the request. So subtract request
- * size to get back to the original request.
+ * size to get back to the original request, add request size to get the PDU.
  */
 static inline struct request *blk_mq_rq_from_pdu(void *pdu)
 {
@@ -238,7 +238,7 @@ static inline struct request *blk_mq_rq_from_pdu(void *pdu)
 }
 static inline void *blk_mq_rq_to_pdu(struct request *rq)
 {
-	return (void *) rq + sizeof(*rq);
+	return rq + 1;
 }
 
 #define queue_for_each_hw_ctx(q, hctx, i)				\