diff options
| author | Mina Almasry <[email protected]> | 2024-12-11 21:20:31 +0000 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2024-12-13 02:49:08 +0000 |
| commit | 7dba339faae991a23c54f7b93a58798c58f8c16f (patch) | |
| tree | e6cdfdf0c2deb471a5e640b5c652005b456aab79 /net/core/devmem.c | |
| parent | page_pool: Set `dma_sync` to false for devmem memory provider (diff) | |
| download | kernel-7dba339faae991a23c54f7b93a58798c58f8c16f.tar.gz kernel-7dba339faae991a23c54f7b93a58798c58f8c16f.zip | |
page_pool: disable sync for cpu for dmabuf memory provider
dmabuf dma-addresses should not be dma_sync'd for CPU/device. Typically
its the driver responsibility to dma_sync for CPU, but the driver should
not dma_sync for CPU if the netmem is actually coming from a dmabuf
memory provider.
The page_pool already exposes a helper for dma_sync_for_cpu:
page_pool_dma_sync_for_cpu. Upgrade this existing helper to handle
netmem, and have it skip dma_sync if the memory is from a dmabuf memory
provider. Drivers should migrate to using this helper when adding
support for netmem.
Also minimize the impact on the dma syncing performance for pages. Special
case the dma-sync path for pages to not go through the overhead checks
for dma-syncing and conversion to netmem.
Cc: Alexander Lobakin <[email protected]>
Cc: Jason Gunthorpe <[email protected]>
Signed-off-by: Mina Almasry <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'net/core/devmem.c')
| -rw-r--r-- | net/core/devmem.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/core/devmem.c b/net/core/devmem.c index 3ebdeed2bf18..0b6ed7525b22 100644 --- a/net/core/devmem.c +++ b/net/core/devmem.c @@ -335,6 +335,7 @@ int mp_dmabuf_devmem_init(struct page_pool *pool) * dma_sync_for_cpu/device. Force disable dma_sync. */ pool->dma_sync = false; + pool->dma_sync_for_cpu = false; if (pool->p.order != 0) return -E2BIG; |