[PATCH xf86-video-ati] Bail if there's a problem with ShadowFB

[Deucher, Alexander]

> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Michel Dänzer
> Sent: Wednesday, October 04, 2017 5:13 AM
> To: amd-gfx at lists.freedesktop.org
> Subject: [PATCH xf86-video-ati] Bail if there's a problem with ShadowFB
> 
> From: Michel Dänzer <michel.daenzer at amd.com>
> 
> 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 <michel.daenzer at amd.com>

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

> ---
>  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
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx