No, if the vm size is small, there may only on root pd entry. we need to make sure the mask >= 0;
Maybe this change revert Christian's commit: commit 72af632549b97ead9251bb155f08fefd1fb6f5c3 Author: Christian König <christian.koe...@amd.com> Date: Sat Sep 15 10:02:13 2018 +0200 drm/amdgpu: add amdgpu_vm_entries_mask v2 We can't get the mask for the root directory from the number of entries. So add a new function to avoid that problem. Best Regards Rex ________________________________ From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> on behalf of Zhang, Jerry(Junwei) <jerry.zh...@amd.com> Sent: Tuesday, October 23, 2018 1:12 PM To: Zhu, Rex; amd-gfx@lists.freedesktop.org; Deucher, Alexander; Koenig, Christian Subject: Re: [PATCH] drm/amdgpu: Fix amdgpu_vm_alloc_pts failed On 10/23/2018 11:29 AM, Rex Zhu wrote: > when the VA address located in the last PD entries, > the alloc_pts will faile. > > Use the right PD mask instand of hardcode, suggested > by jerry.zhang. > > Signed-off-by: Rex Zhu <rex....@amd.com> Thanks to verify that. Feel free to add Reviewed-by: Junwei Zhang <jerry.zh...@amd.com> Also like to get to know some background about these two functions from Christian. Perhaps we may make it more simple, e.g. merging them together. Regards, Jerry > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > index 054633b..3939013 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > @@ -202,8 +202,11 @@ static unsigned amdgpu_vm_num_entries(struct > amdgpu_device *adev, > static uint32_t amdgpu_vm_entries_mask(struct amdgpu_device *adev, > unsigned int level) > { > + unsigned shift = amdgpu_vm_level_shift(adev, > + adev->vm_manager.root_level); > + > if (level <= adev->vm_manager.root_level) > - return 0xffffffff; > + return (round_up(adev->vm_manager.max_pfn, 1 << shift) >> > shift) - 1; > else if (level != AMDGPU_VM_PTB) > return 0x1ff; > else _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx amd-gfx Info Page - freedesktop.org<https://lists.freedesktop.org/mailman/listinfo/amd-gfx> lists.freedesktop.org To see the collection of prior postings to the list, visit the amd-gfx Archives.. Using amd-gfx: To post a message to all the list members, send email to amd-gfx@lists.freedesktop.org. You can subscribe to the list, or change your existing subscription, in the sections below.
_______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx