[Mesa-dev] [PATCH 1/2] winsys: Correct Haiku winsys display target code

[Alexander von Gluck IV]

* Instead of assuming the displaytarget is the same
  stride / colorspace as the destination, lets
  actually check the source bitmap.
* Fixes random stride issues in rendering
---
 src/gallium/winsys/sw/hgl/bitmap_wrapper.cpp |   20 ++++++++++++++++----
 src/gallium/winsys/sw/hgl/bitmap_wrapper.h   |    5 ++++-
 src/gallium/winsys/sw/hgl/hgl_sw_winsys.c    |   20 +++++++++++++++++---
 src/gallium/winsys/sw/hgl/hgl_sw_winsys.h    |    4 ++++
 4 files changed, 41 insertions(+), 8 deletions(-)

diff --git a/src/gallium/winsys/sw/hgl/bitmap_wrapper.cpp 
b/src/gallium/winsys/sw/hgl/bitmap_wrapper.cpp
index 4015a22..ef81edc 100644
--- a/src/gallium/winsys/sw/hgl/bitmap_wrapper.cpp
+++ b/src/gallium/winsys/sw/hgl/bitmap_wrapper.cpp
@@ -77,10 +77,22 @@ void
 copy_bitmap_bits(const Bitmap* bitmap, void* data, int32 length)
 {
        BBitmap *bb = (BBitmap*)bitmap;
-       if (bb) {
-               color_space cs = bb->ColorSpace();
-               bb->ImportBits(data, length, bb->BytesPerRow(), 0, cs);
-       }
+
+       // We assume the data is 1:1 the format of the bitmap
+       if (bb)
+               bb->ImportBits(data, length, bb->BytesPerRow(), 0, 
bb->ColorSpace());
+}
+
+
+void
+import_bitmap_bits(const Bitmap* bitmap, void* data, int32 length,
+       unsigned srcStride, color_space srcColorSpace)
+{
+       BBitmap *bb = (BBitmap*)bitmap;
+
+       // Import image and adjust image format from source to dest
+       if (bb)
+               bb->ImportBits(data, length, srcStride, 0, srcColorSpace);
 }
 
 
diff --git a/src/gallium/winsys/sw/hgl/bitmap_wrapper.h 
b/src/gallium/winsys/sw/hgl/bitmap_wrapper.h
index 7c5ff2d..65ba140 100644
--- a/src/gallium/winsys/sw/hgl/bitmap_wrapper.h
+++ b/src/gallium/winsys/sw/hgl/bitmap_wrapper.h
@@ -40,14 +40,17 @@ extern "C" {
 
 
 Bitmap* create_bitmap(int32 width, int32 height, color_space colorSpace);
+void delete_bitmap(Bitmap* bitmap);
+
 void copy_bitmap_bits(const Bitmap* bitmap, void* data, int32 length);
+void import_bitmap_bits(const Bitmap* bitmap, void* data, int32 length,
+       unsigned srcStride, color_space srcColorSpace);
 
 void get_bitmap_size(const Bitmap* bitmap, int32* width, int32* height);
 color_space get_bitmap_color_space(const Bitmap* bitmap);
 int32 get_bitmap_bytes_per_row(const Bitmap* bitmap);
 int32 get_bitmap_bits_length(const Bitmap* bitmap);
 
-void delete_bitmap(Bitmap* bitmap);
 void dump_bitmap(const Bitmap* bitmap);
 
 
diff --git a/src/gallium/winsys/sw/hgl/hgl_sw_winsys.c 
b/src/gallium/winsys/sw/hgl/hgl_sw_winsys.c
index 1d51dd6..b09584c 100644
--- a/src/gallium/winsys/sw/hgl/hgl_sw_winsys.c
+++ b/src/gallium/winsys/sw/hgl/hgl_sw_winsys.c
@@ -34,7 +34,6 @@
 #include "util/u_memory.h"
 
 #include "hgl_sw_winsys.h"
-#include "bitmap_wrapper.h"
 
 
 // Cast
@@ -60,6 +59,19 @@ hgl_winsys_is_displaytarget_format_supported(struct 
sw_winsys* winsys,
        return true;
 }
 
+static color_space
+hgl_winsys_convert_cs(enum pipe_format format)
+{
+       // TODO: B_RGB24, B_RGB16, B_RGB15?
+       switch(format) {
+               case PIPE_FORMAT_B5G6R5_UNORM:
+                       return B_CMAP8;
+               case PIPE_FORMAT_A8R8G8B8_UNORM:
+               case PIPE_FORMAT_X8R8G8B8_UNORM:
+               default:
+                       return B_RGB32;
+       }
+}
 
 static struct sw_displaytarget*
 hgl_winsys_displaytarget_create(struct sw_winsys* winsys,
@@ -70,6 +82,7 @@ hgl_winsys_displaytarget_create(struct sw_winsys* winsys,
                = CALLOC_STRUCT(haiku_displaytarget);
        assert(haikuDisplayTarget);
 
+       haikuDisplayTarget->colorSpace = hgl_winsys_convert_cs(format);
        haikuDisplayTarget->format = format;
        haikuDisplayTarget->width = width;
        haikuDisplayTarget->height = height;
@@ -156,8 +169,9 @@ hgl_winsys_displaytarget_display(struct sw_winsys* winsys,
        struct haiku_displaytarget* haikuDisplayTarget
                = hgl_sw_displaytarget(displayTarget);
 
-       copy_bitmap_bits(bitmap, haikuDisplayTarget->data,
-               haikuDisplayTarget->size);
+       import_bitmap_bits(bitmap, haikuDisplayTarget->data,
+               haikuDisplayTarget->size, haikuDisplayTarget->stride,
+               haikuDisplayTarget->colorSpace);
 
        return;
 }
diff --git a/src/gallium/winsys/sw/hgl/hgl_sw_winsys.h 
b/src/gallium/winsys/sw/hgl/hgl_sw_winsys.h
index 4c706a5..5a2bef7 100644
--- a/src/gallium/winsys/sw/hgl/hgl_sw_winsys.h
+++ b/src/gallium/winsys/sw/hgl/hgl_sw_winsys.h
@@ -32,10 +32,14 @@
 #include "state_tracker/st_api.h"
 #include "state_tracker/sw_winsys.h"
 
+#include "bitmap_wrapper.h"
+
 
 struct haiku_displaytarget
 {
        enum pipe_format format;
+       color_space colorSpace;
+
        unsigned width;
        unsigned height;
        unsigned stride;
-- 
1.7.9.5

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev