This patch-set aims to fix r600g's big-endian support, which was broken due to the big color formats rework that was done in mesa/gallium a few years ago. Because piglit run on BE crashes with upstream code (even in -1 mode), I reduced the OpenGL version to the lowest possible - 1.3 - while developing this patch-set.
At first, I tried to fix everything using the existing code and just adjust the swapping/format functions. Soon I found out it is not enough, as the same configuration fixed one use-case but broke another use-case and vice-versa. After some thinking, I came to the conclusion that I needed more information about the current operation being done on the resource (texture, pixels, etc.). I then added a new flag to struct pipe_resource that tells me in which format the resource is (if its a source) or should be (if its a destination). Then, in all the format/swapping functions, I check this flag and act accordingly. The flag is never checked/acted-upon in other layers except the H/W driver (r600g), so there is zero impact on other drivers. See more detailed explanation in patch 3 commit msg. Tests were done using piglit and by visual observance (I often looked at the screen to verify the colors are correct). For BE, I used a POWER7 machine with a CEDAR GL card (Evergreen). For LE, I used a skylake machine with the same card. I fixed most of the major tests (and some minor ones) - readpixsanity, draw-pixels, texwrap, copy-pixels and more. I started with 223 failed tests and finished with 129 failed tests. Most of the tests I fixed were already working in llvmpipe BE, so I knew that at least the core mesa code is working. In accordance to that, most of the remaining failed tests also don't work in llvmpipe BE, so there is probably something more fundamentaly wrong there. Results of piglit tests are found in my fdo web folder: https://people.freedesktop.org/~gabbayo/piglit_results/r600g-big-endian/ There are three comparisons, all of them done after reducing OpenGL version to 1.3: 1. r600g-1.3-base-le-vs-be - Diff between LE and BE with upstream code to see the current status 2. r600g-1.3-le-vs-be - Diff between LE and BE with upstream + patch-set to see the remaining tests to be fixed 3. r600g-be-1.3-all-fixes - Diff between upstream code and upstream + patch-set, both running in BE. This shows all the tests my patch-set fixed (and no regressions) In addition, I of course checked for regressions on LE, without reducing OpenGL version. Thanks, Oded Oded Gabbay (10): r600g: fix code indentation r600g: Move R600_BIG_ENDIAN to r600_pipe_common.h mesa/st: add endian_format field to struct pipe_resource r600g: propagate endian_format to relevant functions radeonsi: fix calls to r600 functions that take endian_format r600g: update endian_format after operations r600g: set staging texture endianess r600g: set endianess of 16/32-bit buffers according to endian_format r600g: use endian_format in color swapping functions r600g: use endian_format in texture swapping function src/gallium/drivers/r600/evergreen_state.c | 24 +++++----- src/gallium/drivers/r600/r600_blit.c | 18 ++++---- src/gallium/drivers/r600/r600_pipe.c | 2 +- src/gallium/drivers/r600/r600_pipe.h | 14 +++--- src/gallium/drivers/r600/r600_state.c | 18 ++++---- src/gallium/drivers/r600/r600_state_common.c | 64 ++++++++++++++++++++++----- src/gallium/drivers/radeon/r600_pipe_common.h | 8 +++- src/gallium/drivers/radeon/r600_texture.c | 36 ++++++++++++--- src/gallium/drivers/radeonsi/si_state.c | 4 +- src/gallium/include/pipe/p_state.h | 1 + src/mesa/state_tracker/st_cb_drawpixels.c | 11 ++++- src/mesa/state_tracker/st_cb_fbo.c | 16 ++++++- src/mesa/state_tracker/st_cb_readpixels.c | 5 +++ src/mesa/state_tracker/st_cb_texture.c | 13 ++++++ src/mesa/state_tracker/st_texture.c | 48 ++++++++++++++++++++ 15 files changed, 222 insertions(+), 60 deletions(-) -- 2.5.5 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
