drm/ttm: fix the fix for huge compound pages
authorChristian König <christian.koenig@amd.com>
Thu, 12 Oct 2017 17:28:42 +0000 (19:28 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 19 Oct 2017 19:27:10 +0000 (15:27 -0400)
We don't use compound pages at the moment. Take this into account when
freeing them.

Signed-off-by: Christian König <christian.koenig@amd.comd>
Reviewed-and-Tested-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/ttm/ttm_page_alloc.c

index 95022473704b025335234035272d8690439b9d93..4d688c8d7853e4bba377cc9cebffbb23a8103d96 100644 (file)
@@ -733,22 +733,33 @@ static void ttm_put_pages(struct page **pages, unsigned npages, int flags,
                /* No pool for this memory type so free the pages */
                i = 0;
                while (i < npages) {
-                       unsigned order;
+#ifdef CONFIG_TRANSPARENT_HUGEPAGE
+                       struct page *p = pages[i];
+#endif
+                       unsigned order = 0, j;
 
                        if (!pages[i]) {
                                ++i;
                                continue;
                        }
 
+#ifdef CONFIG_TRANSPARENT_HUGEPAGE
+                       for (j = 0; j < HPAGE_PMD_NR; ++j)
+                               if (p++ != pages[i + j])
+                                   break;
+
+                       if (j == HPAGE_PMD_NR)
+                               order = HPAGE_PMD_ORDER;
+#endif
+
                        if (page_count(pages[i]) != 1)
                                pr_err("Erroneous page count. Leaking pages.\n");
-                       order = compound_order(pages[i]);
                        __free_pages(pages[i], order);
 
-                       order = 1 << order;
-                       while (order) {
+                       j = 1 << order;
+                       while (j) {
                                pages[i++] = NULL;
-                               --order;
+                               --j;
                        }
                }
                return;