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.Zhu@xxxxxxx>
Thanks to verify that.
Feel free to add
Reviewed-by: Junwei Zhang
<Jerry.Zhang@xxxxxxx>
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@xxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxx. You can
subscribe to the list, or change your
existing subscription, in the sections
below. |
