drm/amdkfd: drop warning in event_interrupt_isr_v1*()

Commit ded8b3c36f17 ("drm/amdgpu: properly handle GC vs MM in amdgpu_vmid_mgr_init()")
enables all 16 vmids for MMHUB on GC 10 and newer for KGD since
there are no KFD resources using MMHUB.  With this change, KFD
starts seeing MMHUB vmids in it's range with no pasid set.  As
such there is no need to warn, we can just ignore those interrupts.

Fixes: aded8b3c36f1 ("drm/amdgpu: properly handle GC vs MM in amdgpu_vmid_mgr_init()")
Reviewed-by: Felix Kuehling <felix.kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_int_process_v10.c b/drivers/gpu/drm/amd/amdkfd/kfd_int_process_v10.c
index 54870b4c50000b288c460607aa6d6f8eeb730827..3e1ad89747979b158fa7ebe5b43ba49ac14c00c6 100644 (file)
--- a/drivers/gpu/drm/amd/amdkfd/kfd_int_process_v10.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_int_process_v10.c
@@ -175,8 +175,7 @@ static bool event_interrupt_isr_v10(struct kfd_node *dev,
                data[0], data[1], data[2], data[3], data[4], data[5], data[6],
                data[7]);
 
-       /* If there is no valid PASID, it's likely a bug */
-       if (WARN_ONCE(pasid == 0, "Bug: No PASID in KFD interrupt"))
+       if (pasid == 0)
                return 0;
 
        /* Interrupt types we care about: various signals and faults.

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_int_process_v11.c b/drivers/gpu/drm/amd/amdkfd/kfd_int_process_v11.c
index eb88ba8d8b01247fa2aa7651e518023bf1d6b444..2788a52714d1047345a39f36b1d8373c19e99f84 100644 (file)
--- a/drivers/gpu/drm/amd/amdkfd/kfd_int_process_v11.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_int_process_v11.c
@@ -287,8 +287,7 @@ static bool event_interrupt_isr_v11(struct kfd_node *dev,
                data[0], data[1], data[2], data[3], data[4], data[5], data[6],
                data[7]);
 
-       /* If there is no valid PASID, it's likely a bug */
-       if (WARN_ONCE(pasid == 0, "Bug: No PASID in KFD interrupt"))
+       if (pasid == 0)
                return false;
 
        /* Interrupt types we care about: various signals and faults.