Patch 2 ~ 6 are Reviewed-by: Junwei Zhang <Jerry.Zhang at amd.com> Jerry On 08/22/2018 11:05 PM, Christian König wrote: > Preparation for following changes. This validates the root PD twice, > but the overhead of that should be minimal. > > Signed-off-by: Christian König <christian.koenig at amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > index 73b8dcaf66e6..53ce9982a5ee 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > @@ -291,11 +291,11 @@ int amdgpu_vm_validate_pt_bos(struct amdgpu_device *adev, struct amdgpu_vm *vm, > list_for_each_entry_safe(bo_base, tmp, &vm->evicted, vm_status) { > struct amdgpu_bo *bo = bo_base->bo; > > - if (bo->parent) { > - r = validate(param, bo); > - if (r) > - break; > + r = validate(param, bo); > + if (r) > + break; > > + if (bo->parent) { > spin_lock(&glob->lru_lock); > ttm_bo_move_to_lru_tail(&bo->tbo); > if (bo->shadow) >