diff options
| author | Monk Liu <Monk.Liu@amd.com> | 2016-08-25 03:40:48 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2016-09-12 18:12:17 -0400 |
| commit | 3aecd24c65b9539b6faac2a52a9aaa7bc90f4677 (patch) | |
| tree | 60f9c78fd04b2865ccdfe093e96b0608ceefc7df /drivers/gpu | |
| parent | d4946ccfd0373610ffa6181cffae236bf07d3aff (diff) | |
drm/amdgpu: change job->ctx field name
job->ctx actually is a fence_context of the entity
it belongs to, naming it as ctx is too vague, and
we'll need add amdgpu_ctx into the job structure
later.
Signed-off-by: Monk Liu <Monk.Liu@amd.com>
Reviewed-by: Christian König <christian.koenig@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.h | 2 | ||||
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 2 | ||||
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 12 | ||||
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 2 |
4 files changed, 9 insertions, 9 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h index 39baabe5f092..10ec29c50077 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h | |||
| @@ -1241,7 +1241,7 @@ struct amdgpu_job { | |||
| 1241 | struct fence *fence; /* the hw fence */ | 1241 | struct fence *fence; /* the hw fence */ |
| 1242 | uint32_t num_ibs; | 1242 | uint32_t num_ibs; |
| 1243 | void *owner; | 1243 | void *owner; |
| 1244 | uint64_t ctx; | 1244 | uint64_t fence_ctx; /* the fence_context this job uses */ |
| 1245 | bool vm_needs_flush; | 1245 | bool vm_needs_flush; |
| 1246 | unsigned vm_id; | 1246 | unsigned vm_id; |
| 1247 | uint64_t vm_pd_addr; | 1247 | uint64_t vm_pd_addr; |
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c index e29e7b9ca3a6..56bde6436a1f 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | |||
| @@ -989,7 +989,7 @@ static int amdgpu_cs_submit(struct amdgpu_cs_parser *p, | |||
| 989 | } | 989 | } |
| 990 | 990 | ||
| 991 | job->owner = p->filp; | 991 | job->owner = p->filp; |
| 992 | job->ctx = entity->fence_context; | 992 | job->fence_ctx = entity->fence_context; |
| 993 | p->fence = fence_get(&job->base.s_fence->finished); | 993 | p->fence = fence_get(&job->base.s_fence->finished); |
| 994 | cs->out.handle = amdgpu_ctx_add_fence(p->ctx, ring, p->fence); | 994 | cs->out.handle = amdgpu_ctx_add_fence(p->ctx, ring, p->fence); |
| 995 | job->uf_sequence = cs->out.handle; | 995 | job->uf_sequence = cs->out.handle; |
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c index 11f2fba4653d..04263f0fd1af 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | |||
| @@ -124,7 +124,7 @@ int amdgpu_ib_schedule(struct amdgpu_ring *ring, unsigned num_ibs, | |||
| 124 | bool skip_preamble, need_ctx_switch; | 124 | bool skip_preamble, need_ctx_switch; |
| 125 | unsigned patch_offset = ~0; | 125 | unsigned patch_offset = ~0; |
| 126 | struct amdgpu_vm *vm; | 126 | struct amdgpu_vm *vm; |
| 127 | uint64_t ctx; | 127 | uint64_t fence_ctx; |
| 128 | 128 | ||
| 129 | unsigned i; | 129 | unsigned i; |
| 130 | int r = 0; | 130 | int r = 0; |
| @@ -135,10 +135,10 @@ int amdgpu_ib_schedule(struct amdgpu_ring *ring, unsigned num_ibs, | |||
| 135 | /* ring tests don't use a job */ | 135 | /* ring tests don't use a job */ |
| 136 | if (job) { | 136 | if (job) { |
| 137 | vm = job->vm; | 137 | vm = job->vm; |
| 138 | ctx = job->ctx; | 138 | fence_ctx = job->fence_ctx; |
| 139 | } else { | 139 | } else { |
| 140 | vm = NULL; | 140 | vm = NULL; |
| 141 | ctx = 0; | 141 | fence_ctx = 0; |
| 142 | } | 142 | } |
| 143 | 143 | ||
| 144 | if (!ring->ready) { | 144 | if (!ring->ready) { |
| @@ -174,8 +174,8 @@ int amdgpu_ib_schedule(struct amdgpu_ring *ring, unsigned num_ibs, | |||
| 174 | /* always set cond_exec_polling to CONTINUE */ | 174 | /* always set cond_exec_polling to CONTINUE */ |
| 175 | *ring->cond_exe_cpu_addr = 1; | 175 | *ring->cond_exe_cpu_addr = 1; |
| 176 | 176 | ||
| 177 | skip_preamble = ring->current_ctx == ctx; | 177 | skip_preamble = ring->current_ctx == fence_ctx; |
| 178 | need_ctx_switch = ring->current_ctx != ctx; | 178 | need_ctx_switch = ring->current_ctx != fence_ctx; |
| 179 | for (i = 0; i < num_ibs; ++i) { | 179 | for (i = 0; i < num_ibs; ++i) { |
| 180 | ib = &ibs[i]; | 180 | ib = &ibs[i]; |
| 181 | 181 | ||
| @@ -209,7 +209,7 @@ int amdgpu_ib_schedule(struct amdgpu_ring *ring, unsigned num_ibs, | |||
| 209 | if (patch_offset != ~0 && ring->funcs->patch_cond_exec) | 209 | if (patch_offset != ~0 && ring->funcs->patch_cond_exec) |
| 210 | amdgpu_ring_patch_cond_exec(ring, patch_offset); | 210 | amdgpu_ring_patch_cond_exec(ring, patch_offset); |
| 211 | 211 | ||
| 212 | ring->current_ctx = ctx; | 212 | ring->current_ctx = fence_ctx; |
| 213 | if (ring->funcs->emit_switch_buffer) | 213 | if (ring->funcs->emit_switch_buffer) |
| 214 | amdgpu_ring_emit_switch_buffer(ring); | 214 | amdgpu_ring_emit_switch_buffer(ring); |
| 215 | amdgpu_ring_commit(ring); | 215 | amdgpu_ring_commit(ring); |
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c index 6674d40eb3ab..ac8d40168a47 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | |||
| @@ -124,7 +124,7 @@ int amdgpu_job_submit(struct amdgpu_job *job, struct amdgpu_ring *ring, | |||
| 124 | return r; | 124 | return r; |
| 125 | 125 | ||
| 126 | job->owner = owner; | 126 | job->owner = owner; |
| 127 | job->ctx = entity->fence_context; | 127 | job->fence_ctx = entity->fence_context; |
| 128 | *f = fence_get(&job->base.s_fence->finished); | 128 | *f = fence_get(&job->base.s_fence->finished); |
| 129 | amdgpu_job_free_resources(job); | 129 | amdgpu_job_free_resources(job); |
| 130 | amd_sched_entity_push_job(&job->base); | 130 | amd_sched_entity_push_job(&job->base); |
