[PATCH xf86-video-ati] Don't enable DRI3 by default with EXA

[Deucher, Alexander]

> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Michel Dänzer
> Sent: Tuesday, July 12, 2016 4:40 AM
> To: amd-gfx at lists.freedesktop.org
> Subject: [PATCH xf86-video-ati] Don't enable DRI3 by default with EXA
> 
> From: Michel Dänzer <michel.daenzer at amd.com>
> 
> It doesn't work correctly in all cases, see e.g.
> https://bugs.freedesktop.org/show_bug.cgi?id=95475 . I'm not sure this
> is fixable, given EXA's architecture.
> 
> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  man/radeon.man   | 5 +++--
>  src/radeon_kms.c | 2 +-
>  2 files changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/man/radeon.man b/man/radeon.man
> index cacafb5..44603a5 100644
> --- a/man/radeon.man
> +++ b/man/radeon.man
> @@ -271,8 +271,9 @@ Sea Islands.
>  Define the maximum level of DRI to enable. Valid values are 2 for DRI2 or 3
> for DRI3.
>  The default is
>  .B 3 for DRI3
> -if the driver was compiled for Xorg >= 1.18.3, otherwise
> -.B 2 for DRI2.
> +if the driver was compiled for Xorg >= 1.18.3 and glamor is enabled,
> otherwise
> +.B 2 for DRI2. Note:
> +DRI3 may not work correctly in all cases with EXA, enable at your own risk.
>  .TP
>  .BI "Option \*qEnablePageFlip\*q \*q" boolean \*q
>  Enable DRI2 page flipping.  The default is
> diff --git a/src/radeon_kms.c b/src/radeon_kms.c
> index d971a39..d21ff2b 100644
> --- a/src/radeon_kms.c
> +++ b/src/radeon_kms.c
> @@ -1788,7 +1788,7 @@ Bool
> RADEONScreenInit_KMS(SCREEN_INIT_ARGS_DECL)
>  #endif
> 
>  #if XORG_VERSION_CURRENT >= XORG_VERSION_NUMERIC(1,18,3,0,0)
> -    value = TRUE;
> +    value = info->use_glamor;
>  #else
>      value = FALSE;
>  #endif
> --
> 2.8.1
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx