[Mesa-dev] [PATCH] nvc0: add hardware ETC2 and ASTC support where possible

Ilia Mirkin imirkin at alum.mit.edu
Sat Dec 19 15:06:21 PST 2015


On Sat, Dec 19, 2015 at 1:53 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> These are supported on GK20A and GM107.
>
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> ---
>
> Was a bit torn on where to place the enums... we're about to gut all
> the xml definitions so this seemed appropriate for now.
>
> Tested on GK20A only.
>
>  src/gallium/drivers/nouveau/nv50/nv50_formats.c | 64 +++++++++++++++++++++++++
>  src/gallium/drivers/nouveau/nvc0/nvc0_screen.c  | 10 ++++
>  src/gallium/drivers/nouveau/nvc0/nvc0_tex.c     |  4 +-
>  3 files changed, 77 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/nouveau/nv50/nv50_formats.c b/src/gallium/drivers/nouveau/nv50/nv50_formats.c
> index 49a93bf..971fc64 100644
> --- a/src/gallium/drivers/nouveau/nv50/nv50_formats.c
> +++ b/src/gallium/drivers/nouveau/nv50/nv50_formats.c
> @@ -84,6 +84,29 @@
>  #define NV50_TIC_0_FMT_BPTC_FLOAT   NVC0_TIC_0_FMT_BPTC_FLOAT
>  #define NV50_TIC_0_FMT_BPTC_UFLOAT  NVC0_TIC_0_FMT_BPTC_UFLOAT
>
> +/* GK20A and GM107+ formats */
> +#define NV50_TIC_0_FMT_ETC2_RGB      0x06
> +#define NV50_TIC_0_FMT_ETC2_RGBA1    0x0a
> +#define NV50_TIC_0_FMT_ETC2_RGBA     0x0b
> +#define NV50_TIC_0_FMT_ETC2_R11      0x19
> +#define NV50_TIC_0_FMT_ETC2_RG11     0x1a
> +
> +/* On GK20A, the high bit goes elsewhere */
> +#define NV50_TIC_0_FMT_ASTC_2D_4X4   0x40
> +#define NV50_TIC_0_FMT_ASTC_2D_5X4   0x50
> +#define NV50_TIC_0_FMT_ASTC_2D_5X5   0x41
> +#define NV50_TIC_0_FMT_ASTC_2D_6X5   0x51
> +#define NV50_TIC_0_FMT_ASTC_2D_6X6   0x42
> +#define NV50_TIC_0_FMT_ASTC_2D_8X5   0x55
> +#define NV50_TIC_0_FMT_ASTC_2D_8X6   0x52
> +#define NV50_TIC_0_FMT_ASTC_2D_8X8   0x44
> +#define NV50_TIC_0_FMT_ASTC_2D_10X5  0x56
> +#define NV50_TIC_0_FMT_ASTC_2D_10X6  0x57
> +#define NV50_TIC_0_FMT_ASTC_2D_10X8  0x53
> +#define NV50_TIC_0_FMT_ASTC_2D_10X10 0x45
> +#define NV50_TIC_0_FMT_ASTC_2D_12X10 0x54
> +#define NV50_TIC_0_FMT_ASTC_2D_12X12 0x46
> +
>  #if NOUVEAU_DRIVER == 0xc0
>  # define NVXX_3D_VAF_SIZE(s) NVC0_3D_VERTEX_ATTRIB_FORMAT_SIZE_##s
>  # define NVXX_3D_VAF_TYPE(t) NVC0_3D_VERTEX_ATTRIB_FORMAT_TYPE_##t
> @@ -296,6 +319,47 @@ const struct nv50_format nv50_format_table[PIPE_FORMAT_COUNT] =
>     F3B(BPTC_RGB_FLOAT,  NONE, C0, C1, C2, xx, FLOAT, BPTC_FLOAT, t),
>     F3B(BPTC_RGB_UFLOAT, NONE, C0, C1, C2, xx, FLOAT, BPTC_UFLOAT, t),
>
> +   F3B(ETC2_RGB8,       NONE, C0, C1, C2, xx, UNORM, ETC2_RGB,   t),
> +   F3B(ETC2_SRGB8,      NONE, C0, C1, C2, xx, UNORM, ETC2_RGB,   t),
> +   C4B(ETC2_RGB8A1,     NONE, C0, C1, C2, C3, UNORM, ETC2_RGBA1, t),
> +   C4B(ETC2_SRGB8A1,    NONE, C0, C1, C2, C3, UNORM, ETC2_RGBA1, t),
> +   C4B(ETC2_RGBA8,      NONE, C0, C1, C2, C3, UNORM, ETC2_RGBA,  t),
> +   C4B(ETC2_SRGBA8,     NONE, C0, C1, C2, C3, UNORM, ETC2_RGBA,  t),
> +   F1B(ETC2_R11_UNORM,  NONE, C0, xx, xx, xx, UNORM, ETC2_R11,   t),
> +   F1B(ETC2_R11_SNORM,  NONE, C0, xx, xx, xx, SNORM, ETC2_R11,   t),
> +   F2B(ETC2_RG11_UNORM, NONE, C0, C1, xx, xx, UNORM, ETC2_RG11,  t),
> +   F2B(ETC2_RG11_SNORM, NONE, C0, C1, xx, xx, SNORM, ETC2_RG11,  t),
> +
> +   C4B(ASTC_4x4,        NONE, C0, C1, C2, C3, UNORM, ASTC_2D_4X4,   t),
> +   C4B(ASTC_5x4,        NONE, C0, C1, C2, C3, UNORM, ASTC_2D_5X4,   t),
> +   C4B(ASTC_5x5,        NONE, C0, C1, C2, C3, UNORM, ASTC_2D_5X5,   t),
> +   C4B(ASTC_6x5,        NONE, C0, C1, C2, C3, UNORM, ASTC_2D_6X5,   t),
> +   C4B(ASTC_6x6,        NONE, C0, C1, C2, C3, UNORM, ASTC_2D_6X6,   t),
> +   C4B(ASTC_8x5,        NONE, C0, C1, C2, C3, UNORM, ASTC_2D_8X5,   t),
> +   C4B(ASTC_8x6,        NONE, C0, C1, C2, C3, UNORM, ASTC_2D_8X6,   t),
> +   C4B(ASTC_8x8,        NONE, C0, C1, C2, C3, UNORM, ASTC_2D_8X8,   t),
> +   C4B(ASTC_10x5,       NONE, C0, C1, C2, C3, UNORM, ASTC_2D_10X5,  t),
> +   C4B(ASTC_10x6,       NONE, C0, C1, C2, C3, UNORM, ASTC_2D_10X6,  t),
> +   C4B(ASTC_10x8,       NONE, C0, C1, C2, C3, UNORM, ASTC_2D_10X8,  t),
> +   C4B(ASTC_10x10,      NONE, C0, C1, C2, C3, UNORM, ASTC_2D_10X10, t),
> +   C4B(ASTC_12x10,      NONE, C0, C1, C2, C3, UNORM, ASTC_2D_12X10, t),
> +   C4B(ASTC_12x12,      NONE, C0, C1, C2, C3, UNORM, ASTC_2D_12X12, t),
> +
> +   C4B(ASTC_4x4_SRGB,   NONE, C0, C1, C2, C3, UNORM, ASTC_2D_4X4,   t),
> +   C4B(ASTC_5x4_SRGB,   NONE, C0, C1, C2, C3, UNORM, ASTC_2D_5X4,   t),
> +   C4B(ASTC_5x5_SRGB,   NONE, C0, C1, C2, C3, UNORM, ASTC_2D_5X5,   t),
> +   C4B(ASTC_6x5_SRGB,   NONE, C0, C1, C2, C3, UNORM, ASTC_2D_6X5,   t),
> +   C4B(ASTC_6x6_SRGB,   NONE, C0, C1, C2, C3, UNORM, ASTC_2D_6X6,   t),
> +   C4B(ASTC_8x5_SRGB,   NONE, C0, C1, C2, C3, UNORM, ASTC_2D_8X5,   t),
> +   C4B(ASTC_8x6_SRGB,   NONE, C0, C1, C2, C3, UNORM, ASTC_2D_8X6,   t),
> +   C4B(ASTC_8x8_SRGB,   NONE, C0, C1, C2, C3, UNORM, ASTC_2D_8X8,   t),
> +   C4B(ASTC_10x5_SRGB,  NONE, C0, C1, C2, C3, UNORM, ASTC_2D_10X5,  t),
> +   C4B(ASTC_10x6_SRGB,  NONE, C0, C1, C2, C3, UNORM, ASTC_2D_10X6,  t),
> +   C4B(ASTC_10x8_SRGB,  NONE, C0, C1, C2, C3, UNORM, ASTC_2D_10X8,  t),
> +   C4B(ASTC_10x10_SRGB, NONE, C0, C1, C2, C3, UNORM, ASTC_2D_10X10, t),
> +   C4B(ASTC_12x10_SRGB, NONE, C0, C1, C2, C3, UNORM, ASTC_2D_12X10, t),
> +   C4B(ASTC_12x12_SRGB, NONE, C0, C1, C2, C3, UNORM, ASTC_2D_12X12, t),
> +
>     C4A(R32G32B32A32_FLOAT, RGBA32_FLOAT, C0, C1, C2, C3, FLOAT, 32_32_32_32,
>         IBV, 0),
>     C4A(R32G32B32A32_UNORM, NONE, C0, C1, C2, C3, UNORM, 32_32_32_32, TV, 0),
> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
> index 4757fe2..1c88580 100644
> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
> @@ -43,6 +43,7 @@ nvc0_screen_is_format_supported(struct pipe_screen *pscreen,
>                                  unsigned sample_count,
>                                  unsigned bindings)
>  {
> +   const struct util_format_description *desc = util_format_description(format);
>     if (sample_count > 8)
>        return false;
>     if (!(0x117 & (1 << sample_count))) /* 0, 1, 2, 4 or 8 */
> @@ -55,6 +56,15 @@ nvc0_screen_is_format_supported(struct pipe_screen *pscreen,
>        if (util_format_get_blocksizebits(format) == 3 * 32)
>           return false;
>
> +   /* Restrict ETC2 and ASTC formats here. These are only supported on GM107+
> +    * and GK20A.
> +    */
> +   if ((desc->layout == UTIL_FORMAT_LAYOUT_ETC ||
> +        desc->layout == UTIL_FORMAT_LAYOUT_ASTC) &&
> +       nouveau_screen(pscreen)->class_3d < GM107_3D_CLASS &&
> +       nouveau_screen(pscreen)->class_3d != NVEA_3D_CLASS)
> +      return false;
> +
>     /* transfers & shared are always supported */
>     bindings &= ~(PIPE_BIND_TRANSFER_READ |
>                   PIPE_BIND_TRANSFER_WRITE |
> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_tex.c b/src/gallium/drivers/nouveau/nvc0/nvc0_tex.c
> index 74090ce..bc648fe 100644
> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_tex.c
> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_tex.c
> @@ -102,7 +102,9 @@ nvc0_create_texture_view(struct pipe_context *pipe,
>
>     desc = util_format_description(view->pipe.format);
>
> -   tic[0] = nvc0_format_table[view->pipe.format].tic;
> +   tic[0] = nvc0_format_table[view->pipe.format].tic & 0x3f;

Naturally this needs to be ~0x40. Oops. Fixed locally.

> +   if (nvc0_format_table[view->pipe.format].tic & 0x40)
> +      tic[0] |= 1U << 31;
>
>     tex_int = util_format_is_pure_integer(view->pipe.format);
>
> --
> 2.4.10
>


More information about the mesa-dev mailing list