drm/amdgpu: use sched_job_init to initialize sched_job

Consolidate job initialization in one place rather than
duplicating it in multiple places.

Signed-off-by: Monk Liu <Monk.Liu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
index 9392e50a7ba48243ad0d26e3385e57f3603bdd83..23266b454aecb91ae47122f025207f4ae8bfd243 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
@@ -862,27 +862,24 @@ static int amdgpu_cs_submit(struct amdgpu_cs_parser *p,
                            union drm_amdgpu_cs *cs)
 {
        struct amdgpu_ring *ring = p->job->ring;
-       struct amd_sched_fence *fence;
+       struct fence *fence;
        struct amdgpu_job *job;
+       int r;
 
        job = p->job;
        p->job = NULL;
 
-       job->base.sched = &ring->sched;
-       job->base.s_entity = &p->ctx->rings[ring->idx].entity;
-       job->owner = p->filp;
-
-       fence = amd_sched_fence_create(job->base.s_entity, p->filp);
-       if (!fence) {
+       r = amd_sched_job_init(&job->base, &ring->sched,
+                                               &p->ctx->rings[ring->idx].entity,
+                                               p->filp, &fence);
+       if (r) {
                amdgpu_job_free(job);
-               return -ENOMEM;
+               return r;
        }
 
-       job->base.s_fence = fence;
-       p->fence = fence_get(&fence->base);
-
-       cs->out.handle = amdgpu_ctx_add_fence(p->ctx, ring,
-                                             &fence->base);
+       job->owner = p->filp;
+       p->fence = fence_get(fence);
+       cs->out.handle = amdgpu_ctx_add_fence(p->ctx, ring, fence);
        job->ibs[job->num_ibs - 1].sequence = cs->out.handle;
 
        trace_amdgpu_cs_ioctl(job);

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
index 9c9b19e2f353b4f1d52a405164038078262c74f7..eb0f7890401abe6b71fb5df79ca659278777792c 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
@@ -87,16 +87,19 @@ int amdgpu_job_submit(struct amdgpu_job *job, struct amdgpu_ring *ring,
                      struct amd_sched_entity *entity, void *owner,
                      struct fence **f)
 {
+       struct fence *fence;
+       int r;
        job->ring = ring;
-       job->base.sched = &ring->sched;
-       job->base.s_entity = entity;
-       job->base.s_fence = amd_sched_fence_create(job->base.s_entity, owner);
-       if (!job->base.s_fence)
-               return -ENOMEM;
 
-       *f = fence_get(&job->base.s_fence->base);
+       if (!f)
+               return -EINVAL;
+
+       r = amd_sched_job_init(&job->base, &ring->sched, entity, owner, &fence);
+       if (r)
+               return r;
 
        job->owner = owner;
+       *f = fence_get(fence);
        amd_sched_entity_push_job(&job->base);
 
        return 0;

diff --git a/drivers/gpu/drm/amd/scheduler/gpu_scheduler.c b/drivers/gpu/drm/amd/scheduler/gpu_scheduler.c
index a5ff9458d359099ce5b57a033351b54219af4c54..b9d5822bece8dd13b375a9191fa891f42ce97ab8 100644 (file)
--- a/drivers/gpu/drm/amd/scheduler/gpu_scheduler.c
+++ b/drivers/gpu/drm/amd/scheduler/gpu_scheduler.c
@@ -335,6 +335,23 @@ void amd_sched_entity_push_job(struct amd_sched_job *sched_job)
                   amd_sched_entity_in(sched_job));
 }
 
+/* init a sched_job with basic field */
+int amd_sched_job_init(struct amd_sched_job *job,
+                                               struct amd_gpu_scheduler *sched,
+                                               struct amd_sched_entity *entity,
+                                               void *owner, struct fence **fence)
+{
+       job->sched = sched;
+       job->s_entity = entity;
+       job->s_fence = amd_sched_fence_create(entity, owner);
+       if (!job->s_fence)
+               return -ENOMEM;
+
+       if (fence)
+               *fence = &job->s_fence->base;
+       return 0;
+}
+
 /**
  * Return ture if we can push more jobs to the hw.
  */

diff --git a/drivers/gpu/drm/amd/scheduler/gpu_scheduler.h b/drivers/gpu/drm/amd/scheduler/gpu_scheduler.h
index 9403145d7beebf326768672ed75bf8091801ad38..74bbec837f58b7c75debdc29e88335b4eb6a8c9c 100644 (file)
--- a/drivers/gpu/drm/amd/scheduler/gpu_scheduler.h
+++ b/drivers/gpu/drm/amd/scheduler/gpu_scheduler.h
@@ -144,5 +144,8 @@ struct amd_sched_fence *amd_sched_fence_create(
        struct amd_sched_entity *s_entity, void *owner);
 void amd_sched_fence_scheduled(struct amd_sched_fence *fence);
 void amd_sched_fence_signal(struct amd_sched_fence *fence);
-
+int amd_sched_job_init(struct amd_sched_job *job,
+                                       struct amd_gpu_scheduler *sched,
+                                       struct amd_sched_entity *entity,
+                                       void *owner, struct fence **fence);
 #endif