The wrong define was used to check if the hqd is still active

Signed-off-by: Oded Gabbay <oded.gabbay at gmail.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c
index 2daad33..6a45330 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c
@@ -450,7 +450,8 @@ static int kgd_hqd_destroy(struct kgd_dev *kgd, uint32_t 
reset_type,

        while (true) {
                temp = RREG32(mmCP_HQD_ACTIVE);
-               if (temp & CP_HQD_ACTIVE__ACTIVE__SHIFT)
+               if (temp & (CP_HQD_ACTIVE__ACTIVE_MASK <<
+                               CP_HQD_ACTIVE__ACTIVE__SHIFT))
                        break;
                if (timeout == 0) {
                        pr_err("kfd: cp queue preemption time out (%dms)\n",
-- 
2.4.3

Reply via email to