Attachment 108267 Details for Bug 84944 – Add some debugging output to DRI2CanFlip

[patch] Add some debugging output to DRI2CanFlip

DRI2CanFlip-debug.diff (text/plain), 1.24 KB, created by Michel Dänzer on 2014-10-23 01:39:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Michel Dänzer

Created: 2014-10-23 01:39:54 UTC

Size: 1.24 KB

patch

obsolete

>diff --git a/hw/xfree86/dri2/dri2.c b/hw/xfree86/dri2/dri2.c
>index 6459f11..bbda360 100644
>--- a/hw/xfree86/dri2/dri2.c
>+++ b/hw/xfree86/dri2/dri2.c
>@@ -927,10 +927,15 @@ DRI2CanFlip(DrawablePtr pDraw)
> 
>     pWin = (WindowPtr) pDraw;
>     pWinPixmap = pScreen->GetWindowPixmap(pWin);
>-    if (pRootPixmap != pWinPixmap)
>+    if (pRootPixmap != pWinPixmap) {
>+        ErrorF("%s: Window pixmap != screen pixmap\n", __func__);
>         return FALSE;
>-    if (!RegionEqual(&pWin->clipList, &pRoot->winSize))
>+    }
>+    if (!RegionEqual(&pWin->clipList, &pRoot->winSize)) {
>+        ErrorF("%s: Window clipList doesn't match root window dimensions\n",
>+               __func__);
>         return FALSE;
>+    }
> 
>     /* Does the window match the pixmap exactly? */
>     if (pDraw->x != 0 || pDraw->y != 0 ||
>@@ -938,8 +943,11 @@ DRI2CanFlip(DrawablePtr pDraw)
>         pDraw->x != pWinPixmap->screen_x || pDraw->y != pWinPixmap->screen_y ||
> #endif
>         pDraw->width != pWinPixmap->drawable.width ||
>-        pDraw->height != pWinPixmap->drawable.height)
>+        pDraw->height != pWinPixmap->drawable.height) {
>+        ErrorF("%s: Window pixmap dimensions dosn't match window dimensions\n",
>+               __func__);
>         return FALSE;
>+    }
> 
>     return TRUE;
> }

Actions: View | Splinter Review

Attachments on bug 84944: 107747 | 107748 | 108227 | 108267 | 108289

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.