[PATCH v2 1/4] etnaviv: implement ETC2 block patching for HALTI0

Christian Gmeiner christian.gmeiner at gmail.com
Tue Feb 26 18:15:28 UTC 2019


ETC2 is supported with HALTI0, however that implementation is buggy
in hardware. The blob driver does per-block patching to work around
this. We need to swap colors for t-mode etc2 blocks.

Signed-off-by: Christian Gmeiner <christian.gmeiner at gmail.com>
---
 src/gallium/drivers/etnaviv/Makefile.sources |   2 +
 src/gallium/drivers/etnaviv/etnaviv_etc2.c   | 149 +++++++++++++++++++
 src/gallium/drivers/etnaviv/etnaviv_etc2.h   |  51 +++++++
 src/gallium/drivers/etnaviv/meson.build      |   2 +
 4 files changed, 204 insertions(+)
 create mode 100644 src/gallium/drivers/etnaviv/etnaviv_etc2.c
 create mode 100644 src/gallium/drivers/etnaviv/etnaviv_etc2.h

diff --git a/src/gallium/drivers/etnaviv/Makefile.sources b/src/gallium/drivers/etnaviv/Makefile.sources
index 0b208122999..01e7e49a38a 100644
--- a/src/gallium/drivers/etnaviv/Makefile.sources
+++ b/src/gallium/drivers/etnaviv/Makefile.sources
@@ -25,6 +25,8 @@ C_SOURCES :=  \
 	etnaviv_disasm.h \
 	etnaviv_emit.c \
 	etnaviv_emit.h \
+	etnaviv_etc2.c \
+	etnaviv_etc2.h \
 	etnaviv_fence.c \
 	etnaviv_fence.h \
 	etnaviv_format.c \
diff --git a/src/gallium/drivers/etnaviv/etnaviv_etc2.c b/src/gallium/drivers/etnaviv/etnaviv_etc2.c
new file mode 100644
index 00000000000..64811c3038d
--- /dev/null
+++ b/src/gallium/drivers/etnaviv/etnaviv_etc2.c
@@ -0,0 +1,149 @@
+/*
+ * Copyright (c) 2019 Etnaviv Project
+ * Copyright (C) 2019 Zodiac Inflight Innovations
+ *
+ * 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
+ * THE AUTHORS OR COPYRIGHT HOLDERS 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.
+ *
+ * Authors:
+ *    Christian Gmeiner <christian.gmeiner at gmail.com>
+ */
+
+#include "etnaviv_etc2.h"
+#include "etnaviv_resource.h"
+#include "etnaviv_screen.h"
+#include "hw/common.xml.h"
+#include "util/u_format.h"
+
+bool
+etna_etc2_needs_patching(const struct pipe_resource *prsc)
+{
+   const struct etna_screen *screen = etna_screen(prsc->screen);
+
+   if (!util_format_is_etc(prsc->format))
+      return false;
+
+   if (prsc->format == PIPE_FORMAT_ETC1_RGB8)
+      return false;
+
+   if (VIV_FEATURE(screen, chipMinorFeatures2, HALTI1))
+      return false;
+
+   switch (prsc->format) {
+   case PIPE_FORMAT_ETC2_RGB8:
+   case PIPE_FORMAT_ETC2_SRGB8:
+   case PIPE_FORMAT_ETC2_RGB8A1:
+   case PIPE_FORMAT_ETC2_SRGB8A1:
+   case PIPE_FORMAT_ETC2_RGBA8:
+   case PIPE_FORMAT_ETC2_SRGBA8:
+      return true;
+      break;
+
+   default:
+      return false;
+   }
+}
+
+static inline bool
+needs_patching(uint8_t *buffer, bool punchthrough_alpha)
+{
+   /* punchthrough_alpha or etc2 individual mode? */
+   if (!punchthrough_alpha && !(buffer[3] & 0x2))
+     return false;
+
+   /* etc2 t-mode? */
+   static const int lookup[8] = { 0, 1, 2, 3, -4, -3, -2, -1 };
+   const int R_plus_dR = (buffer[0] >> 3) + lookup[buffer[0] & 0x7];
+
+   if (R_plus_dR < 0 || R_plus_dR > 31)
+      return true;
+
+   return false;
+}
+
+void
+etna_etc2_calculate_blocks(uint8_t *buffer, unsigned stride,
+                           unsigned width, unsigned height,
+                           enum pipe_format format,
+                           struct util_dynarray *offsets)
+{
+   const unsigned bw = util_format_get_blockwidth(format);
+   const unsigned bh = util_format_get_blockheight(format);
+   const unsigned bs = util_format_get_blocksize(format);
+   bool punchthrough_alpha = false;
+   unsigned offset = 0;
+   const uint8_t *base = buffer;
+
+   if (format == PIPE_FORMAT_ETC2_RGB8A1 ||
+       format == PIPE_FORMAT_ETC2_SRGB8A1)
+      punchthrough_alpha = true;
+
+   if (format == PIPE_FORMAT_ETC2_RGBA8 ||
+       format == PIPE_FORMAT_ETC2_SRGBA8 ||
+       format == PIPE_FORMAT_ETC2_SRGB8A1)
+      offset = 8;
+
+   for (unsigned y = 0; y < height; y += bh) {
+      uint8_t *src = buffer;
+
+      for (unsigned x = 0; x < width; x+= bw) {
+         if (needs_patching(src + offset, punchthrough_alpha))
+            util_dynarray_append(offsets, unsigned, src + offset - base);
+
+         src += bs;
+      }
+
+      buffer += stride;
+   }
+}
+
+static inline void
+swap_colors(uint8_t *buffer)
+{
+   const uint8_t r1a = (buffer[0] & 0x18) >> 3;
+   const uint8_t r1b = (buffer[0] & 0x3);
+   const uint8_t r1 = (r1a << 2) | r1b;
+   const uint8_t g1 = (buffer[1] & 0xf0) >> 4;
+   const uint8_t b1 = (buffer[1] & 0x0f);
+   const uint8_t r2 = (buffer[2] & 0xf0) >> 4;
+   const uint8_t g2 = buffer[2] & 0x0f;
+   const uint8_t b2 = (buffer[3] & 0xf0) >> 4;
+   const uint8_t rest = (buffer[3] & 0x0f);
+
+   /* fixup R and dR used for t-mode detection */
+   static const uint8_t fixup[16] = {
+      0x04, 0x04, 0x04, 0x04,
+      0x04, 0x04, 0x04, 0xe0,
+      0x04, 0x04, 0xe0, 0xe0,
+      0x04, 0xe0, 0xe0, 0xe0
+   };
+
+   /* swap colors */
+   buffer[0] = fixup[r2] | ((r2 & 0x0c) << 1) | (r2 & 0x03);
+   buffer[1] = (g2 << 4) | b2;
+   buffer[2] = (r1 << 4) | g1;
+   buffer[3] = (b1 << 4) | rest;
+}
+
+void
+etna_etc2_patch(uint8_t *buffer, const struct util_dynarray *offsets)
+{
+   util_dynarray_foreach(offsets, unsigned, offset)
+      swap_colors(buffer + *offset);
+}
diff --git a/src/gallium/drivers/etnaviv/etnaviv_etc2.h b/src/gallium/drivers/etnaviv/etnaviv_etc2.h
new file mode 100644
index 00000000000..334f05ae669
--- /dev/null
+++ b/src/gallium/drivers/etnaviv/etnaviv_etc2.h
@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) 2019 Etnaviv Project
+ * Copyright (C) 2019 Zodiac Inflight Innovations
+ *
+ * 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
+ * THE AUTHORS OR COPYRIGHT HOLDERS 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.
+ *
+ * Authors:
+ *    Christian Gmeiner <christian.gmeiner at gmail.com>
+ */
+
+#ifndef H_ETNA_ETC2
+#define H_ETNA_ETC2
+
+#include <stdbool.h>
+#include <stdint.h>
+
+#include "pipe/p_format.h"
+#include "util/u_dynarray.h"
+
+struct pipe_resource;
+
+bool
+etna_etc2_needs_patching(const struct pipe_resource *prsc);
+
+void
+etna_etc2_calculate_blocks(uint8_t *buffer, unsigned stride,
+                           unsigned width, unsigned height,
+                           enum pipe_format format,
+                           struct util_dynarray *offsets);
+
+void
+etna_etc2_patch(uint8_t *buffer, const struct util_dynarray *offsets);
+
+#endif
diff --git a/src/gallium/drivers/etnaviv/meson.build b/src/gallium/drivers/etnaviv/meson.build
index 63553dec510..75ed93eccc7 100644
--- a/src/gallium/drivers/etnaviv/meson.build
+++ b/src/gallium/drivers/etnaviv/meson.build
@@ -44,6 +44,8 @@ files_etnaviv = files(
   'etnaviv_disasm.h',
   'etnaviv_emit.c',
   'etnaviv_emit.h',
+  'etnaviv_etc2.c',
+  'etnaviv_etc2.h',
   'etnaviv_fence.c',
   'etnaviv_fence.h',
   'etnaviv_format.c',
-- 
2.20.1



More information about the etnaviv mailing list