[PATCH] drm/amdgpu: sync the VM PD/PT before clearing it

felix.kuehling@xxxxxxx (Felix Kuehling) · Mon, 5 Feb 2018 11:40:21 -0500



Reviewed-by: Felix Kuehling <Felix.Kuehling at amd.com>


On 2018-02-05 07:28 AM, Christian KÃ¶nig wrote:
> Otherwise we might overwrite stuff which is still in use.
>
> Signed-off-by: Christian KÃ¶nig <christian.koenig at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> index 18ce47608bf1..0572d6072baa 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> @@ -329,6 +329,11 @@ static int amdgpu_vm_clear_bo(struct amdgpu_device *adev,
>  	amdgpu_ring_pad_ib(ring, &job->ibs[0]);
>  
>  	WARN_ON(job->ibs[0].length_dw > 64);
> +	r = amdgpu_sync_resv(adev, &job->sync, bo->tbo.resv,
> +			     AMDGPU_FENCE_OWNER_UNDEFINED, false);
> +	if (r)
> +		goto error_free;
> +
>  	r = amdgpu_job_submit(job, ring, &vm->entity,
>  			      AMDGPU_FENCE_OWNER_UNDEFINED, &fence);
>  	if (r)