[PATCH xf86-video-ati 2/2] Don't destroy current FB if drmModeAddFB fails

Michel Dänzer michel at daenzer.net
Tue Feb 7 09:47:37 UTC 2017


From: Michel Dänzer <michel.daenzer at amd.com>

It would probably result in a black screen.

Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
---
 src/drmmode_display.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/drmmode_display.c b/src/drmmode_display.c
index b64be462d..c39b32558 100644
--- a/src/drmmode_display.c
+++ b/src/drmmode_display.c
@@ -2949,7 +2949,8 @@ flip_error:
 		   strerror(errno));
 
 error:
-	if (flipdata && flipdata->flip_count <= 1) {
+	if (flipdata && flipdata->flip_count <= 1 &&
+	    drmmode->fb_id != flipdata->old_fb_id) {
 		drmModeRmFB(drmmode->fd, drmmode->fb_id);
 		drmmode->fb_id = flipdata->old_fb_id;
 	}
-- 
2.11.0



More information about the amd-gfx mailing list