From 0007c2f018ba663303d91d847e7c085269a23062 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michel=20D=C3=A4nzer?= Date: Wed, 8 Jun 2016 16:27:33 +0900 Subject: [PATCH] glamor: Reallocate linear pixmap BO if necessary for DRI2 PRIME Fixes corruption when using DRI2 PRIME render offloading with the master screen using this driver. Reported-by: Qiang Yu Reviewed-by: Alex Deucher Tested-by: Qiang Yu --- src/amdgpu_glamor.c | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/src/amdgpu_glamor.c b/src/amdgpu_glamor.c index 6fbe089..3f4b1af 100644 --- a/src/amdgpu_glamor.c +++ b/src/amdgpu_glamor.c @@ -327,10 +327,31 @@ amdgpu_glamor_share_pixmap_backing(PixmapPtr pixmap, ScreenPtr slave, void **handle_p) { ScreenPtr screen = pixmap->drawable.pScreen; + uint64_t tiling_info; CARD16 stride; CARD32 size; int fd; + tiling_info = amdgpu_pixmap_get_tiling_info(pixmap); + if (AMDGPU_TILING_GET(tiling_info, ARRAY_MODE) != 0) { + PixmapPtr linear; + + /* We don't want to re-allocate the screen pixmap as + * linear, to avoid trouble with page flipping + */ + if (screen->GetScreenPixmap(screen) == pixmap) + return FALSE; + + linear = screen->CreatePixmap(screen, pixmap->drawable.width, + pixmap->drawable.height, + pixmap->drawable.depth, + CREATE_PIXMAP_USAGE_SHARED); + if (!linear) + return FALSE; + + amdgpu_glamor_set_pixmap_bo(&pixmap->drawable, linear); + } + fd = glamor_fd_from_pixmap(screen, pixmap, &stride, &size); if (fd < 0) return FALSE;