Ver Fonte

NVMe: embedded iod mask cleanup

Remove unused mask in nvme_alloc_iod

Signed-off-by: Chong Yuan <chong.yuan@memblaze.com>
Reviewed-by: Wenbo Wang  <wenbo.wang@memblaze.com>
Acked-by: Keith Busch <keith.busch@intel.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
Chong Yuan há 10 anos atrás
pai
commit
fda631ffe5
1 ficheiros alterados com 3 adições e 4 exclusões
  1. 3 4
      drivers/block/nvme-core.c

+ 3 - 4
drivers/block/nvme-core.c

@@ -152,6 +152,7 @@ struct nvme_cmd_info {
  */
  */
 #define NVME_INT_PAGES		2
 #define NVME_INT_PAGES		2
 #define NVME_INT_BYTES(dev)	(NVME_INT_PAGES * (dev)->page_size)
 #define NVME_INT_BYTES(dev)	(NVME_INT_PAGES * (dev)->page_size)
+#define NVME_INT_MASK		0x01
 
 
 /*
 /*
  * Will slightly overestimate the number of pages needed.  This is OK
  * Will slightly overestimate the number of pages needed.  This is OK
@@ -257,7 +258,7 @@ static void *iod_get_private(struct nvme_iod *iod)
  */
  */
 static bool iod_should_kfree(struct nvme_iod *iod)
 static bool iod_should_kfree(struct nvme_iod *iod)
 {
 {
-	return (iod->private & 0x01) == 0;
+	return (iod->private & NVME_INT_MASK) == 0;
 }
 }
 
 
 /* Special values must be less than 0x1000 */
 /* Special values must be less than 0x1000 */
@@ -432,7 +433,6 @@ static struct nvme_iod *nvme_alloc_iod(struct request *rq, struct nvme_dev *dev,
 {
 {
 	unsigned size = !(rq->cmd_flags & REQ_DISCARD) ? blk_rq_bytes(rq) :
 	unsigned size = !(rq->cmd_flags & REQ_DISCARD) ? blk_rq_bytes(rq) :
                                                 sizeof(struct nvme_dsm_range);
                                                 sizeof(struct nvme_dsm_range);
-	unsigned long mask = 0;
 	struct nvme_iod *iod;
 	struct nvme_iod *iod;
 
 
 	if (rq->nr_phys_segments <= NVME_INT_PAGES &&
 	if (rq->nr_phys_segments <= NVME_INT_PAGES &&
@@ -440,9 +440,8 @@ static struct nvme_iod *nvme_alloc_iod(struct request *rq, struct nvme_dev *dev,
 		struct nvme_cmd_info *cmd = blk_mq_rq_to_pdu(rq);
 		struct nvme_cmd_info *cmd = blk_mq_rq_to_pdu(rq);
 
 
 		iod = cmd->iod;
 		iod = cmd->iod;
-		mask = 0x01;
 		iod_init(iod, size, rq->nr_phys_segments,
 		iod_init(iod, size, rq->nr_phys_segments,
-				(unsigned long) rq | 0x01);
+				(unsigned long) rq | NVME_INT_MASK);
 		return iod;
 		return iod;
 	}
 	}