Attachment 37366 Details for Bug 28341 – Add flush, invalidate on dri2 swap, fixed.

[patch] Add flush, invalidate on dri2 swap, fixed.

0001-mesa-dri-radeon-Add-DRI2-flush-extension-to-so-we-sy.patch (text/plain), 1.96 KB, created by Mario Kleiner on 2010-07-25 08:15:20 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mario Kleiner

Created: 2010-07-25 08:15:20 UTC

Size: 1.96 KB

patch

obsolete

>From c31f53b9f54c986b65f11cd9dfec020cdb635b7a Mon Sep 17 00:00:00 2001
>From: Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
>Date: Sun, 25 Jul 2010 16:29:24 +0200
>Subject: [PATCH 1/2] mesa/dri/radeon: Add DRI2 flush extension to so we synchronize properly.
>
>When DRI2 swap buffer is pending (copy buffer not pageflipping)
>we need to make sure we have the flush extension so radeon doesn't
>resume rendering on the not yet blitted front buffer.
>
>Modified version of Jerome's patch to add flush extension
>in the correct place.
>
>This prepares a possible fix for:
>
>https://bugs.freedesktop.org/show_bug.cgi?id=28341
>https://bugs.freedesktop.org/show_bug.cgi?id=28410
>
>Signed-off-by: Jerome Glisse <jglisse@redhat.com>
>Signed-off-by: Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
>---
> src/mesa/drivers/dri/radeon/radeon_screen.c |   17 +++++++++++++++++
> 1 files changed, 17 insertions(+), 0 deletions(-)
>
>diff --git a/src/mesa/drivers/dri/radeon/radeon_screen.c b/src/mesa/drivers/dri/radeon/radeon_screen.c
>index 82107cc..d3d7b21 100644
>--- a/src/mesa/drivers/dri/radeon/radeon_screen.c
>+++ b/src/mesa/drivers/dri/radeon/radeon_screen.c
>@@ -376,6 +376,21 @@ static const __DRItexBufferExtension r600TexBufferExtension = {
> };
> #endif
> 
>+static void
>+radeonDRI2Flush(__DRIdrawable *drawable)
>+{
>+    radeonContextPtr rmesa;
>+
>+    rmesa = (radeonContextPtr) drawable->driContextPriv->driverPrivate;
>+    radeonFlush(rmesa->glCtx);
>+}
>+
>+static const struct __DRI2flushExtensionRec radeonFlushExtension = {
>+    { __DRI2_FLUSH, __DRI2_FLUSH_VERSION },
>+    radeonDRI2Flush,
>+    dri2InvalidateDrawable,
>+};
>+
> static int radeon_set_screen_flags(radeonScreenPtr screen, int device_id)
> {
>    screen->device_id = device_id;
>@@ -1379,6 +1394,8 @@ radeonCreateScreen2(__DRIscreen *sPriv)
>    screen->extensions[i++] = &r600TexBufferExtension.base;
> #endif
> 
>+   screen->extensions[i++] = &radeonFlushExtension.base;
>+
>    screen->extensions[i++] = NULL;
>    sPriv->extensions = screen->extensions;
> 
>-- 
>1.7.0.4
>

Actions: View | Splinter Review

Attachments on bug 28341: 35995 | 35997 | 37035 | 37317 | 37366 | 37368 | 37401 | 37414 | 37513

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.