[PATCH] drm/amdkfd: pages_addr offset must be 0 for system range

[Felix Kuehling]

Am 2021-06-04 um 7:10 p.m. schrieb Philip Yang:
> prange->offset is for VRAM range mm_nodes, if multiple ranges share same
> mm_nodes, migrate range back to VRAM will reuse the VRAM at offset of
> the same mm_nodes. For system memory pages_addr array, the offset is
> always 0, otherwise, update GPU mapping will use incorrect system memory
> page, and cause system memory corruption.
>
> Signed-off-by: Philip Yang <Philip.Yang at amd.com>

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


> ---
>  drivers/gpu/drm/amd/amdkfd/kfd_svm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
> index f2901e7a993e..0f18bd0dc64e 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
> @@ -1153,7 +1153,7 @@ svm_range_map_to_gpu(struct amdgpu_device *adev, struct amdgpu_vm *vm,
>  
>  	prange->mapping.start = prange->start;
>  	prange->mapping.last = prange->last;
> -	prange->mapping.offset = prange->offset;
> +	prange->mapping.offset = prange->ttm_res ? prange->offset : 0;
>  	pte_flags = svm_range_get_pte_flags(adev, prange);
>  
>  	r = amdgpu_vm_bo_update_mapping(adev, bo_adev, vm, false, false, NULL,