drm/amdgpu: add amdgpu_job_submit_direct helper

Make sure that we properly initialize at least the sched member.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Junwei Zhang <Jerry.Zhang@amd.com>
Acked-by: Chunming Zhou <david1.zhou@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

1 files changed, 6 insertions, 11 deletions

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
index 9958e76d1c78..13977ea6a097 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
@@ -2075,24 +2075,19 @@ int amdgpu_copy_buffer(struct amdgpu_ring *ring, uint64_t src_offset,
 
         amdgpu_ring_pad_ib(ring, &job->ibs[0]);
         WARN_ON(job->ibs[0].length_dw > num_dw);
-        if (direct_submit) {
-                r = amdgpu_ib_schedule(ring, job->num_ibs, job->ibs,
-                                       NULL, fence);
-                job->fence = dma_fence_get(*fence);
-                if (r)
-                        DRM_ERROR("Error scheduling IBs (%d)\n", r);
-                amdgpu_job_free(job);
-        } else {
+        if (direct_submit)
+                r = amdgpu_job_submit_direct(job, ring, fence);
+        else
                 r = amdgpu_job_submit(job, &adev->mman.entity,
                                       AMDGPU_FENCE_OWNER_UNDEFINED, fence);
-                if (r)
-                        goto error_free;
-        }
+        if (r)
+                goto error_free;
 
         return r;
 
 error_free:
         amdgpu_job_free(job);
+        DRM_ERROR("Error scheduling IBs (%d)\n", r);
         return r;
 }