i915: Restore the accelerated PBO pixel path functions after GEM changes.
authorEric Anholt <eric@anholt.net>
Wed, 18 Jun 2008 21:19:18 +0000 (14:19 -0700)
committerEric Anholt <eric@anholt.net>
Wed, 18 Jun 2008 21:19:18 +0000 (14:19 -0700)
The fencing code is not required, and waiting on the fences defeated one of
the purposes of the extension, which is to allow asynchronous readpixels.

src/mesa/drivers/dri/i915/Makefile
src/mesa/drivers/dri/i915/i830_context.c
src/mesa/drivers/dri/i915/i915_context.c
src/mesa/drivers/dri/i915/intel_pixel_read.c
src/mesa/drivers/dri/intel/intel_pixel_draw.c

index 74f6169b2eaa59b98d66b9fd9b41302143f03d05..ccba34d229398cf05ee5a799c99b76ee54774c05 100644 (file)
@@ -6,11 +6,6 @@ LIBNAME = i915_dri.so
 
 MINIGLX_SOURCES = server/intel_dri.c
 
-PIXEL_SOURCES = \
-       intel_pixel.c \
-       intel_pixel_read.c \
-       intel_pixel_draw.c
-
 DRIVER_SOURCES = \
        i830_context.c \
        i830_metaops.c \
@@ -48,6 +43,9 @@ DRIVER_SOURCES = \
        intel_context.c \
        intel_decode.c \
        intel_ioctl.c \
+       intel_pixel.c \
+       intel_pixel_draw.c \
+       intel_pixel_read.c \
        intel_screen.c \
        intel_span.c \
        intel_state.c \
index d6542f8b3a25cc8d811000362c1402cc2e187b07..66f1566f16b264f6e2e5c139a00c0683adb9fecd 100644 (file)
@@ -52,7 +52,7 @@ static void
 i830InitDriverFunctions(struct dd_function_table *functions)
 {
    intelInitDriverFunctions(functions);
-//   intelInitPixelFuncs(functions);
+   intelInitPixelFuncs(functions);
    i830InitStateFuncs(functions);
    i830InitTextureFuncs(functions);
 }
index bb77cff96caef2aef7eb219ec1a09f0114eff3a7..1128f497dbde3e5fd17580bf8d0c6f0f122c513c 100644 (file)
@@ -94,7 +94,7 @@ static void
 i915InitDriverFunctions(struct dd_function_table *functions)
 {
    intelInitDriverFunctions(functions);
-//   intelInitPixelFuncs(functions);
+   intelInitPixelFuncs(functions);
    i915InitStateFunctions(functions);
    i915InitTextureFuncs(functions);
    i915InitFragProgFuncs(functions);
index 2e31656e984080fe6a3a7c28caa0b9283ade9993..0b95421a25fbe2695e34e884050615f04a517489 100644 (file)
@@ -173,7 +173,6 @@ do_blit_readpixels(GLcontext * ctx,
    struct intel_buffer_object *dst = intel_buffer_object(pack->BufferObj);
    GLuint dst_offset;
    GLuint rowLength;
-   dri_fence *fence = NULL;
 
    if (INTEL_DEBUG & DEBUG_PIXEL)
       _mesa_printf("%s\n", __FUNCTION__);
@@ -275,17 +274,9 @@ do_blit_readpixels(GLcontext * ctx,
       }
 
       intel_batchbuffer_flush(intel->batch);
-      fence = intel->batch->last_fence;
-      dri_fence_reference(fence);
-
    }
    UNLOCK_HARDWARE(intel);
 
-   if (fence) {
-      dri_fence_wait(fence);
-      dri_fence_unreference(fence);
-   }
-
    if (INTEL_DEBUG & DEBUG_PIXEL)
       _mesa_printf("%s - DONE\n", __FUNCTION__);
 
index 2804c8deeaf6faab6a7037a419091f6ba0959228..f7082f299e101c28ccce61e726c683d5bd629f19 100644 (file)
@@ -218,7 +218,6 @@ do_blit_drawpixels(GLcontext * ctx,
    struct intel_buffer_object *src = intel_buffer_object(unpack->BufferObj);
    GLuint src_offset;
    GLuint rowLength;
-   dri_fence *fence = NULL;
 
    if (INTEL_DEBUG & DEBUG_PIXEL)
       _mesa_printf("%s\n", __FUNCTION__);
@@ -323,16 +322,9 @@ do_blit_drawpixels(GLcontext * ctx,
                           ctx->Color.LogicOp : GL_COPY);
       }
       intel_batchbuffer_flush(intel->batch);
-      fence = intel->batch->last_fence;
-      dri_fence_reference(fence);
    }
    UNLOCK_HARDWARE(intel);
 
-   if (fence) {
-      dri_fence_wait(fence);
-      dri_fence_unreference(fence);
-   }
-
    if (INTEL_DEBUG & DEBUG_PIXEL)
       _mesa_printf("%s - DONE\n", __FUNCTION__);