drm/amdgpu: use more than 64KB fragment size if possible
authorChristian König <christian.koenig@amd.com>
Mon, 8 Aug 2016 12:40:18 +0000 (14:40 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 10 Aug 2016 18:06:08 +0000 (14:06 -0400)
We align to 64KB, but when userspace aligns even more we can easily use more.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c

index d2796bb4004bd18419d2734eea61053c7951fb18..7f385e78a265f12f6076fba64e08bfbf16d83c7e 100644 (file)
@@ -818,13 +818,13 @@ static void amdgpu_vm_frag_ptes(struct amdgpu_pte_update_params   *params,
         * allocation size to the fragment size.
         */
 
-       /* SI and newer are optimized for 64KB */
-       uint64_t frag_flags = AMDGPU_PTE_FRAG(AMDGPU_LOG2_PAGES_PER_FRAG);
-       uint64_t frag_align = 1 << AMDGPU_LOG2_PAGES_PER_FRAG;
+       const uint64_t frag_align = 1 << AMDGPU_LOG2_PAGES_PER_FRAG;
 
        uint64_t frag_start = ALIGN(start, frag_align);
        uint64_t frag_end = end & ~(frag_align - 1);
 
+       uint32_t frag;
+
        /* system pages are non continuously */
        if (params->src || params->pages_addr || !(flags & AMDGPU_PTE_VALID) ||
            (frag_start >= frag_end)) {
@@ -833,6 +833,10 @@ static void amdgpu_vm_frag_ptes(struct amdgpu_pte_update_params    *params,
                return;
        }
 
+       /* use more than 64KB fragment size if possible */
+       frag = lower_32_bits(frag_start | frag_end);
+       frag = likely(frag) ? __ffs(frag) : 31;
+
        /* handle the 4K area at the beginning */
        if (start != frag_start) {
                amdgpu_vm_update_ptes(params, vm, start, frag_start,
@@ -842,7 +846,7 @@ static void amdgpu_vm_frag_ptes(struct amdgpu_pte_update_params     *params,
 
        /* handle the area in the middle */
        amdgpu_vm_update_ptes(params, vm, frag_start, frag_end, dst,
-                             flags | frag_flags);
+                             flags | AMDGPU_PTE_FRAG(frag));
 
        /* handle the 4K area at the end */
        if (frag_end != end) {