[PATCH v9 01/24] PCI/P2PDMA: Refactor the p2pdma mapping helpers

Luis Chamberlain mcgrof at kernel.org
Fri Apr 25 17:21:59 PDT 2025


On Wed, Apr 23, 2025 at 11:12:52AM +0300, Leon Romanovsky wrote:
> From: Christoph Hellwig <hch at lst.de>
> 
> The current scheme with a single helper to determine the P2P status
> and map a scatterlist segment force users to always use the map_sg
> helper to DMA map, which we're trying to get away from because they
> are very cache inefficient.
> 
> Refactor the code so that there is a single helper that checks the P2P
> state for a page, including the result that it is not a P2P page to
> simplify the callers, and a second one to perform the address translation
> for a bus mapped P2P transfer that does not depend on the scatterlist
> structure.
> 
> Signed-off-by: Christoph Hellwig <hch at lst.de>
> Reviewed-by: Logan Gunthorpe <logang at deltatee.com>
> Acked-by: Bjorn Helgaas <bhelgaas at google.com>
> Tested-by: Jens Axboe <axboe at kernel.dk>
> Signed-off-by: Leon Romanovsky <leonro at nvidia.com>

Might make it easier for patch review to split off adding
__pci_p2pdma_update_state() in a seprate patch first. Other than that,
looks good.

Reviewed-by: Luis Chamberlain <mcgrof at kenrel.org>

  Luis



More information about the Linux-nvme mailing list