aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
diff options
context:
space:
mode:
authorMonk Liu <Monk.Liu@amd.com>2016-03-09 23:14:44 -0500
committerAlex Deucher <alexander.deucher@amd.com>2016-05-02 15:20:07 -0400
commitb6723c8da55af5309cf06e71a5228f3c02846c5a (patch)
tree268b16b48fca9556569ce72c02b9c1a0985b3df3 /drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
parent0de2479c953ae07fd11e7b1bc8d4fc831e6842bb (diff)
drm/amdgpu: use ref to keep job alive
this is to fix fatal page fault error that occured if: job is signaled/released after its timeout work is already put to the global queue (in this case the cancel_delayed_work will return false), which will lead to NX-protection error page fault during job_timeout_func. Signed-off-by: Monk Liu <Monk.Liu@amd.com> Reviewed-by: Chunming Zhou <david1.zhou@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c')
-rw-r--r--drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
index 9025671d21c3..d7e0b0b9a1bc 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
@@ -872,6 +872,7 @@ static int amdgpu_cs_submit(struct amdgpu_cs_parser *p,
872 r = amd_sched_job_init(&job->base, &ring->sched, 872 r = amd_sched_job_init(&job->base, &ring->sched,
873 &p->ctx->rings[ring->idx].entity, 873 &p->ctx->rings[ring->idx].entity,
874 amdgpu_job_timeout_func, 874 amdgpu_job_timeout_func,
875 amdgpu_job_free_func,
875 p->filp, &fence); 876 p->filp, &fence);
876 if (r) { 877 if (r) {
877 amdgpu_job_free(job); 878 amdgpu_job_free(job);