<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 3, 2015 at 1:00 PM, 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"><div class="HOEnZb"><div class="h5">On Tue, Mar 3, 2015 at 3:58 PM, Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> wrote:<br>
> On Monday, March 02, 2015 02:33:31 PM Ilia Mirkin wrote:<br>
>> On Mon, Mar 2, 2015 at 2:32 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
>> ><br>
>> ><br>
>> > On Mon, Mar 2, 2015 at 11:18 AM, Ilia Mirkin <<a href="mailto:imirkin@alum.mit.edu">imirkin@alum.mit.edu</a>> wrote:<br>
>> >><br>
>> >> 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>
>> ><br>
>> ><br>
>> > I just pushed it because it does fix a bug...  However, you do raise a good<br>
>> > point.  Maybe we should change the i965 driver to use GL_STENCIL_INDEX for<br>
>> > the internal format for our internal stencil buffers.<br>
>><br>
>> How do you end up with a GL_STENCIL_INDEX internal format in the first<br>
>> place? Without ARB_texture_stencil8 that's not a thing, is it?<br>
><br>
> Eric added a ctx->Driver.BindRenderbufferTexImage hook which takes a<br>
> (non-texture) renderbuffer and wraps it in a fake texture image.<br>
><br>
> We use this in Meta's BlitFramebuffer implementation to be able to blit<br>
> from non-texture renderbuffers.  i965 also has a custom meta blit path<br>
> for stencil, which uses the same approach.  Thus, stencil textures :)<br>
<br>
</div></div>None of which has to do with ARB_stencil_texturing being enabled as<br>
far as mesa/main is concerned, right? IOW, why is that condition<br>
there?<br></blockquote><div><br></div><div>Well, the blit path does require it because it requires us to be able to sample from stencil.  Why does it require the internal format?  That's another question.<br></div></div></div></div>