Switch to using a job structure for IBs.

Signed-off-by: Alex Deucher <[email protected]>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c | 37 ++++++++++++++-----------
 1 file changed, 21 insertions(+), 16 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c
index fd881388d6125..9fb1946be1ba2 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c
@@ -817,7 +817,8 @@ static int vpe_ring_test_ib(struct amdgpu_ring *ring, long 
timeout)
 {
        struct amdgpu_device *adev = ring->adev;
        const uint32_t test_pattern = 0xdeadbeef;
-       struct amdgpu_ib ib = {};
+       struct amdgpu_job *job;
+       struct amdgpu_ib *ib;
        struct dma_fence *f = NULL;
        uint32_t index;
        uint64_t wb_addr;
@@ -832,23 +833,28 @@ static int vpe_ring_test_ib(struct amdgpu_ring *ring, 
long timeout)
        adev->wb.wb[index] = 0;
        wb_addr = adev->wb.gpu_addr + (index * 4);
 
-       ret = amdgpu_ib_get(adev, NULL, 256, AMDGPU_IB_POOL_DIRECT, &ib);
+       ret = amdgpu_job_alloc_with_ib(ring->adev, NULL, NULL, 256,
+                                      AMDGPU_IB_POOL_DIRECT, &job,
+                                      AMDGPU_KERNEL_JOB_ID_VPE_RING_TEST);
        if (ret)
                goto err0;
-
-       ib.ptr[0] = VPE_CMD_HEADER(VPE_CMD_OPCODE_FENCE, 0);
-       ib.ptr[1] = lower_32_bits(wb_addr);
-       ib.ptr[2] = upper_32_bits(wb_addr);
-       ib.ptr[3] = test_pattern;
-       ib.ptr[4] = VPE_CMD_HEADER(VPE_CMD_OPCODE_NOP, 0);
-       ib.ptr[5] = VPE_CMD_HEADER(VPE_CMD_OPCODE_NOP, 0);
-       ib.ptr[6] = VPE_CMD_HEADER(VPE_CMD_OPCODE_NOP, 0);
-       ib.ptr[7] = VPE_CMD_HEADER(VPE_CMD_OPCODE_NOP, 0);
-       ib.length_dw = 8;
-
-       ret = amdgpu_ib_schedule(ring, 1, &ib, NULL, &f);
-       if (ret)
+       ib = &job->ibs[0];
+
+       ib->ptr[0] = VPE_CMD_HEADER(VPE_CMD_OPCODE_FENCE, 0);
+       ib->ptr[1] = lower_32_bits(wb_addr);
+       ib->ptr[2] = upper_32_bits(wb_addr);
+       ib->ptr[3] = test_pattern;
+       ib->ptr[4] = VPE_CMD_HEADER(VPE_CMD_OPCODE_NOP, 0);
+       ib->ptr[5] = VPE_CMD_HEADER(VPE_CMD_OPCODE_NOP, 0);
+       ib->ptr[6] = VPE_CMD_HEADER(VPE_CMD_OPCODE_NOP, 0);
+       ib->ptr[7] = VPE_CMD_HEADER(VPE_CMD_OPCODE_NOP, 0);
+       ib->length_dw = 8;
+
+       ret = amdgpu_job_submit_direct(job, ring, &f);
+       if (ret) {
+               amdgpu_job_free(job);
                goto err1;
+       }
 
        ret = dma_fence_wait_timeout(f, false, timeout);
        if (ret <= 0) {
@@ -859,7 +865,6 @@ static int vpe_ring_test_ib(struct amdgpu_ring *ring, long 
timeout)
        ret = (le32_to_cpu(adev->wb.wb[index]) == test_pattern) ? 0 : -EINVAL;
 
 err1:
-       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        amdgpu_device_wb_free(adev, index);
-- 
2.52.0

Reply via email to