drm/amdgpu: fix and cleanup gmc_v9_0_flush_gpu_tlb
authorChristian König <christian.koenig@amd.com>
Fri, 1 Sep 2023 05:45:52 +0000 (07:45 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 26 Sep 2023 20:54:51 +0000 (16:54 -0400)
The KIQ code path was ignoring the second flush. Also avoid long lines and
re-calculating the register offsets over and over again.

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/gmc_v9_0.c

index 2936a0f..0f82ab4 100644 (file)
@@ -816,13 +816,17 @@ static void gmc_v9_0_flush_gpu_tlb(struct amdgpu_device *adev, uint32_t vmid,
                                        uint32_t vmhub, uint32_t flush_type)
 {
        bool use_semaphore = gmc_v9_0_use_invalidate_semaphore(adev, vmhub);
+       u32 j, inv_req, inv_req2, tmp, sem, req, ack;
        const unsigned int eng = 17;
-       u32 j, inv_req, inv_req2, tmp;
        struct amdgpu_vmhub *hub;
 
        BUG_ON(vmhub >= AMDGPU_MAX_VMHUBS);
 
        hub = &adev->vmhub[vmhub];
+       sem = hub->vm_inv_eng0_sem + hub->eng_distance * eng;
+       req = hub->vm_inv_eng0_req + hub->eng_distance * eng;
+       ack = hub->vm_inv_eng0_ack + hub->eng_distance * eng;
+
        if (adev->gmc.xgmi.num_physical_nodes &&
            amdgpu_ip_version(adev, GC_HWIP, 0) == IP_VERSION(9, 4, 0)) {
                /* Vega20+XGMI caches PTEs in TC and TLB. Add a
@@ -854,6 +858,10 @@ static void gmc_v9_0_flush_gpu_tlb(struct amdgpu_device *adev, uint32_t vmid,
 
                amdgpu_virt_kiq_reg_write_reg_wait(adev, req, ack, inv_req,
                                                   1 << vmid);
+               if (inv_req2)
+                       amdgpu_virt_kiq_reg_write_reg_wait(adev, req, ack,
+                                                          inv_req2, 1 << vmid);
+
                up_read(&adev->reset_domain->sem);
                return;
        }
@@ -872,9 +880,9 @@ static void gmc_v9_0_flush_gpu_tlb(struct amdgpu_device *adev, uint32_t vmid,
                for (j = 0; j < adev->usec_timeout; j++) {
                        /* a read return value of 1 means semaphore acquire */
                        if (vmhub >= AMDGPU_MMHUB0(0))
-                               tmp = RREG32_SOC15_IP_NO_KIQ(MMHUB, hub->vm_inv_eng0_sem + hub->eng_distance * eng);
+                               tmp = RREG32_SOC15_IP_NO_KIQ(MMHUB, sem);
                        else
-                               tmp = RREG32_SOC15_IP_NO_KIQ(GC, hub->vm_inv_eng0_sem + hub->eng_distance * eng);
+                               tmp = RREG32_SOC15_IP_NO_KIQ(GC, sem);
                        if (tmp & 0x1)
                                break;
                        udelay(1);
@@ -886,9 +894,9 @@ static void gmc_v9_0_flush_gpu_tlb(struct amdgpu_device *adev, uint32_t vmid,
 
        do {
                if (vmhub >= AMDGPU_MMHUB0(0))
-                       WREG32_SOC15_IP_NO_KIQ(MMHUB, hub->vm_inv_eng0_req + hub->eng_distance * eng, inv_req);
+                       WREG32_SOC15_IP_NO_KIQ(MMHUB, req, inv_req);
                else
-                       WREG32_SOC15_IP_NO_KIQ(GC, hub->vm_inv_eng0_req + hub->eng_distance * eng, inv_req);
+                       WREG32_SOC15_IP_NO_KIQ(GC, req, inv_req);
 
                /*
                 * Issue a dummy read to wait for the ACK register to
@@ -897,14 +905,13 @@ static void gmc_v9_0_flush_gpu_tlb(struct amdgpu_device *adev, uint32_t vmid,
                 */
                if ((vmhub == AMDGPU_GFXHUB(0)) &&
                    (amdgpu_ip_version(adev, GC_HWIP, 0) < IP_VERSION(9, 4, 2)))
-                       RREG32_NO_KIQ(hub->vm_inv_eng0_req +
-                                     hub->eng_distance * eng);
+                       RREG32_SOC15_IP_NO_KIQ(GC, req);
 
                for (j = 0; j < adev->usec_timeout; j++) {
                        if (vmhub >= AMDGPU_MMHUB0(0))
-                               tmp = RREG32_SOC15_IP_NO_KIQ(MMHUB, hub->vm_inv_eng0_ack + hub->eng_distance * eng);
+                               tmp = RREG32_SOC15_IP_NO_KIQ(MMHUB, ack);
                        else
-                               tmp = RREG32_SOC15_IP_NO_KIQ(GC, hub->vm_inv_eng0_ack + hub->eng_distance * eng);
+                               tmp = RREG32_SOC15_IP_NO_KIQ(GC, ack);
                        if (tmp & (1 << vmid))
                                break;
                        udelay(1);
@@ -921,9 +928,9 @@ static void gmc_v9_0_flush_gpu_tlb(struct amdgpu_device *adev, uint32_t vmid,
                 * write with 0 means semaphore release
                 */
                if (vmhub >= AMDGPU_MMHUB0(0))
-                       WREG32_SOC15_IP_NO_KIQ(MMHUB, hub->vm_inv_eng0_sem + hub->eng_distance * eng, 0);
+                       WREG32_SOC15_IP_NO_KIQ(MMHUB, sem, 0);
                else
-                       WREG32_SOC15_IP_NO_KIQ(GC, hub->vm_inv_eng0_sem + hub->eng_distance * eng, 0);
+                       WREG32_SOC15_IP_NO_KIQ(GC, sem, 0);
        }
 
        spin_unlock(&adev->gmc.invalidate_lock);