[PATCH] drm/amdgpu: fix off by one in amdgpu_vm_handle_fault

Felix Kuehling felix.kuehling at amd.com
Mon Nov 2 18:00:38 UTC 2020


Am 2020-11-02 um 11:23 a.m. schrieb Christian König:
> The value is inclusive, not exclusive.
>
> Signed-off-by: Christian König <christian.koenig at amd.com>

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


> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> index dc0bc550e42b..fdbe7d4e8b8b 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> @@ -3351,7 +3351,7 @@ bool amdgpu_vm_handle_fault(struct amdgpu_device *adev, unsigned int pasid,
>  	}
>  
>  	r = amdgpu_vm_bo_update_mapping(adev, adev, vm, true, false, NULL, addr,
> -					addr + 1, flags, value, NULL, NULL,
> +					addr, flags, value, NULL, NULL,
>  					NULL);
>  	if (r)
>  		goto error_unlock;


More information about the amd-gfx mailing list