On 10/16/26 14:09, Sunil Khatri wrote: > In function amdgpu_vm_bo_insert_map, amdgpu_vm_bo_needs_update is > skipped due to amdgpu_vm_is_bo_always_valid but it should not skip > for bo's which are allocate have it's own dma_resv object or NULL. > > Signed-off-by: Sunil Khatri <[email protected]>
Reviewed-by: Christian König <[email protected]> for both. > --- > 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 de992405ed718..3eb778add39be 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > @@ -1798,7 +1798,7 @@ static void amdgpu_vm_bo_insert_map(struct > amdgpu_device *adev, > if (mapping->flags & AMDGPU_VM_PAGE_PRT) > amdgpu_vm_prt_get(adev); > > - if (amdgpu_vm_is_bo_always_valid(vm, bo) && !bo_va->base.moved) > + if (!bo_va->base.moved) > amdgpu_vm_bo_needs_update(&bo_va->base); > > trace_amdgpu_vm_bo_map(bo_va, mapping);
