public inbox for linux-block@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] block, nvme: remove unused dma_iova_state function parameter
       [not found] <CGME20260112144230epcas5p34403fd2f51e32c77ad4de7bac8c98a18@epcas5p3.samsung.com>
@ 2026-01-12 14:38 ` Nitesh Shetty
  2026-01-13  7:15   ` Christoph Hellwig
  2026-01-13 14:27   ` Jens Axboe
  0 siblings, 2 replies; 3+ messages in thread
From: Nitesh Shetty @ 2026-01-12 14:38 UTC (permalink / raw)
  To: Jens Axboe, Keith Busch, Christoph Hellwig, Sagi Grimberg
  Cc: nitheshshetty, Nitesh Shetty, linux-block, linux-kernel,
	linux-nvme

DMA IOVA state is not used inside blk_rq_dma_map_iter_next

Signed-off-by: Nitesh Shetty <nj.shetty@samsung.com>
---
 block/blk-mq-dma.c         | 3 +--
 drivers/nvme/host/pci.c    | 5 ++---
 include/linux/blk-mq-dma.h | 2 +-
 3 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/block/blk-mq-dma.c b/block/blk-mq-dma.c
index 752060d7261cb..3c87779cdc19d 100644
--- a/block/blk-mq-dma.c
+++ b/block/blk-mq-dma.c
@@ -233,7 +233,6 @@ EXPORT_SYMBOL_GPL(blk_rq_dma_map_iter_start);
  * blk_rq_dma_map_iter_next - map the next DMA segment for a request
  * @req:	request to map
  * @dma_dev:	device to map to
- * @state:	DMA IOVA state
  * @iter:	block layer DMA iterator
  *
  * Iterate to the next mapping after a previous call to
@@ -248,7 +247,7 @@ EXPORT_SYMBOL_GPL(blk_rq_dma_map_iter_start);
  * returned in @iter.status.
  */
 bool blk_rq_dma_map_iter_next(struct request *req, struct device *dma_dev,
-		struct dma_iova_state *state, struct blk_dma_iter *iter)
+		struct blk_dma_iter *iter)
 {
 	struct phys_vec vec;
 
diff --git a/drivers/nvme/host/pci.c b/drivers/nvme/host/pci.c
index 3b528369f5454..065555576d2f9 100644
--- a/drivers/nvme/host/pci.c
+++ b/drivers/nvme/host/pci.c
@@ -823,7 +823,7 @@ static bool nvme_pci_prp_iter_next(struct request *req, struct device *dma_dev,
 
 	if (iter->len)
 		return true;
-	if (!blk_rq_dma_map_iter_next(req, dma_dev, &iod->dma_state, iter))
+	if (!blk_rq_dma_map_iter_next(req, dma_dev, iter))
 		return false;
 	if (!dma_use_iova(&iod->dma_state) && dma_need_unmap(dma_dev)) {
 		iod->dma_vecs[iod->nr_dma_vecs].addr = iter->addr;
@@ -1010,8 +1010,7 @@ static blk_status_t nvme_pci_setup_data_sgl(struct request *req,
 		}
 		nvme_pci_sgl_set_data(&sg_list[mapped++], iter);
 		iod->total_len += iter->len;
-	} while (blk_rq_dma_map_iter_next(req, nvmeq->dev->dev, &iod->dma_state,
-				iter));
+	} while (blk_rq_dma_map_iter_next(req, nvmeq->dev->dev, iter));
 
 	nvme_pci_sgl_set_seg(&iod->cmd.common.dptr.sgl, sgl_dma, mapped);
 	if (unlikely(iter->status))
diff --git a/include/linux/blk-mq-dma.h b/include/linux/blk-mq-dma.h
index cb88fc791fbd1..214c181ff2c9c 100644
--- a/include/linux/blk-mq-dma.h
+++ b/include/linux/blk-mq-dma.h
@@ -28,7 +28,7 @@ struct blk_dma_iter {
 bool blk_rq_dma_map_iter_start(struct request *req, struct device *dma_dev,
 		struct dma_iova_state *state, struct blk_dma_iter *iter);
 bool blk_rq_dma_map_iter_next(struct request *req, struct device *dma_dev,
-		struct dma_iova_state *state, struct blk_dma_iter *iter);
+		struct blk_dma_iter *iter);
 
 /**
  * blk_rq_dma_map_coalesce - were all segments coalesced?
-- 
2.39.5


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] block, nvme: remove unused dma_iova_state function parameter
  2026-01-12 14:38 ` [PATCH] block, nvme: remove unused dma_iova_state function parameter Nitesh Shetty
@ 2026-01-13  7:15   ` Christoph Hellwig
  2026-01-13 14:27   ` Jens Axboe
  1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2026-01-13  7:15 UTC (permalink / raw)
  To: Nitesh Shetty
  Cc: Jens Axboe, Keith Busch, Christoph Hellwig, Sagi Grimberg,
	nitheshshetty, linux-block, linux-kernel, linux-nvme

Looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] block, nvme: remove unused dma_iova_state function parameter
  2026-01-12 14:38 ` [PATCH] block, nvme: remove unused dma_iova_state function parameter Nitesh Shetty
  2026-01-13  7:15   ` Christoph Hellwig
@ 2026-01-13 14:27   ` Jens Axboe
  1 sibling, 0 replies; 3+ messages in thread
From: Jens Axboe @ 2026-01-13 14:27 UTC (permalink / raw)
  To: Keith Busch, Christoph Hellwig, Sagi Grimberg, Nitesh Shetty
  Cc: nitheshshetty, linux-block, linux-kernel, linux-nvme


On Mon, 12 Jan 2026 20:08:08 +0530, Nitesh Shetty wrote:
> DMA IOVA state is not used inside blk_rq_dma_map_iter_next
> 
> 

Applied, thanks!

[1/1] block, nvme: remove unused dma_iova_state function parameter
      commit: 91e1c1bcf0f2376f40ac859cf17d0a64a605e662

Best regards,
-- 
Jens Axboe




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-01-13 14:27 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <CGME20260112144230epcas5p34403fd2f51e32c77ad4de7bac8c98a18@epcas5p3.samsung.com>
2026-01-12 14:38 ` [PATCH] block, nvme: remove unused dma_iova_state function parameter Nitesh Shetty
2026-01-13  7:15   ` Christoph Hellwig
2026-01-13 14:27   ` Jens Axboe

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox