replace _mesa_logbase2 with util_logbase2
[mesa.git] / src / mesa / state_tracker / st_cb_flush.c
index 8375308c3e24c1e4dfaaebd28e9130c55eafb309..08b645433ac40d15a33434b904e784285ea322f5 100644 (file)
@@ -1,8 +1,8 @@
 /**************************************************************************
- * 
+ *
  * Copyright 2007 VMware, Inc.
  * All Rights Reserved.
- * 
+ *
  * Permission is hereby granted, free of charge, to any person obtaining a
  * copy of this software and associated documentation files (the
  * "Software"), to deal in the Software without restriction, including
  * distribute, sub license, and/or sell copies of the Software, and to
  * permit persons to whom the Software is furnished to do so, subject to
  * the following conditions:
- * 
+ *
  * The above copyright notice and this permission notice (including the
  * next paragraph) shall be included in all copies or substantial portions
  * of the Software.
- * 
+ *
  * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
  * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
  * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT.
@@ -22,7 +22,7 @@
  * ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
  * TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
  * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
- * 
+ *
  **************************************************************************/
 
  /*
@@ -39,6 +39,7 @@
 #include "st_cb_flush.h"
 #include "st_cb_clear.h"
 #include "st_cb_fbo.h"
+#include "st_context.h"
 #include "st_manager.h"
 #include "pipe/p_context.h"
 #include "pipe/p_defines.h"
 #include "util/u_gen_mipmap.h"
 
 
-void st_flush(struct st_context *st,
-              struct pipe_fence_handle **fence,
-              unsigned flags)
+void
+st_flush(struct st_context *st,
+         struct pipe_fence_handle **fence,
+         unsigned flags)
 {
-   FLUSH_VERTICES(st->ctx, 0);
-   FLUSH_CURRENT(st->ctx, 0);
-
-   st_flush_bitmap_cache(st);
+   /* We want to call this function periodically.
+    * Typically, it has nothing to do so it shouldn't be expensive.
+    */
+   st_context_free_zombie_objects(st);
 
    st->pipe->flush(st->pipe, fence, flags);
 }
@@ -62,17 +64,21 @@ void st_flush(struct st_context *st,
 /**
  * Flush, and wait for completion.
  */
-void st_finish( struct st_context *st )
+void
+st_finish(struct st_context *st)
 {
    struct pipe_fence_handle *fence = NULL;
 
-   st_flush(st, &fence, 0);
+   st_flush_bitmap_cache(st);
+   st_flush(st, &fence, PIPE_FLUSH_ASYNC | PIPE_FLUSH_HINT_FINISH);
 
-   if(fence) {
+   if (fence) {
       st->pipe->screen->fence_finish(st->pipe->screen, NULL, fence,
                                      PIPE_TIMEOUT_INFINITE);
       st->pipe->screen->fence_reference(st->pipe->screen, &fence, NULL);
    }
+
+   st_manager_flush_swapbuffers();
 }
 
 
@@ -80,10 +86,13 @@ void st_finish( struct st_context *st )
 /**
  * Called via ctx->Driver.Flush()
  */
-static void st_glFlush(struct gl_context *ctx)
+static void
+st_glFlush(struct gl_context *ctx)
 {
    struct st_context *st = st_context(ctx);
 
+   st_flush_bitmap_cache(st);
+
    /* Don't call st_finish() here.  It is not the state tracker's
     * responsibilty to inject sleeps in the hope of avoiding buffer
     * synchronization issues.  Calling finish() here will just hide
@@ -98,7 +107,8 @@ static void st_glFlush(struct gl_context *ctx)
 /**
  * Called via ctx->Driver.Finish()
  */
-static void st_glFinish(struct gl_context *ctx)
+static void
+st_glFinish(struct gl_context *ctx)
 {
    struct st_context *st = st_context(ctx);
 
@@ -127,6 +137,18 @@ gl_reset_status_from_pipe_reset_status(enum pipe_reset_status status)
 }
 
 
+static void
+st_device_reset_callback(void *data, enum pipe_reset_status status)
+{
+   struct st_context *st = data;
+
+   assert(status != PIPE_NO_RESET);
+
+   st->reset_status = status;
+   _mesa_set_context_lost_dispatch(st->ctx);
+}
+
+
 /**
  * Query information about GPU resets observed by this context
  *
@@ -143,24 +165,14 @@ st_get_graphics_reset_status(struct gl_context *ctx)
       st->reset_status = PIPE_NO_RESET;
    } else {
       status = st->pipe->get_device_reset_status(st->pipe);
+      if (status != PIPE_NO_RESET)
+         st_device_reset_callback(st, status);
    }
 
    return gl_reset_status_from_pipe_reset_status(status);
 }
 
 
-static void
-st_device_reset_callback(void *data, enum pipe_reset_status status)
-{
-   struct st_context *st = data;
-
-   assert(status != PIPE_NO_RESET);
-
-   st->reset_status = status;
-   _mesa_set_context_lost_dispatch(st->ctx);
-}
-
-
 void
 st_install_device_reset_callback(struct st_context *st)
 {
@@ -173,24 +185,13 @@ st_install_device_reset_callback(struct st_context *st)
 }
 
 
-void st_init_flush_functions(struct pipe_screen *screen,
-                             struct dd_function_table *functions)
+void
+st_init_flush_functions(struct pipe_screen *screen,
+                        struct dd_function_table *functions)
 {
    functions->Flush = st_glFlush;
    functions->Finish = st_glFinish;
 
    if (screen->get_param(screen, PIPE_CAP_DEVICE_RESET_STATUS_QUERY))
       functions->GetGraphicsResetStatus = st_get_graphics_reset_status;
-
-   /* Windows opengl32.dll calls glFinish prior to every swapbuffers.
-    * This is unnecessary and degrades performance.  Luckily we have some
-    * scope to work around this, as the externally-visible behaviour of
-    * Finish() is identical to Flush() in all cases - no differences in
-    * rendering or ReadPixels are visible if we opt not to wait here.
-    *
-    * Only set this up on Windows to avoid surprise elsewhere.
-    */
-#ifdef PIPE_OS_WINDOWS
-   functions->Finish = st_glFlush;
-#endif
 }