On Fri, Nov 08, 2024 at 04:29:56PM +0100, Christoph Hellwig wrote: > On Fri, Nov 08, 2024 at 11:25:37AM -0400, Jason Gunthorpe wrote: > > I'm asking how it will work if you change the struct page argument to > > physical, because today dma_direct_map_page() has: > > > > if (is_pci_p2pdma_page(page)) > > return DMA_MAPPING_ERROR; > > > > Which is exactly the sorts of things I'm looking at when when I say to > > get rid of struct page. > > It will have to look up the page from the physical address obviously. > But at least only in the error path. I'm thinking we can largely avoid searching on physical, or at least we can optimize this so there is only one search on physical at the start of the DMA mapping. (since we are now saying all pages are the same type) > > What I'm thinking about is replacing code like the above with something like: > > > > if (p2p_provider) > > return DMA_MAPPING_ERROR; > > > > And the caller is the one that would have done is_pci_p2pdma_page() > > and either passes p2p_provider=NULL or page->pgmap->p2p_provider. > > And where do you get that one from? Which one? The caller must know the p2p properties of what it is doing because it is driving all the P2P logic around what APIs to call. Either because it is already working with struct page and gets it out of the pgmap. Or it is working with non-struct page memory and has a (MMIO address, p2p_provider) tuple that it got from the original driver that gave it the MMIO address. Or it really does have a naked phys_addr_t and it did the search on physical, but only once. Jason