From: Christian König <christian.koe...@amd.com>

Instead of mapping them bit by bit map/unmap the whole
compound pages as in one call.

Signed-off-by: Christian König <christian.koe...@amd.com>
---
 drivers/gpu/drm/ttm/ttm_page_alloc.c | 31 +++++++++++++++++++++++--------
 1 file changed, 23 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/ttm/ttm_page_alloc.c 
b/drivers/gpu/drm/ttm/ttm_page_alloc.c
index 482dd9a..3309926 100644
--- a/drivers/gpu/drm/ttm/ttm_page_alloc.c
+++ b/drivers/gpu/drm/ttm/ttm_page_alloc.c
@@ -922,16 +922,18 @@ EXPORT_SYMBOL(ttm_pool_unpopulate);
 #if defined(CONFIG_SWIOTLB) || defined(CONFIG_INTEL_IOMMU)
 int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt *tt)
 {
-       unsigned i;
+       unsigned i, j;
        int r;
 
        r = ttm_pool_populate(&tt->ttm);
        if (r)
                return r;
 
-       for (i = 0; i < tt->ttm.num_pages; i++) {
+       for (i = 0; i < tt->ttm.num_pages; ++i) {
+               size_t num_pages = 1 << compound_order(tt->ttm.pages[i]);
+
                tt->dma_address[i] = dma_map_page(dev, tt->ttm.pages[i],
-                                                 0, PAGE_SIZE,
+                                                 0, num_pages * PAGE_SIZE,
                                                  DMA_BIDIRECTIONAL);
                if (dma_mapping_error(dev, tt->dma_address[i])) {
                        while (i--) {
@@ -942,6 +944,11 @@ int ttm_populate_and_map_pages(struct device *dev, struct 
ttm_dma_tt *tt)
                        ttm_pool_unpopulate(&tt->ttm);
                        return -EFAULT;
                }
+
+               for (j = 1; j < num_pages; ++j) {
+                       tt->dma_address[i + 1] = tt->dma_address[i] + PAGE_SIZE;
+                       ++i;
+               }
        }
        return 0;
 }
@@ -950,12 +957,20 @@ EXPORT_SYMBOL(ttm_populate_and_map_pages);
 void ttm_unmap_and_unpopulate_pages(struct device *dev, struct ttm_dma_tt *tt)
 {
        unsigned i;
-       
-       for (i = 0; i < tt->ttm.num_pages; i++) {
-               if (tt->dma_address[i]) {
-                       dma_unmap_page(dev, tt->dma_address[i],
-                                      PAGE_SIZE, DMA_BIDIRECTIONAL);
+
+       for (i = 0; i < tt->ttm.num_pages;) {
+               size_t num_pages;
+
+               if (!tt->dma_address[i] || !tt->ttm.pages[i]) {
+                       ++i;
+                       continue;
                }
+
+               num_pages = 1 << compound_order(tt->ttm.pages[i]);
+               dma_unmap_page(dev, tt->dma_address[i], num_pages * PAGE_SIZE,
+                              DMA_BIDIRECTIONAL);
+
+               i += num_pages;
        }
        ttm_pool_unpopulate(&tt->ttm);
 }
-- 
2.7.4

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to