amdgpu_userq_fence_driver_alloc is always called from user context so use
xa_store_irq to streamline it a bit.

Signed-off-by: Tvrtko Ursulin <[email protected]>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c
index a96b84c48acb..38d120c72edf 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c
@@ -79,7 +79,6 @@ int amdgpu_userq_fence_driver_alloc(struct amdgpu_device 
*adev,
                                    struct amdgpu_usermode_queue *userq)
 {
        struct amdgpu_userq_fence_driver *fence_drv;
-       unsigned long flags;
        int r;
 
        fence_drv = kzalloc(sizeof(*fence_drv), GFP_KERNEL);
@@ -102,10 +101,8 @@ int amdgpu_userq_fence_driver_alloc(struct amdgpu_device 
*adev,
        fence_drv->context = dma_fence_context_alloc(1);
        get_task_comm(fence_drv->timeline_name, current);
 
-       xa_lock_irqsave(&adev->userq_xa, flags);
-       r = xa_err(__xa_store(&adev->userq_xa, userq->doorbell_index,
-                             fence_drv, GFP_KERNEL));
-       xa_unlock_irqrestore(&adev->userq_xa, flags);
+       r = xa_err(xa_store_irq(&adev->userq_xa, userq->doorbell_index,
+                               fence_drv, GFP_KERNEL));
        if (r)
                goto free_seq64;
 
-- 
2.51.1

Reply via email to