> -----Original Message-----
> From: amd-gfx [mailto:[email protected]] On Behalf
> Of Michel Dänzer
> Sent: Wednesday, October 04, 2017 5:13 AM
> To: [email protected]
> Subject: [PATCH xf86-video-ati] Bail if there's a problem with ShadowFB
> 
> From: Michel Dänzer <[email protected]>
> 
> If we hit a problem while setting up ShadowFB, just carying on trying
> to set up HW acceleration instead is unlikely to work.
> 
> Signed-off-by: Michel Dänzer <[email protected]>

Reviewed-by: Alex Deucher <[email protected]>

> ---
>  src/radeon_kms.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/src/radeon_kms.c b/src/radeon_kms.c
> index b982e425b..9ef51693a 100644
> --- a/src/radeon_kms.c
> +++ b/src/radeon_kms.c
> @@ -1331,9 +1331,10 @@ static Bool RADEONPreInitAccel_KMS(ScrnInfoPtr
> pScrn)
>       xf86DrvMsg(pScrn->scrnIndex, X_INFO,
>                  "GPU accel disabled or not working, using shadowfb for
> KMS\n");
>  shadowfb:
> -     info->r600_shadow_fb = TRUE;
>       if (!xf86LoadSubModule(pScrn, "shadow"))
> -         info->r600_shadow_fb = FALSE;
> +         return FALSE;
> +
> +     info->r600_shadow_fb = TRUE;
>       return TRUE;
>      }
> 
> @@ -2215,7 +2216,7 @@ Bool RADEONScreenInit_KMS(ScreenPtr pScreen,
> int argc, char **argv)
>       if (info->fb_shadow == NULL) {
>           xf86DrvMsg(pScrn->scrnIndex, X_ERROR,
>                         "Failed to allocate shadow framebuffer\n");
> -         info->r600_shadow_fb = FALSE;
> +         return FALSE;
>       } else {
>           if (!fbScreenInit(pScreen, info->fb_shadow,
>                             pScrn->virtualX, pScrn->virtualY,
> --
> 2.14.2
> 
> _______________________________________________
> amd-gfx mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to