[Linaro-mm-sig] Re: [RFC PATCH v3 06/12] memory-provider: dmabuf devmem memory provider

On Sun, 5 Nov 2023 18:44:05 -0800 Mina Almasry wrote:

+static int mp_dmabuf_devmem_init(struct page_pool *pool) +{

• struct netdev_dmabuf_binding *binding = pool->mp_priv;
• if (!binding)

• return -EINVAL;
  

• if (pool->p.flags & PP_FLAG_DMA_MAP ||

•    pool->p.flags & PP_FLAG_DMA_SYNC_DEV)
  

• return -EOPNOTSUPP;
  

This looks backwards, we should _force_ the driver to use the dma mapping built into the page pool APIs, to isolate the driver from how the DMA addr actually gets obtained. Right?

Maybe seeing driver patches would illuminate.