On 17-07-19 11:26 AM, Christian König wrote:
> From: Christian König <christian.koe...@amd.com>
>
> The fragment bits work differently for Vega10 compared to previous 
> generations.
>
> Increase the fragment size to 2MB for now to better handle that.
>
> v2: handle the hardware setup as well
>
> Signed-off-by: Christian König <christian.koe...@amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c  | 5 +++--
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c   | 5 +++--
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h   | 4 +++-
>  drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c | 4 +++-
>  drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c  | 4 +++-
>  5 files changed, 15 insertions(+), 7 deletions(-)
>
>
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
> @@ -51,7 +51,9 @@ struct amdgpu_bo_list_entry;
>  #define AMDGPU_VM_PTB_ALIGN_SIZE   32768
>  
>  /* LOG2 number of continuous pages for the fragment field */
> -#define AMDGPU_LOG2_PAGES_PER_FRAG 4
> +#define AMDGPU_LOG2_PAGES_PER_FRAG(adev) \
> +     ((adev)->asic_type < CHIP_VEGA10 ? 4 : \
> +      (adev)->vm_manager.block_size)

With the hardware configuration changed for 256KB BigK TLBs, should this
be changed to 256KB fragments on GFX9 now to get another small TLB boost
for allocations smaller than 2MB?

Regards,
  Felix


_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to