Am 29.06.2016 um 10:09 schrieb Chunming Zhou:
> We will re-submit jobs to recovery hw ring after gpu reset.
>
> Change-Id: I0f99bd14673ce0e0dbb7b3b6c2b050245824b9ca
> Signed-off-by: Chunming Zhou <David1.Zhou at amd.com>
NAK, this can lead to a deadlock in the SA because it isn't informed any
more about the fence protecting the IBs.
Should also not be necessary if you just want to resubmit the IBs.
Christian.
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
> index b50a845..83771c1 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
> @@ -98,7 +98,7 @@ static void amdgpu_job_free_resources(struct amdgpu_job *job)
> void amdgpu_job_free_cb(struct amd_sched_job *s_job)
> {
> struct amdgpu_job *job = container_of(s_job, struct amdgpu_job, base);
> -
> + amdgpu_job_free_resources(job);
> kfree(job);
> }
>
> @@ -178,7 +178,6 @@ static struct fence *amdgpu_job_run(struct amd_sched_job *sched_job)
>
> err:
> job->fence = fence;
> - amdgpu_job_free_resources(job);
> return fence;
> }
>
