[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH 7/9] vdpa: Convert to physical address DMA mapping



From: Leon Romanovsky <leonro@xxxxxxxxxx>

Use physical address directly in DMA mapping flow.

Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxx>
---
 drivers/vdpa/vdpa_user/iova_domain.c | 11 +++++------
 drivers/vdpa/vdpa_user/iova_domain.h |  8 ++++----
 drivers/vdpa/vdpa_user/vduse_dev.c   | 18 ++++++++++--------
 3 files changed, 19 insertions(+), 18 deletions(-)

diff --git a/drivers/vdpa/vdpa_user/iova_domain.c 
b/drivers/vdpa/vdpa_user/iova_domain.c
index 58116f89d8dae..c0ecf01003cd3 100644
--- a/drivers/vdpa/vdpa_user/iova_domain.c
+++ b/drivers/vdpa/vdpa_user/iova_domain.c
@@ -396,17 +396,16 @@ void vduse_domain_sync_single_for_cpu(struct 
vduse_iova_domain *domain,
        read_unlock(&domain->bounce_lock);
 }
 
-dma_addr_t vduse_domain_map_page(struct vduse_iova_domain *domain,
-                                struct page *page, unsigned long offset,
-                                size_t size, enum dma_data_direction dir,
+dma_addr_t vduse_domain_map_phys(struct vduse_iova_domain *domain,
+                                phys_addr_t pa, size_t size,
+                                enum dma_data_direction dir,
                                 unsigned long attrs)
 {
        struct iova_domain *iovad = &domain->stream_iovad;
        unsigned long limit = domain->bounce_size - 1;
-       phys_addr_t pa = page_to_phys(page) + offset;
        dma_addr_t iova = vduse_domain_alloc_iova(iovad, size, limit);
 
-       if (!iova)
+       if (!iova || (attrs & DMA_ATTR_MMIO))
                return DMA_MAPPING_ERROR;
 
        if (vduse_domain_init_bounce_map(domain))
@@ -430,7 +429,7 @@ dma_addr_t vduse_domain_map_page(struct vduse_iova_domain 
*domain,
        return DMA_MAPPING_ERROR;
 }
 
-void vduse_domain_unmap_page(struct vduse_iova_domain *domain,
+void vduse_domain_unmap_phys(struct vduse_iova_domain *domain,
                             dma_addr_t dma_addr, size_t size,
                             enum dma_data_direction dir, unsigned long attrs)
 {
diff --git a/drivers/vdpa/vdpa_user/iova_domain.h 
b/drivers/vdpa/vdpa_user/iova_domain.h
index 7f3f0928ec781..7c4546fd856ab 100644
--- a/drivers/vdpa/vdpa_user/iova_domain.h
+++ b/drivers/vdpa/vdpa_user/iova_domain.h
@@ -53,12 +53,12 @@ void vduse_domain_sync_single_for_cpu(struct 
vduse_iova_domain *domain,
                                      dma_addr_t dma_addr, size_t size,
                                      enum dma_data_direction dir);
 
-dma_addr_t vduse_domain_map_page(struct vduse_iova_domain *domain,
-                                struct page *page, unsigned long offset,
-                                size_t size, enum dma_data_direction dir,
+dma_addr_t vduse_domain_map_phys(struct vduse_iova_domain *domain,
+                                phys_addr_t phys, size_t size,
+                                enum dma_data_direction dir,
                                 unsigned long attrs);
 
-void vduse_domain_unmap_page(struct vduse_iova_domain *domain,
+void vduse_domain_unmap_phys(struct vduse_iova_domain *domain,
                             dma_addr_t dma_addr, size_t size,
                             enum dma_data_direction dir, unsigned long attrs);
 
diff --git a/drivers/vdpa/vdpa_user/vduse_dev.c 
b/drivers/vdpa/vdpa_user/vduse_dev.c
index 04620bb77203d..75aa3c9f83fb5 100644
--- a/drivers/vdpa/vdpa_user/vduse_dev.c
+++ b/drivers/vdpa/vdpa_user/vduse_dev.c
@@ -834,25 +834,27 @@ static void vduse_dev_sync_single_for_cpu(struct device 
*dev,
        vduse_domain_sync_single_for_cpu(domain, dma_addr, size, dir);
 }
 
-static dma_addr_t vduse_dev_map_page(struct device *dev, struct page *page,
-                                    unsigned long offset, size_t size,
-                                    enum dma_data_direction dir,
+static dma_addr_t vduse_dev_map_phys(struct device *dev, phys_addr_t phys,
+                                    size_t size, enum dma_data_direction dir,
                                     unsigned long attrs)
 {
        struct vduse_dev *vdev = dev_to_vduse(dev);
        struct vduse_iova_domain *domain = vdev->domain;
 
-       return vduse_domain_map_page(domain, page, offset, size, dir, attrs);
+       if (attrs & DMA_ATTR_MMIO)
+               return DMA_MAPPING_ERROR;
+
+       return vduse_domain_map_phys(domain, phys, size, dir, attrs);
 }
 
-static void vduse_dev_unmap_page(struct device *dev, dma_addr_t dma_addr,
+static void vduse_dev_unmap_phys(struct device *dev, dma_addr_t dma_addr,
                                size_t size, enum dma_data_direction dir,
                                unsigned long attrs)
 {
        struct vduse_dev *vdev = dev_to_vduse(dev);
        struct vduse_iova_domain *domain = vdev->domain;
 
-       return vduse_domain_unmap_page(domain, dma_addr, size, dir, attrs);
+       return vduse_domain_unmap_phys(domain, dma_addr, size, dir, attrs);
 }
 
 static void *vduse_dev_alloc_coherent(struct device *dev, size_t size,
@@ -896,8 +898,8 @@ static size_t vduse_dev_max_mapping_size(struct device *dev)
 static const struct dma_map_ops vduse_dev_dma_ops = {
        .sync_single_for_device = vduse_dev_sync_single_for_device,
        .sync_single_for_cpu = vduse_dev_sync_single_for_cpu,
-       .map_page = vduse_dev_map_page,
-       .unmap_page = vduse_dev_unmap_page,
+       .map_phys = vduse_dev_map_phys,
+       .unmap_phys = vduse_dev_unmap_phys,
        .alloc = vduse_dev_alloc_coherent,
        .free = vduse_dev_free_coherent,
        .max_mapping_size = vduse_dev_max_mapping_size,
-- 
2.51.0




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.