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 <[email protected]>
Reviewed-by: Felix Kuehling <[email protected]> > --- > 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, _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
