Add a new interface between softpipe and the window system / buffer manager.

This interface is defined by softpipe and any window system (eg i915pipe)
wishing to use softpipe is required to implement the interface.
Currently the interface is all about buffer management.

Generalizing, each pipe driver will advertise an interface in a similar
spirit to this, and again any window system driver wishing to use that
rendering pipeline will have to implement the interface it defines.  It
clearly isn't a one-way street however, as softpipe could just do its own
buffer management with malloc.  The interaction with a buffer manager is
desired to allow us to exercise the hardware swapbuffers functionality of
the i915pipe driver, and also to get a feel for the way hardware drivers
which really need a buffer manager will work.

7 files changed, 299 insertions, 116 deletions

diff --git a/src/mesa/pipe/p_context.h b/src/mesa/pipe/p_context.h
index ac6567f009..92670c7733 100644
--- a/src/mesa/pipe/p_context.h
+++ b/src/mesa/pipe/p_context.h
@@ -32,10 +32,6 @@
 #include "p_state.h"
 
 
-/* Kludge:
- */
-extern struct pipe_context *softpipe_create( void );
-				      
 /* Drawing currently kludged up via the existing tnl/ module.  
  */
 struct vertex_buffer;
@@ -161,7 +157,8 @@ struct pipe_context {
                        struct pipe_region *dest,
                        GLuint dest_offset,
                        GLuint destx, GLuint desty,
-                       const struct pipe_region *src,
+                       struct pipe_region *src,	/* don't make this const - 
+						   need to map/unmap */
                        GLuint src_offset,
                        GLuint srcx, GLuint srcy, GLuint width, GLuint height);
 
@@ -170,14 +167,11 @@ struct pipe_context {
                        GLuint dst_offset,
                        GLuint dstx, GLuint dsty,
                        GLuint width, GLuint height,
-                       GLuint value, GLuint mask);
+                       GLuint value);
 
    struct _DriBufferObject *(*region_buffer)(struct pipe_context *pipe,
                                              struct pipe_region *region,
                                              GLuint flag);
-
-   void *screen;  /**< temporary */
-   void *glctx;   /**< temporary */
 };
 
 
diff --git a/src/mesa/pipe/p_state.h b/src/mesa/pipe/p_state.h
index 699efd5877..3be1010007 100644
--- a/src/mesa/pipe/p_state.h
+++ b/src/mesa/pipe/p_state.h
@@ -235,12 +235,9 @@ struct pipe_sampler_state
  *** Resource Objects
  ***/
 
-
-struct _DriBufferObject;
-
 struct pipe_region
 {
-   struct _DriBufferObject *buffer;   /**< buffer manager's buffer ID */
+   void *buffer;    /**< driver private buffer handle */
 
    GLuint refcount; /**< Reference count for region */
    GLuint cpp;      /**< bytes per pixel */
diff --git a/src/mesa/pipe/softpipe/sp_clear.c b/src/mesa/pipe/softpipe/sp_clear.c
index d7684d2044..2d69b6e0dc 100644
--- a/src/mesa/pipe/softpipe/sp_clear.c
+++ b/src/mesa/pipe/softpipe/sp_clear.c
@@ -56,5 +56,5 @@ softpipe_clear(struct pipe_context *pipe, struct pipe_surface *ps,
    w = softpipe->framebuffer.cbufs[0]->width;
    h = softpipe->framebuffer.cbufs[0]->height;
 
-   pipe->region_fill(pipe, ps->region, 0, x, y, w, h, clearValue, ~0);
+   pipe->region_fill(pipe, ps->region, 0, x, y, w, h, clearValue);
 }
diff --git a/src/mesa/pipe/softpipe/sp_context.c b/src/mesa/pipe/softpipe/sp_context.c
index fe7f2f351a..22928f7bd6 100644
--- a/src/mesa/pipe/softpipe/sp_context.c
+++ b/src/mesa/pipe/softpipe/sp_context.c
@@ -39,6 +39,7 @@
 #include "sp_state.h"
 #include "sp_surface.h"
 #include "sp_prim_setup.h"
+#include "sp_winsys.h"
 
 
 
@@ -168,7 +169,7 @@ static GLuint softpipe_get_occlusion_counter(struct pipe_context *pipe)
 }
 
 
-struct pipe_context *softpipe_create( void )
+struct pipe_context *softpipe_create( struct softpipe_winsys *sws )
 {
    struct softpipe_context *softpipe = CALLOC_STRUCT(softpipe_context);
 
@@ -209,6 +210,8 @@ struct pipe_context *softpipe_create( void )
    softpipe->quad.colormask = sp_quad_colormask_stage(softpipe);
    softpipe->quad.output = sp_quad_output_stage(softpipe);
 
+   softpipe->winsys = sws;
+
    /*
     * Create drawing context and plug our rendering stage into it.
     */
diff --git a/src/mesa/pipe/softpipe/sp_context.h b/src/mesa/pipe/softpipe/sp_context.h
index 3c379c9091..43fa8acd76 100644
--- a/src/mesa/pipe/softpipe/sp_context.h
+++ b/src/mesa/pipe/softpipe/sp_context.h
@@ -38,8 +38,8 @@
 
 #include "sp_quad.h"
 
-
 struct softpipe_surface;
+struct softpipe_winsys;
 struct draw_context;
 struct draw_stage;
 
@@ -68,6 +68,8 @@ enum interp_mode {
 
 struct softpipe_context {     
    struct pipe_context pipe;  /**< base class */
+   struct softpipe_winsys *winsys;	/**< window system interface */
+
 
    /* The most recent drawing state as set by the driver:
     */
diff --git a/src/mesa/pipe/softpipe/sp_region.c b/src/mesa/pipe/softpipe/sp_region.c
index 1dbd1609e3..78a0919ad5 100644
--- a/src/mesa/pipe/softpipe/sp_region.c
+++ b/src/mesa/pipe/softpipe/sp_region.c
@@ -1,6 +1,6 @@
 /**************************************************************************
  * 
- * Copyright 2007 Tungsten Graphics, Inc., Cedar Park, Texas.
+ * Copyright 2006 Tungsten Graphics, Inc., Cedar Park, Texas.
  * All Rights Reserved.
  * 
  * Permission is hereby granted, free of charge, to any person obtaining a
@@ -25,74 +25,185 @@
  * 
  **************************************************************************/
 
-/**
- * Software-based region allocation and management.
- * A hardware driver would override these functions.
+/* Provide additional functionality on top of bufmgr buffers:
+ *   - 2d semantics and blit operations (XXX: remove/simplify blits??)
+ *   - refcounting of buffers for multiple images in a buffer.
+ *   - refcounting of buffer mappings.
  */
 
-
 #include "sp_context.h"
+#include "sp_winsys.h"
 #include "sp_region.h"
-#include "sp_surface.h"
-#include "main/imports.h"
 
 
+static void
+sp_region_idle(struct pipe_context *pipe, struct pipe_region *region)
+{
+   
+}
+
+
+static GLubyte *
+sp_region_map(struct pipe_context *pipe, struct pipe_region *region)
+{
+   struct softpipe_context *sp = softpipe_context( pipe );
+
+   if (!region->map_refcount++) {
+      region->map = sp->winsys->buffer_map( sp->winsys,
+					    region->buffer );
+   }
+
+   return region->map;
+}
+
+static void
+sp_region_unmap(struct pipe_context *pipe, struct pipe_region *region)
+{
+   struct softpipe_context *sp = softpipe_context( pipe );
+
+   if (!--region->map_refcount) {
+      sp->winsys->buffer_unmap( sp->winsys,
+				region->buffer );
+      region->map = NULL;
+   }
+}
+
 static struct pipe_region *
 sp_region_alloc(struct pipe_context *pipe,
-                GLuint cpp, GLuint pitch, GLuint height)
+		GLuint cpp, GLuint pitch, GLuint height)
 {
-   struct pipe_region *region = CALLOC_STRUCT(pipe_region);
-   if (!region)
-      return NULL;
+   struct softpipe_context *sp = softpipe_context( pipe );
+   struct pipe_region *region = calloc(sizeof(*region), 1);
 
-   region->refcount = 1;
    region->cpp = cpp;
    region->pitch = pitch;
-   region->height = height;
-   region->map = malloc(cpp * pitch * height);
+   region->height = height;     /* needed? */
+   region->refcount = 1;
+
+   region->buffer = sp->winsys->create_buffer(sp->winsys, "region", 64 );
+
+   sp->winsys->buffer_data( sp->winsys,
+			    region->buffer, 
+			    pitch * cpp * height, 
+			    NULL );
 
    return region;
 }
 
-
 static void
 sp_region_release(struct pipe_context *pipe, struct pipe_region **region)
 {
-   assert((*region)->refcount > 0);
+   struct softpipe_context *sp = softpipe_context( pipe );
+
+   if (!*region)
+      return;
+
+   ASSERT((*region)->refcount > 0);
    (*region)->refcount--;
 
    if ((*region)->refcount == 0) {
       assert((*region)->map_refcount == 0);
 
-#if 0
-      if ((*region)->pbo)
-	 (*region)->pbo->region = NULL;
-      (*region)->pbo = NULL;
-#endif
-
+      sp->winsys->buffer_unreference( sp->winsys,
+				      (*region)->buffer );
       free(*region);
    }
    *region = NULL;
 }
 
 
-
-static GLubyte *
-sp_region_map(struct pipe_context *pipe, struct pipe_region *region)
+/*
+ * XXX Move this into core Mesa?
+ */
+static void
+_mesa_copy_rect(GLubyte * dst,
+                GLuint cpp,
+                GLuint dst_pitch,
+                GLuint dst_x,
+                GLuint dst_y,
+                GLuint width,
+                GLuint height,
+                const GLubyte * src,
+                GLuint src_pitch,
+		GLuint src_x, 
+		GLuint src_y)
 {
-   region->map_refcount++;
-   return region->map;
+   GLuint i;
+
+   dst_pitch *= cpp;
+   src_pitch *= cpp;
+   dst += dst_x * cpp;
+   src += src_x * cpp;
+   dst += dst_y * dst_pitch;
+   src += src_y * dst_pitch;
+   width *= cpp;
+
+   if (width == dst_pitch && width == src_pitch)
+      memcpy(dst, src, height * width);
+   else {
+      for (i = 0; i < height; i++) {
+         memcpy(dst, src, width);
+         dst += dst_pitch;
+         src += src_pitch;
+      }
+   }
 }
 
 
+/* Upload data to a rectangular sub-region.  Lots of choices how to do this:
+ *
+ * - memcpy by span to current destination
+ * - upload data as new buffer and blit
+ *
+ * Currently always memcpy.
+ */
 static void
-sp_region_unmap(struct pipe_context *pipe, struct pipe_region *region)
+sp_region_data(struct pipe_context *pipe,
+	       struct pipe_region *dst,
+	       GLuint dst_offset,
+	       GLuint dstx, GLuint dsty,
+	       const void *src, GLuint src_pitch,
+	       GLuint srcx, GLuint srcy, GLuint width, GLuint height)
 {
-   region->map_refcount--;
+   _mesa_copy_rect(pipe->region_map(pipe, dst) + dst_offset,
+                   dst->cpp,
+                   dst->pitch,
+                   dstx, dsty, width, height, src, src_pitch, srcx, srcy);
+
+   pipe->region_unmap(pipe, dst);
 }
 
+/* Assumes all values are within bounds -- no checking at this level -
+ * do it higher up if required.
+ */
+static void
+sp_region_copy(struct pipe_context *pipe,
+	       struct pipe_region *dst,
+	       GLuint dst_offset,
+	       GLuint dstx, GLuint dsty,
+	       struct pipe_region *src,
+	       GLuint src_offset,
+	       GLuint srcx, GLuint srcy, GLuint width, GLuint height)
+{
+   assert( dst != src );
+   assert( dst->cpp == src->cpp );
 
+   _mesa_copy_rect(pipe->region_map(pipe, dst) + dst_offset,
+                   dst->cpp,
+                   dst->pitch,
+                   dstx, dsty, 
+		   width, height, 
+		   pipe->region_map(pipe, src) + src_offset, 
+		   src->pitch, 
+		   srcx, srcy);
 
+   pipe->region_unmap(pipe, src);
+   pipe->region_unmap(pipe, dst);
+}
+
+/* Fill a rectangular sub-region.  Need better logic about when to
+ * push buffers into AGP - will currently do so whenever possible.
+ */
 static GLubyte *
 get_pointer(struct pipe_region *dst, GLuint x, GLuint y)
 {
@@ -105,93 +216,69 @@ sp_region_fill(struct pipe_context *pipe,
                struct pipe_region *dst,
                GLuint dst_offset,
                GLuint dstx, GLuint dsty,
-               GLuint width, GLuint height, GLuint value, GLuint mask)
+               GLuint width, GLuint height, GLuint value)
 {
    GLuint i, j;
+
+   (void)pipe->region_map(pipe, dst);
+
    switch (dst->cpp) {
-   case 1:
-      {
-         GLubyte *row = get_pointer(dst, dstx, dsty);
-         if ((mask & 0xff) == 0xff) {
-            /* no masking */
-            for (i = 0; i < height; i++) {
-               memset(row, value, width);
-               row += dst->pitch;
-            }
-         }
-         else {
-            value &= mask;
-            mask = ~mask;
-            for (i = 0; i < height; i++) {
-               for (j = 0; j < width; j++) {
-                  row[j] = (row[j] & mask) | value;
-               }
-               row += dst->pitch;
-            }
-         }
+   case 1: {
+      GLubyte *row = get_pointer(dst, dstx, dsty);
+      for (i = 0; i < height; i++) {
+	 memset(row, value, width);
+	 row += dst->pitch;
       }
-      break;
-   case 2:
-      {
-         GLushort *row = (GLushort *) get_pointer(dst, dstx, dsty);
-         if ((mask & 0xffff) == 0xffff) {
-            /* no masking */
-            for (i = 0; i < height; i++) {
-               for (j = 0; j < width; j++)
-                  row[j] = value;
-               row += dst->pitch;
-            }
-         }
-         else {
-            value &= mask;
-            mask = ~mask;
-            for (i = 0; i < height; i++) {
-               for (j = 0; j < width; j++)
-                  row[j] = (row[j] & mask) | value;
-               row += dst->pitch;
-            }
-         }
+   }
+   break;
+   case 2: {
+      GLushort *row = (GLushort *) get_pointer(dst, dstx, dsty);
+      for (i = 0; i < height; i++) {
+	 for (j = 0; j < width; j++)
+	    row[j] = value;
+	 row += dst->pitch;
       }
-      break;
-   case 4:
-      {
-         GLuint *row = (GLuint *) get_pointer(dst, dstx, dsty);
-         if (mask == 0xffffffff) {
-            /* no masking */
-            for (i = 0; i < height; i++) {
-               for (j = 0; j < width; j++)
-                  row[j] = value;
-               row += dst->pitch;
-            }
-         }
-         else {
-            value &= mask;
-            mask = ~mask;
-            for (i = 0; i < height; i++) {
-               for (j = 0; j < width; j++)
-                  row[j] = (row[j] & mask) | value;
-               row += dst->pitch;
-            }
-         }
+   }
+   break;
+   case 4: {
+      GLuint *row = (GLuint *) get_pointer(dst, dstx, dsty);
+      for (i = 0; i < height; i++) {
+	 for (j = 0; j < width; j++)
+	    row[j] = value;
+	 row += dst->pitch;
       }
-      break;
+   }
+   break;
    default:
       assert(0);
-
+      break;
    }
+
+   pipe->region_unmap( pipe, dst );
+}
+
+
+
+static struct _DriBufferObject *
+sp_region_buffer(struct pipe_context *pipe,
+                    struct pipe_region *region, GLuint flag)
+{
+   return region->buffer;
 }
 
 
+
 void
 sp_init_region_functions(struct softpipe_context *sp)
 {
-   sp->pipe.region_alloc = sp_region_alloc;
-   sp->pipe.region_release = sp_region_release;
-
+   sp->pipe.region_idle = sp_region_idle;
    sp->pipe.region_map = sp_region_map;
    sp->pipe.region_unmap = sp_region_unmap;
-
+   sp->pipe.region_alloc = sp_region_alloc;
+   sp->pipe.region_release = sp_region_release;
+   sp->pipe.region_data = sp_region_data;
+   sp->pipe.region_copy = sp_region_copy;
    sp->pipe.region_fill = sp_region_fill;
-
-   /* XXX lots of other region functions needed... */
+   sp->pipe.region_buffer = sp_region_buffer;
 }
+
diff --git a/src/mesa/pipe/softpipe/sp_winsys.h b/src/mesa/pipe/softpipe/sp_winsys.h
new file mode 100644
index 0000000000..43953c648b
--- /dev/null
+++ b/src/mesa/pipe/softpipe/sp_winsys.h
@@ -0,0 +1,100 @@
+/**************************************************************************
+ * 
+ * Copyright 2007 Tungsten Graphics, Inc., Cedar Park, Texas.
+ * 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
+ * without limitation the rights to use, copy, modify, merge, publish,
+ * 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.
+ * IN NO EVENT SHALL TUNGSTEN GRAPHICS AND/OR ITS SUPPLIERS BE LIABLE FOR
+ * 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.
+ * 
+ **************************************************************************/
+
+#ifndef SP_WINSYS_H
+#define SP_WINSYS_H
+
+#include "main/mtypes.h"
+
+/* This is the interface that softpipe requires any window system
+ * hosting it to implement.  This is the only include file in softpipe
+ * which is public.
+ */
+
+
+/* Pipe drivers are (meant to be!) independent of both GL and the
+ * window system.  The window system provides a buffer manager and a
+ * set of additional hooks for things like command buffer submission,
+ * etc.
+ *
+ * There clearly has to be some agreement between the window system
+ * driver and the hardware driver about the format of command buffers,
+ * etc.
+ */
+
+struct softpipe_buffer_handle;
+
+struct softpipe_winsys {
+
+
+   /* The buffer manager is modeled after the dri_bugmgr interface,
+    * but this is the subset that softpipe cares about.  Remember that
+    * softpipe gets to choose the interface it needs, and the window
+    * systems must then implement that interface (rather than the
+    * other way around...).
+    *
+    * Softpipe only really wants to make system memory allocations,
+    * right?? 
+    */
+   struct softpipe_buffer_handle *(*create_buffer)(struct softpipe_winsys *sws, 
+						   const char *name,
+						   unsigned alignment );
+
+   void *(*buffer_map)( struct softpipe_winsys *sws, 
+			struct softpipe_buffer_handle *buf );
+   
+   void (*buffer_unmap)( struct softpipe_winsys *sws, 
+			 struct softpipe_buffer_handle *buf );
+
+   struct softpipe_buffer_handle *(*buffer_reference)( struct softpipe_winsys *sws,
+						       struct softpipe_buffer_handle *buf );
+
+   void (*buffer_unreference)( struct softpipe_winsys *sws, 
+			       struct softpipe_buffer_handle *buf );
+
+   void (*buffer_data)(struct softpipe_winsys *sws, 
+		       struct softpipe_buffer_handle *buf,
+		       unsigned size, const void *data );
+
+   void (*buffer_subdata)(struct softpipe_winsys *sws, 
+			  struct softpipe_buffer_handle *buf,
+			  unsigned long offset, 
+			  unsigned long size, 
+			  const void *data);
+
+   void (*buffer_get_subdata)(struct softpipe_winsys *sws, 
+			      struct softpipe_buffer_handle *buf,
+			      unsigned long offset, 
+			      unsigned long size, 
+			      void *data);
+};
+
+
+struct pipe_context *softpipe_create( struct softpipe_winsys * );
+
+
+#endif /* SP_WINSYS_H */