[PATCH] drm/amdgpu: Handle job is NULL use case in amdgpu_device_gpu_recover

Christian König ckoenig.leichtzumerken at gmail.com
Wed Aug 28 09:21:07 UTC 2019


Am 27.08.19 um 20:17 schrieb Andrey Grodzovsky:
> This should be checked at all places job is accessed.
>
> Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com>

Reviewed-by: Christian König <christian.koenig at amd.com>

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index 02b3e7d..190d9a3 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -3790,14 +3790,14 @@ int amdgpu_device_gpu_recover(struct amdgpu_device *adev,
>   
>   	if (hive && !mutex_trylock(&hive->reset_lock)) {
>   		DRM_INFO("Bailing on TDR for s_job:%llx, hive: %llx as another already in progress",
> -			 job->base.id, hive->hive_id);
> +			  job ? job->base.id : -1, hive->hive_id);
>   		return 0;
>   	}
>   
>   	/* Start with adev pre asic reset first for soft reset check.*/
>   	if (!amdgpu_device_lock_adev(adev, !hive)) {
>   		DRM_INFO("Bailing on TDR for s_job:%llx, as another already in progress",
> -					 job->base.id);
> +			  job ? job->base.id : -1);
>   		return 0;
>   	}
>   
> @@ -3838,7 +3838,7 @@ int amdgpu_device_gpu_recover(struct amdgpu_device *adev,
>   			if (!ring || !ring->sched.thread)
>   				continue;
>   
> -			drm_sched_stop(&ring->sched, &job->base);
> +			drm_sched_stop(&ring->sched, job ? &job->base : NULL);
>   		}
>   	}
>   
> @@ -3864,7 +3864,7 @@ int amdgpu_device_gpu_recover(struct amdgpu_device *adev,
>   
>   	/* Guilty job will be freed after this*/
>   	r = amdgpu_device_pre_asic_reset(adev,
> -					 job,
> +					 job ? job : NULL,
>   					 &need_full_reset);
>   	if (r) {
>   		/*TODO Should we stop ?*/



More information about the amd-gfx mailing list