* [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