Re: [PATCH] drm/amd/display: Don't ASSERT when total_planes == AMDGPU_MAX_PLANES

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2019-03-05 6:24 p.m., Nicholas Kazlauskas wrote:
> [Why]
> Can happen on ASICs with 6 planes, but this isn't a bug since we haven't
> written outside the array.
> 
> [How]
> Use <= instead of <.
> 
> Cc: Leo Li <sunpeng.li@xxxxxxx>
> Cc: Michel Dänzer <michel@xxxxxxxxxxx>
> Reported-by: Michel Dänzer <michel@xxxxxxxxxxx>
> Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@xxxxxxx>
> ---
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index bdbc8006d9ea..b1208883da1b 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -1923,7 +1923,7 @@ static int amdgpu_dm_initialize_drm_device(struct amdgpu_device *adev)
>  	primary_planes = dm->dc->caps.max_streams;
>  
>  	total_planes = primary_planes + overlay_planes;
> -	ASSERT(total_planes < AMDGPU_MAX_PLANES);
> +	ASSERT(total_planes <= AMDGPU_MAX_PLANES);
>  
>  	/*
>  	 * Initialize primary planes, implicit planes for legacy IOCTLS.
> 

Reviewed-by: Michel Dänzer <michel.daenzer@xxxxxxx>
Tested-by: Michel Dänzer <michel.daenzer@xxxxxxx>

Thanks!


-- 
Earthling Michel Dänzer               |              https://www.amd.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux