Re: [PATCH] drm/amdgpu: cleanup gmc_v9_0_process_interrupt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[AMD Official Use Only - Internal Distribution Only]


Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> on behalf of Christian König <ckoenig.leichtzumerken@xxxxxxxxx>
Sent: Wednesday, October 28, 2020 9:49 AM
To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>
Subject: [PATCH] drm/amdgpu: cleanup gmc_v9_0_process_interrupt
 
First of all don't snprintf into a char buffer allocated on the stack with
a constant hubname.

Then cleanup to exit the function early in case of a ratelimit or SRIOV.

Signed-off-by: Christian König <christian.koenig@xxxxxxx>
---
 drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 183 +++++++++++++-------------
 1 file changed, 91 insertions(+), 92 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
index a9929d1b6b3d..0c3421d587e8 100644
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
@@ -510,15 +510,16 @@ static int gmc_v9_0_vm_fault_interrupt_state(struct amdgpu_device *adev,
 }
 
 static int gmc_v9_0_process_interrupt(struct amdgpu_device *adev,
-                               struct amdgpu_irq_src *source,
-                               struct amdgpu_iv_entry *entry)
+                                     struct amdgpu_irq_src *source,
+                                     struct amdgpu_iv_entry *entry)
 {
-       struct amdgpu_vmhub *hub;
         bool retry_fault = !!(entry->src_data[1] & 0x80);
         uint32_t status = 0, cid = 0, rw = 0;
-       u64 addr;
-       char hub_name[10];
+       struct amdgpu_task_info task_info;
+       struct amdgpu_vmhub *hub;
         const char *mmhub_cid;
+       const char *hub_name;
+       u64 addr;
 
         addr = (u64)entry->src_data[0] << 12;
         addr |= ((u64)entry->src_data[1] & 0xf) << 44;
@@ -527,105 +528,103 @@ static int gmc_v9_0_process_interrupt(struct amdgpu_device *adev,
                                                     entry->timestamp))
                 return 1; /* This also prevents sending it to KFD */
 
+       /* If it's the first fault for this address, process it normally */
+       if (retry_fault && !in_interrupt() &&
+           amdgpu_vm_handle_fault(adev, entry->pasid, addr))
+               return 1; /* This also prevents sending it to KFD */
+
+       if (!printk_ratelimit())
+               return 0;
+
         if (entry->client_id == SOC15_IH_CLIENTID_VMC) {
-               snprintf(hub_name, sizeof(hub_name), "mmhub0");
+               hub_name = "mmhub0";
                 hub = &adev->vmhub[AMDGPU_MMHUB_0];
         } else if (entry->client_id == SOC15_IH_CLIENTID_VMC1) {
-               snprintf(hub_name, sizeof(hub_name), "mmhub1");
+               hub_name = "mmhub1";
                 hub = &adev->vmhub[AMDGPU_MMHUB_1];
         } else {
-               snprintf(hub_name, sizeof(hub_name), "gfxhub0");
+               hub_name = "gfxhub0";
                 hub = &adev->vmhub[AMDGPU_GFXHUB_0];
         }
 
-       /* If it's the first fault for this address, process it normally */
-       if (retry_fault && !in_interrupt() &&
-           amdgpu_vm_handle_fault(adev, entry->pasid, addr))
-               return 1; /* This also prevents sending it to KFD */
+       memset(&task_info, 0, sizeof(struct amdgpu_task_info));
+       amdgpu_vm_get_task_info(adev, entry->pasid, &task_info);
 
-       if (!amdgpu_sriov_vf(adev)) {
-               /*
-                * Issue a dummy read to wait for the status register to
-                * be updated to avoid reading an incorrect value due to
-                * the new fast GRBM interface.
-                */
-               if (entry->vmid_src == AMDGPU_GFXHUB_0)
-                       RREG32(hub->vm_l2_pro_fault_status);
-
-               status = RREG32(hub->vm_l2_pro_fault_status);
-               cid = REG_GET_FIELD(status,
-                                   VM_L2_PROTECTION_FAULT_STATUS, CID);
-               rw = REG_GET_FIELD(status,
-                                  VM_L2_PROTECTION_FAULT_STATUS, RW);
-               WREG32_P(hub->vm_l2_pro_fault_cntl, 1, ~1);
-       }
+       dev_err(adev->dev,
+               "[%s] %s page fault (src_id:%u ring:%u vmid:%u "
+               "pasid:%u, for process %s pid %d thread %s pid %d)\n",
+               hub_name, retry_fault ? "retry" : "no-retry",
+               entry->src_id, entry->ring_id, entry->vmid,
+               entry->pasid, task_info.process_name, task_info.tgid,
+               task_info.task_name, task_info.pid);
+       dev_err(adev->dev, "  in page starting at address 0x%016llx from client %d\n",
+               addr, entry->client_id);
 
-       if (printk_ratelimit()) {
-               struct amdgpu_task_info task_info;
-
-               memset(&task_info, 0, sizeof(struct amdgpu_task_info));
-               amdgpu_vm_get_task_info(adev, entry->pasid, &task_info);
-
-               dev_err(adev->dev,
-                       "[%s] %s page fault (src_id:%u ring:%u vmid:%u "
-                       "pasid:%u, for process %s pid %d thread %s pid %d)\n",
-                       hub_name, retry_fault ? "retry" : "no-retry",
-                       entry->src_id, entry->ring_id, entry->vmid,
-                       entry->pasid, task_info.process_name, task_info.tgid,
-                       task_info.task_name, task_info.pid);
-               dev_err(adev->dev, "  in page starting at address 0x%016llx from client %d\n",
-                       addr, entry->client_id);
-               if (!amdgpu_sriov_vf(adev)) {
-                       dev_err(adev->dev,
-                               "VM_L2_PROTECTION_FAULT_STATUS:0x%08X\n",
-                               status);
-                       if (hub == &adev->vmhub[AMDGPU_GFXHUB_0]) {
-                               dev_err(adev->dev, "\t Faulty UTCL2 client ID: %s (0x%x)\n",
-                                       cid >= ARRAY_SIZE(gfxhub_client_ids) ? "unknown" : gfxhub_client_ids[cid],
-                                       cid);
-                       } else {
-                               switch (adev->asic_type) {
-                               case CHIP_VEGA10:
-                                       mmhub_cid = mmhub_client_ids_vega10[cid][rw];
-                                       break;
-                               case CHIP_VEGA12:
-                                       mmhub_cid = mmhub_client_ids_vega12[cid][rw];
-                                       break;
-                               case CHIP_VEGA20:
-                                       mmhub_cid = mmhub_client_ids_vega20[cid][rw];
-                                       break;
-                               case CHIP_ARCTURUS:
-                                       mmhub_cid = mmhub_client_ids_arcturus[cid][rw];
-                                       break;
-                               case CHIP_RAVEN:
-                                       mmhub_cid = mmhub_client_ids_raven[cid][rw];
-                                       break;
-                               case CHIP_RENOIR:
-                                       mmhub_cid = mmhub_client_ids_renoir[cid][rw];
-                                       break;
-                               default:
-                                       mmhub_cid = NULL;
-                                       break;
-                               }
-                               dev_err(adev->dev, "\t Faulty UTCL2 client ID: %s (0x%x)\n",
-                                       mmhub_cid ? mmhub_cid : "unknown", cid);
-                       }
-                       dev_err(adev->dev, "\t MORE_FAULTS: 0x%lx\n",
-                               REG_GET_FIELD(status,
-                               VM_L2_PROTECTION_FAULT_STATUS, MORE_FAULTS));
-                       dev_err(adev->dev, "\t WALKER_ERROR: 0x%lx\n",
-                               REG_GET_FIELD(status,
-                               VM_L2_PROTECTION_FAULT_STATUS, WALKER_ERROR));
-                       dev_err(adev->dev, "\t PERMISSION_FAULTS: 0x%lx\n",
-                               REG_GET_FIELD(status,
-                               VM_L2_PROTECTION_FAULT_STATUS, PERMISSION_FAULTS));
-                       dev_err(adev->dev, "\t MAPPING_ERROR: 0x%lx\n",
-                               REG_GET_FIELD(status,
-                               VM_L2_PROTECTION_FAULT_STATUS, MAPPING_ERROR));
-                       dev_err(adev->dev, "\t RW: 0x%x\n", rw);
+       if (amdgpu_sriov_vf(adev))
+               return 0;
+
+       /*
+        * Issue a dummy read to wait for the status register to
+        * be updated to avoid reading an incorrect value due to
+        * the new fast GRBM interface.
+        */
+       if (entry->vmid_src == AMDGPU_GFXHUB_0)
+               RREG32(hub->vm_l2_pro_fault_status);
+
+       status = RREG32(hub->vm_l2_pro_fault_status);
+       cid = REG_GET_FIELD(status, VM_L2_PROTECTION_FAULT_STATUS, CID);
+       rw = REG_GET_FIELD(status, VM_L2_PROTECTION_FAULT_STATUS, RW);
+       WREG32_P(hub->vm_l2_pro_fault_cntl, 1, ~1);
+
+
+       dev_err(adev->dev,
+               "VM_L2_PROTECTION_FAULT_STATUS:0x%08X\n",
+               status);
+       if (hub == &adev->vmhub[AMDGPU_GFXHUB_0]) {
+               dev_err(adev->dev, "\t Faulty UTCL2 client ID: %s (0x%x)\n",
+                       cid >= ARRAY_SIZE(gfxhub_client_ids) ? "unknown" :
+                       gfxhub_client_ids[cid],
+                       cid);
+       } else {
+               switch (adev->asic_type) {
+               case CHIP_VEGA10:
+                       mmhub_cid = mmhub_client_ids_vega10[cid][rw];
+                       break;
+               case CHIP_VEGA12:
+                       mmhub_cid = mmhub_client_ids_vega12[cid][rw];
+                       break;
+               case CHIP_VEGA20:
+                       mmhub_cid = mmhub_client_ids_vega20[cid][rw];
+                       break;
+               case CHIP_ARCTURUS:
+                       mmhub_cid = mmhub_client_ids_arcturus[cid][rw];
+                       break;
+               case CHIP_RAVEN:
+                       mmhub_cid = mmhub_client_ids_raven[cid][rw];
+                       break;
+               case CHIP_RENOIR:
+                       mmhub_cid = mmhub_client_ids_renoir[cid][rw];
+                       break;
+               default:
+                       mmhub_cid = NULL;
+                       break;
                 }
+               dev_err(adev->dev, "\t Faulty UTCL2 client ID: %s (0x%x)\n",
+                       mmhub_cid ? mmhub_cid : "unknown", cid);
         }
-
+       dev_err(adev->dev, "\t MORE_FAULTS: 0x%lx\n",
+               REG_GET_FIELD(status,
+               VM_L2_PROTECTION_FAULT_STATUS, MORE_FAULTS));
+       dev_err(adev->dev, "\t WALKER_ERROR: 0x%lx\n",
+               REG_GET_FIELD(status,
+               VM_L2_PROTECTION_FAULT_STATUS, WALKER_ERROR));
+       dev_err(adev->dev, "\t PERMISSION_FAULTS: 0x%lx\n",
+               REG_GET_FIELD(status,
+               VM_L2_PROTECTION_FAULT_STATUS, PERMISSION_FAULTS));
+       dev_err(adev->dev, "\t MAPPING_ERROR: 0x%lx\n",
+               REG_GET_FIELD(status,
+               VM_L2_PROTECTION_FAULT_STATUS, MAPPING_ERROR));
+       dev_err(adev->dev, "\t RW: 0x%x\n", rw);
         return 0;
 }
 
--
2.17.1

_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://nam11.safelinks.protection.outlook.com/?url="">
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux