drm/radeon: fix R600_PTE_GART handling
authorChristian König <christian.koenig@amd.com>
Tue, 22 Jul 2014 15:42:20 +0000 (17:42 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 5 Aug 2014 12:53:37 +0000 (08:53 -0400)
That didn't worked correctly any more and opened up a security problem.

Signed-off-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/radeon/cik_sdma.c
drivers/gpu/drm/radeon/radeon.h
drivers/gpu/drm/radeon/si_dma.c

index 4b2ab769cd9916a78c1bd653926b190de2e9ffd8..d629cea865f2a35522894f4465ba65b4dfd7ef12 100644 (file)
@@ -771,8 +771,7 @@ void cik_sdma_vm_set_page(struct radeon_device *rdev,
 
        trace_radeon_vm_set_page(pe, addr, count, incr, flags);
 
-       /* XXX: How to distinguish between GART and other system memory pages? */
-       if (flags & R600_PTE_SYSTEM) {
+       if ((flags & R600_PTE_GART_MASK) == R600_PTE_GART_MASK) {
                uint64_t src = rdev->gart.table_addr + (addr >> 12) * 8;
                while (count) {
                        unsigned bytes = count * 8;
index 89b63b9cff5eda7c448487321515ffebe435f94b..fd878c7bb8f008f4bf71bd0714aff8228afc03af 100644 (file)
@@ -864,9 +864,9 @@ struct radeon_mec {
 #define R600_PTE_FRAG_64KB     (4 << 7)
 #define R600_PTE_FRAG_256KB    (6 << 7)
 
-/* flags used for GART page table entries on R600+ */
-#define R600_PTE_GART  ( R600_PTE_VALID | R600_PTE_SYSTEM | R600_PTE_SNOOPED \
-                       | R600_PTE_READABLE | R600_PTE_WRITEABLE)
+/* flags needed to be set so we can copy directly from the GART table */
+#define R600_PTE_GART_MASK     ( R600_PTE_READABLE | R600_PTE_WRITEABLE | \
+                                 R600_PTE_SYSTEM | R600_PTE_VALID )
 
 struct radeon_vm_pt {
        struct radeon_bo                *bo;
index c9da34181b32987dafbdc5c001c94b33324ee412..a26e842cf39136c6c5ba01ded73538aa5bffc01d 100644 (file)
@@ -79,8 +79,7 @@ void si_dma_vm_set_page(struct radeon_device *rdev,
 
        trace_radeon_vm_set_page(pe, addr, count, incr, flags);
 
-       /* XXX: How to distinguish between GART and other system memory pages? */
-       if (flags & R600_PTE_SYSTEM) {
+       if ((flags & R600_PTE_GART_MASK) == R600_PTE_GART_MASK) {
                uint64_t src = rdev->gart.table_addr + (addr >> 12) * 8;
                while (count) {
                        unsigned bytes = count * 8;