Refactor PBO validate/map code.

We always need to do PBO validation, so do that in core Mesa before calling driver routine.

3 files changed, 7 insertions, 12 deletions

diff --git a/src/mesa/swrast/s_bitmap.c b/src/mesa/swrast/s_bitmap.c
index 17f639fd55..f3dda12e25 100644
--- a/src/mesa/swrast/s_bitmap.c
+++ b/src/mesa/swrast/s_bitmap.c
@@ -57,11 +57,9 @@ _swrast_Bitmap( GLcontext *ctx, GLint px, GLint py,
 
    ASSERT(ctx->RenderMode == GL_RENDER);
 
-   bitmap = _mesa_validate_and_map_bitmap_pbo(ctx, width, height,
-                                              unpack, bitmap);
-   if (!bitmap) {
-      return NULL;
-   }
+   bitmap = _mesa_map_bitmap_pbo(ctx, unpack, bitmap);
+   if (!bitmap)
+      return;
 
    RENDER_START(swrast,ctx);
 
diff --git a/src/mesa/swrast/s_drawpix.c b/src/mesa/swrast/s_drawpix.c
index 09ff96fd32..81f5caa270 100644
--- a/src/mesa/swrast/s_drawpix.c
+++ b/src/mesa/swrast/s_drawpix.c
@@ -839,8 +839,7 @@ _swrast_DrawPixels( GLcontext *ctx,
    if (swrast->NewState)
       _swrast_validate_derived( ctx );
 
-    pixels = _mesa_validate_and_map_drawpix_pbo(ctx, width, height,
-                                                format, type, unpack, pixels);
+    pixels = _mesa_map_drawpix_pbo(ctx, unpack, pixels);
     if (!pixels)
        return;
 
diff --git a/src/mesa/swrast/s_readpix.c b/src/mesa/swrast/s_readpix.c
index 050506b0d0..9140d12ea0 100644
--- a/src/mesa/swrast/s_readpix.c
+++ b/src/mesa/swrast/s_readpix.c
@@ -574,11 +574,9 @@ _swrast_ReadPixels( GLcontext *ctx,
       return;
    }
 
-    pixels = _mesa_validate_and_map_readpix_pbo(ctx, x, y, width, height,
-                                                format, type,
-                                                &clippedPacking, pixels);
-    if (!pixels)
-       return;
+   pixels = _mesa_map_readpix_pbo(ctx, &clippedPacking, pixels);
+   if (!pixels)
+      return;
   
    switch (format) {
       case GL_COLOR_INDEX: