[xorg 1/3] dri2: Allow GetBuffers to match any format

From: Chris Wilson <chris-Y6uKTt2uX1cEflXRtASbqLVCufUGDwFn@public.gmane.org>
To: xorg-devel-go0+a7rfsptAfugRpC6u6w@public.gmane.org,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	mesa-dev-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: [xorg 1/3] dri2: Allow GetBuffers to match any format
Date: Mon, 19 Jan 2015 11:00:38 +0000	[thread overview]
Message-ID: <1421665245-5994-3-git-send-email-chris@chris-wilson.co.uk> (raw)
In-Reply-To: <1421665245-5994-1-git-send-email-chris-Y6uKTt2uX1cEflXRtASbqLVCufUGDwFn@public.gmane.org>

Since the introduction of DRI2GetBuffersWithFormat, the old
DRI2GetBuffers interface would always recreate all buffers all the time
as it was no longer agnostic to the format value being set by the DDXes.
This causes an issue with clients intermixing the two requests,
rendering any sharing or caching of buffers (e.g. for triple buffering)
void.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 hw/xfree86/dri2/dri2.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/hw/xfree86/dri2/dri2.c b/hw/xfree86/dri2/dri2.c
index 43a1899..f9f594d 100644
--- a/hw/xfree86/dri2/dri2.c
+++ b/hw/xfree86/dri2/dri2.c
@@ -464,14 +464,16 @@ find_attachment(DRI2DrawablePtr pPriv, unsigned attachment)
 static Bool
 allocate_or_reuse_buffer(DrawablePtr pDraw, DRI2ScreenPtr ds,
                          DRI2DrawablePtr pPriv,
-                         unsigned int attachment, unsigned int format,
+                         unsigned int attachment,
+                         int has_format, unsigned int format,
                          int dimensions_match, DRI2BufferPtr * buffer)
 {
     int old_buf = find_attachment(pPriv, attachment);
 
     if ((old_buf < 0)
         || attachment == DRI2BufferFrontLeft
-        || !dimensions_match || (pPriv->buffers[old_buf]->format != format)) {
+        || !dimensions_match
+        || (has_format && pPriv->buffers[old_buf]->format != format)) {
         *buffer = create_buffer(ds, pDraw, attachment, format);
         return TRUE;
 
@@ -549,7 +551,8 @@ do_get_buffers(DrawablePtr pDraw, int *width, int *height,
         const unsigned format = (has_format) ? *(attachments++) : 0;
 
         if (allocate_or_reuse_buffer(pDraw, ds, pPriv, attachment,
-                                     format, dimensions_match, &buffers[i]))
+                                     has_format, format, dimensions_match,
+                                     &buffers[i]))
             buffers_changed = 1;
 
         if (buffers[i] == NULL)
@@ -584,7 +587,7 @@ do_get_buffers(DrawablePtr pDraw, int *width, int *height,
 
     if (need_real_front > 0) {
         if (allocate_or_reuse_buffer(pDraw, ds, pPriv, DRI2BufferFrontLeft,
-                                     front_format, dimensions_match,
+                                     has_format, front_format, dimensions_match,
                                      &buffers[i]))
             buffers_changed = 1;
 
@@ -595,7 +598,7 @@ do_get_buffers(DrawablePtr pDraw, int *width, int *height,
 
     if (need_fake_front > 0) {
         if (allocate_or_reuse_buffer(pDraw, ds, pPriv, DRI2BufferFakeFrontLeft,
-                                     front_format, dimensions_match,
+                                     has_format, front_format, dimensions_match,
                                      &buffers[i]))
             buffers_changed = 1;
 
-- 
2.1.4

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
