diff options
| author | Christian König <christian.koenig@amd.com> | 2024-12-12 16:43:45 +0100 | 
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2024-12-18 12:39:07 -0500 | 
| commit | 26c95e838e6301b0230430ec2fadeabfcb07aeda (patch) | |
| tree | 6af1f8e19b229ee022610b77d4b0036914527d02 | |
| parent | 57f812d171af4ba233d3ed7c94dfa5b8e92dcc04 (diff) | |
drm/amdgpu: set the VM pointer to NULL in amdgpu_job_prepare
As soon as the prepare phase is completed the VM might be released,
better set it to NULL.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 7 | 
1 files changed, 7 insertions, 0 deletions
| diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c index 2e7e1c7dfe7c..04691753cabf 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c @@ -361,6 +361,13 @@ amdgpu_job_prepare_job(struct drm_sched_job *sched_job,  			dev_err(ring->adev->dev, "Error getting VM ID (%d)\n", r);  			goto error;  		} +		/* +		 * The VM structure might be released after the VMID is +		 * assigned, we had multiple problems with people trying to use +		 * the VM pointer so better set it to NULL. +		 */ +		if (!fence) +			job->vm = NULL;  	}  	return fence; | 
