[PATCH v2 1/4] etnaviv: implement ETC2 block patching for HALTI0
Lucas Stach
l.stach at pengutronix.de
Wed Feb 27 09:22:57 UTC 2019
Am Dienstag, den 26.02.2019, 19:15 +0100 schrieb Christian Gmeiner:
> 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;
Isn't this format check redundant with the default case of the switch
below?
Other than that:
Acked-by: Lucas Stach <l.stach at pengutronix.de>
Regards,
Lucas
> + 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',
More information about the etnaviv
mailing list