diff options
author | Christian König <christian.koenig@amd.com> | 2018-07-13 03:58:49 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2018-07-16 17:11:52 -0400 |
commit | eb3961a57424a5c3dae44576d1c88e64a818d871 (patch) | |
tree | 81ecfe0228b383abe5f2973bc62543e6ebeb2da4 /drivers/gpu | |
parent | 050d9d43a7d3847fd9726a675bf644b124518425 (diff) |
drm/amdgpu: remove fence context from the job
Can be obtained directly from the fence as well.
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>
Diffstat (limited to 'drivers/gpu')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 1 | ||||
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 2 | ||||
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 1 | ||||
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_job.h | 1 |
4 files changed, 1 insertions, 4 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c index 6d8df76b5a5d..6eb7ee859ffd 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | |||
@@ -1239,7 +1239,6 @@ static int amdgpu_cs_submit(struct amdgpu_cs_parser *p, | |||
1239 | } | 1239 | } |
1240 | 1240 | ||
1241 | job->owner = p->filp; | 1241 | job->owner = p->filp; |
1242 | job->fence_ctx = entity->fence_context; | ||
1243 | p->fence = dma_fence_get(&job->base.s_fence->finished); | 1242 | p->fence = dma_fence_get(&job->base.s_fence->finished); |
1244 | 1243 | ||
1245 | r = amdgpu_ctx_add_fence(p->ctx, ring, p->fence, &seq); | 1244 | r = amdgpu_ctx_add_fence(p->ctx, ring, p->fence, &seq); |
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c index ce7739832d29..5518e623fed2 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | |||
@@ -139,7 +139,7 @@ int amdgpu_ib_schedule(struct amdgpu_ring *ring, unsigned num_ibs, | |||
139 | /* ring tests don't use a job */ | 139 | /* ring tests don't use a job */ |
140 | if (job) { | 140 | if (job) { |
141 | vm = job->vm; | 141 | vm = job->vm; |
142 | fence_ctx = job->fence_ctx; | 142 | fence_ctx = job->base.s_fence->scheduled.context; |
143 | } else { | 143 | } else { |
144 | vm = NULL; | 144 | vm = NULL; |
145 | fence_ctx = 0; | 145 | fence_ctx = 0; |
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c index 2bd56760c744..10e0a97c7c03 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | |||
@@ -132,7 +132,6 @@ int amdgpu_job_submit(struct amdgpu_job *job, struct amdgpu_ring *ring, | |||
132 | return r; | 132 | return r; |
133 | 133 | ||
134 | job->owner = owner; | 134 | job->owner = owner; |
135 | job->fence_ctx = entity->fence_context; | ||
136 | *f = dma_fence_get(&job->base.s_fence->finished); | 135 | *f = dma_fence_get(&job->base.s_fence->finished); |
137 | amdgpu_job_free_resources(job); | 136 | amdgpu_job_free_resources(job); |
138 | amdgpu_ring_priority_get(job->ring, job->base.s_priority); | 137 | amdgpu_ring_priority_get(job->ring, job->base.s_priority); |
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.h index 35bb93254eb2..3151692312bd 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.h +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.h | |||
@@ -45,7 +45,6 @@ struct amdgpu_job { | |||
45 | uint32_t preamble_status; | 45 | uint32_t preamble_status; |
46 | uint32_t num_ibs; | 46 | uint32_t num_ibs; |
47 | void *owner; | 47 | void *owner; |
48 | uint64_t fence_ctx; /* the fence_context this job uses */ | ||
49 | bool vm_needs_flush; | 48 | bool vm_needs_flush; |
50 | uint64_t vm_pd_addr; | 49 | uint64_t vm_pd_addr; |
51 | unsigned vmid; | 50 | unsigned vmid; |