[PATCH 1/2] drm/amdgpu/mes11: fix set_hw_resources_1 calculation
Christian König
christian.koenig at amd.com
Mon Feb 10 19:41:21 UTC 2025
Am 10.02.25 um 17:27 schrieb Alex Deucher:
> It's GPU page size not CPU page size. In most cases they
> are the same, but not always. This can lead to overallocation
> on systems with larger pages.
>
> Cc: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>
> Cc: Christian König <christian.koenig at amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
Reviewed-by: Christian König <christian.koenig at amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/mes_v11_0.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> index bf51f3dcc130e..e862a3febe2b2 100644
> --- a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c
> @@ -743,7 +743,7 @@ static int mes_v11_0_set_hw_resources(struct amdgpu_mes *mes)
>
> static int mes_v11_0_set_hw_resources_1(struct amdgpu_mes *mes)
> {
> - int size = 128 * PAGE_SIZE;
> + int size = 128 * AMDGPU_GPU_PAGE_SIZE;
> int ret = 0;
> struct amdgpu_device *adev = mes->adev;
> union MESAPI_SET_HW_RESOURCES_1 mes_set_hw_res_pkt;
More information about the amd-gfx
mailing list