[Mesa-dev] [PATCH] nv50: limit the maximum number of samplers to 16

Ilia Mirkin imirkin at alum.mit.edu
Mon Jul 13 07:59:18 PDT 2015


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

On Mon, Jul 13, 2015 at 5:24 AM, Samuel Pitoiset
<samuel.pitoiset at gmail.com> wrote:
> NV50_3D_BIND_TSC only allows to bind 16 samplers, and since we don't
> want to do anything with NV50_3D_BIND_TSC2, just limit the maximum
> number of samplers to 16 like for nvc0.
>
> This fixes dmesg fails with the following piglit test:
>  max-samplers
>
> But the test still fails.
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>  src/gallium/drivers/nouveau/nv50/nv50_screen.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/nouveau/nv50/nv50_screen.c b/src/gallium/drivers/nouveau/nv50/nv50_screen.c
> index 6583a35..46ae0b8 100644
> --- a/src/gallium/drivers/nouveau/nv50/nv50_screen.c
> +++ b/src/gallium/drivers/nouveau/nv50/nv50_screen.c
> @@ -286,7 +286,7 @@ nv50_screen_get_shader_param(struct pipe_screen *pscreen, unsigned shader,
>     case PIPE_SHADER_CAP_MAX_TEXTURE_SAMPLERS:
>        /* The chip could handle more sampler views than samplers */
>     case PIPE_SHADER_CAP_MAX_SAMPLER_VIEWS:
> -      return MIN2(32, PIPE_MAX_SAMPLERS);
> +      return MIN2(16, PIPE_MAX_SAMPLERS);
>     case PIPE_SHADER_CAP_DOUBLES:
>     case PIPE_SHADER_CAP_TGSI_DROUND_SUPPORTED:
>     case PIPE_SHADER_CAP_TGSI_DFRACEXP_DLDEXP_SUPPORTED:
> --
> 2.4.5
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list