Re: [PATCH 2/3] drm/amdgpu: Size the vm_pte_scheds array by maximum number of SDMA instances

["Christian König" <christian.koenig@amd.com>]

On 10/20/26 15:27, Tvrtko Ursulin wrote:
> The array is populated from a loop capped by adev->sdma.num_instances so
> it makes sense to size it using AMDGPU_MAX_SDMA_INSTANCES and save some
> memory.
> 
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@igalia.com>
> Cc: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
> Cc: Felix Kuehling <felix.kuehling@amd.com>
> Cc: Christian König <christian.koenig@amd.com>
> Cc: Alex Deucher <alexander.deucher@amd.com>
> Cc: Timur Kristóf <timur.kristof@gmail.com>

Reviewed-by: Christian König <christian.koenig@amd.com>

> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
> index b8aa45db1d84..98c48c45ce61 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
> @@ -395,7 +395,7 @@ struct amdgpu_vm_manager {
>  	u64					vram_base_offset;
>  	/* vm pte handling */
>  	const struct amdgpu_vm_pte_funcs	*vm_pte_funcs;
> -	struct drm_gpu_scheduler		*vm_pte_scheds[AMDGPU_MAX_RINGS];
> +	struct drm_gpu_scheduler		*vm_pte_scheds[AMDGPU_MAX_SDMA_INSTANCES];
>  	unsigned				vm_pte_num_scheds;
>  	struct amdgpu_ring			*page_fault;
>