<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 2, 2015 at 11:18 AM, Ilia Mirkin <span dir="ltr"><<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hmmm... I was just looking at this code in connection to attepmting to<br>
enable ARB_texture_stencil8, and it _seems_ like that should be if<br>
(ARB_texture_stencil8) -- I didn't see what in ARB_stencil_texturing<br>
had to do with being able to have a GL_STENCIL_INDEX* internal<br>
format...<br></blockquote><div><br></div><div>I just pushed it because it does fix a bug...  However, you do raise a good point.  Maybe we should change the i965 driver to use GL_STENCIL_INDEX for the internal format for our internal stencil buffers.<br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  -ilia<br>
<div><div class="h5"><br>
On Fri, Feb 27, 2015 at 6:55 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> From: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
><br>
> This takes "fbo-stencil blit GL_STENCIL_INDEX1/4/16" from crash to pass on<br>
> BDW.<br>
> ---<br>
>  src/mesa/main/teximage.c | 3 +++<br>
>  1 file changed, 3 insertions(+)<br>
><br>
> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c<br>
> index 6e45cc9..a68574d 100644<br>
> --- a/src/mesa/main/teximage.c<br>
> +++ b/src/mesa/main/teximage.c<br>
> @@ -224,7 +224,10 @@ _mesa_base_tex_format( struct gl_context *ctx, GLint internalFormat )<br>
>     if (ctx->Extensions.ARB_stencil_texturing) {<br>
>        switch (internalFormat) {<br>
>        case GL_STENCIL_INDEX:<br>
> +      case GL_STENCIL_INDEX1:<br>
> +      case GL_STENCIL_INDEX4:<br>
>        case GL_STENCIL_INDEX8:<br>
> +      case GL_STENCIL_INDEX16:<br>
>           return GL_STENCIL_INDEX;<br>
>        default:<br>
>           ; /* fallthrough */<br>
> --<br>
> 2.1.0<br>
><br>
</div></div>> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div></div>