Mesa (master): Revert "intel: Fix the client-side swapbuffers throttling."

Eric Anholt anholt at kemper.freedesktop.org
Tue Nov 9 22:46:45 UTC 2010


Module: Mesa
Branch: master
Commit: 00391c79416ffd4b6f3af0ad911b7b8210f746d1
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=00391c79416ffd4b6f3af0ad911b7b8210f746d1

Author: Eric Anholt <eric at anholt.net>
Date:   Tue Nov  9 14:01:52 2010 -0800

Revert "intel: Fix the client-side swapbuffers throttling."

This reverts commit 76360d6abc9e0195bc5c373101ae616e68b2e6e6.  On
second thought, it turned out that sync objects also used the
wait_rendering API like this, and would need the same treatment, and
so wait_rendering itself is fixed in libdrm now.

---

 src/mesa/drivers/dri/intel/intel_context.c |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/src/mesa/drivers/dri/intel/intel_context.c b/src/mesa/drivers/dri/intel/intel_context.c
index 60c401b..de4bcde 100644
--- a/src/mesa/drivers/dri/intel/intel_context.c
+++ b/src/mesa/drivers/dri/intel/intel_context.c
@@ -470,11 +470,7 @@ intel_prepare_render(struct intel_context *intel)
     * so we just us the first batch we emitted after the last swap.
     */
    if (intel->need_throttle && intel->first_post_swapbuffers_batch) {
-      /* Can't use drm_intel_bo_wait_rendering, since that only waits
-       * for rendering *to* the buffer, not from it.
-       */
-      drm_intel_gem_bo_map_gtt(intel->first_post_swapbuffers_batch);
-      drm_intel_gem_bo_unmap_gtt(intel->first_post_swapbuffers_batch);
+      drm_intel_bo_wait_rendering(intel->first_post_swapbuffers_batch);
       drm_intel_bo_unreference(intel->first_post_swapbuffers_batch);
       intel->first_post_swapbuffers_batch = NULL;
       intel->need_throttle = GL_FALSE;




More information about the mesa-commit mailing list