From: Marek Olšák <maraeo@gmail.com>
Date: Fri, 31 May 2013 19:52:01 +0000 (+0200)
Subject: st/dri: always copy new DRI front and back buffers to corresponding MSAA buffers
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=b77316ad7594fa5873717992b6986cb4c0179d23;p=mesa.git

st/dri: always copy new DRI front and back buffers to corresponding MSAA buffers

This commit fixes these piglit tests with an MSAA visual forced on:
- read-front
- glx-copy-sub-buffer

Reviewed-by: Brian Paul <brianp@vmware.com>
---

diff --git a/src/gallium/state_trackers/dri/drm/dri2.c b/src/gallium/state_trackers/dri/drm/dri2.c
index 50628a618e7..a0b29d3a540 100644
--- a/src/gallium/state_trackers/dri/drm/dri2.c
+++ b/src/gallium/state_trackers/dri/drm/dri2.c
@@ -180,6 +180,8 @@ dri2_drawable_process_buffers(struct dri_drawable *drawable,
    struct winsys_handle whandle;
    boolean alloc_depthstencil = FALSE;
    unsigned i, j, bind;
+   struct pipe_screen *pscreen = screen->base.screen;
+   struct pipe_context *pipe = NULL;
 
    if (drawable->old_num == buffer_count &&
        drawable->old_w == dri_drawable->w &&
@@ -296,6 +298,26 @@ dri2_drawable_process_buffers(struct dri_drawable *drawable,
                   screen->base.screen->resource_create(screen->base.screen,
                                                        &templ);
                assert(drawable->msaa_textures[att]);
+
+               /* If there are any MSAA resources, we should initialize them
+                * such that they contain the same data as the single-sample
+                * resources we just got from the X server.
+                *
+                * The reason for this is that the state tracker (and
+                * therefore the app) can access the MSAA resources only.
+                * The single-sample resources are not exposed
+                * to the state tracker.
+                *
+                * We don't have a context here, so create one temporarily.
+                * We may need to create a persistent context if creation and
+                * destruction of the context becomes a bottleneck.
+                */
+               if (!pipe)
+                  pipe = pscreen->context_create(pscreen, NULL);
+
+               dri_pipe_blit(pipe,
+                             drawable->msaa_textures[att],
+                             drawable->textures[att]);
             }
          }
          else {
@@ -349,6 +371,11 @@ dri2_drawable_process_buffers(struct dri_drawable *drawable,
    drawable->old_w = dri_drawable->w;
    drawable->old_h = dri_drawable->h;
    memcpy(drawable->old, buffers, sizeof(__DRIbuffer) * buffer_count);
+
+   if (pipe) {
+      pipe->flush(pipe, NULL, 0);
+      pipe->destroy(pipe);
+   }
 }
 
 static __DRIbuffer *