[PATCH] radeon: disallow glamor on pre-R600 asics

[Michel Dänzer]

On Die, 2013-08-20 at 09:37 -0400, Alex Deucher wrote:
> I'm not sure they can handle the shaders properly, especially
> only older parts like r300.
> 
> This will avoid display corruption problems reported by people
> using glamor on older asics by falling back to EXA if they try
> and enable glamor.
> 
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
>  src/radeon_glamor.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/src/radeon_glamor.c b/src/radeon_glamor.c
> index d527900..7f00d5b 100644
> --- a/src/radeon_glamor.c
> +++ b/src/radeon_glamor.c
> @@ -93,6 +93,12 @@ radeon_glamor_pre_init(ScrnInfoPtr scrn)
>  	if (s && strcasecmp(s, "glamor") != 0)
>  		return FALSE;
>  
> +	if (info->ChipFamily < CHIP_FAMILY_R600) {
> +		xf86DrvMsg(scrn->scrnIndex, s ? X_ERROR : X_WARNING,
> +			   "glamor requires R600 or newer GPU, disabling.\n");
> +		return FALSE;
> +	}

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer           |                   http://www.amd.com
Libre software enthusiast         |          Debian, X and DRI developer