On Tue, Jul 29, 2025 at 12:41:00PM +0200, Christoph Hellwig wrote:
On Tue, Jul 29, 2025 at 11:53:36AM +0300, Leon Romanovsky wrote:
Because the struct page is the only thing that:
a) dma-mapping works on b) is the only place we can discover the routing information, but also more importantly ensure that the underlying page is still present and the device is not hot unplugged, or in a very theoretical worst case replaced by something else.
It is correct in general case, but here we are talking about MMIO memory, which is "connected" to device X and routing information is stable.
MMIO is literally the only thing we support to P2P to/from as that is how PCIe P2P is defined. And not, it's not stable - devices can be unplugged, and BARs can be reenumerated.
I have a feeling that we are drifting from the current patchset to more general discussion.
The whole idea of new DMA API is to provide flexibility to the callers (subsystems) who are perfectly aware of their data and limitations to implement direct addressing natively.
In this series, device is controlled by VFIO and DMABUF. It is not possible to unplug it without VFIO notices it. In such case, p2pdma_provider and related routing information (DMABUF) will be reevaluated.
So for VFIO + DMABUF, the pointer is very stable.
For other cases (general case), the flow is not changed. Users will continue to call to old and well-known pci_p2pdma_state() to calculate p2p type.
Thanks