[Mesa-dev] [PATCH v1 8/8] nvc0: disable MS images on GM107+

[Ilia Mirkin]

Series is

Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

Note that you'll start using PBO downloads via images as a result of
this change, please run some of the deqp pbo tests (might be in gles3,
not gles31) to double-check that all's well.

  -ilia

On Tue, Jul 19, 2016 at 4:15 PM, Samuel Pitoiset
<samuel.pitoiset at gmail.com> wrote:
> MS images have to be handled explicitly and I don't plan to implement
> them for now.
>
> v1: - check that sample_count > 1
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>  src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
> index f681631..a3cd046 100644
> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
> @@ -90,6 +90,13 @@ nvc0_screen_is_format_supported(struct pipe_screen *pscreen,
>                   PIPE_BIND_LINEAR |
>                   PIPE_BIND_SHARED);
>
> +   if (bindings & PIPE_BIND_SHADER_IMAGE && sample_count > 1 &&
> +       nouveau_screen(pscreen)->class_3d >= GM107_3D_CLASS) {
> +      /* MS images are currently unsupported on Maxwell because they have to
> +       * be handled explicitly. */
> +      return false;
> +   }
> +
>     return (( nvc0_format_table[format].usage |
>              nvc0_vertex_format[format].usage) & bindings) == bindings;
>  }
> --
> 2.9.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev