<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 1, 2014 at 6:44 AM, Neil Roberts <span dir="ltr"><<a href="mailto:neil@linux.intel.com" target="_blank">neil@linux.intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Is this patch necessary? I think the read buffer is part of the<br>
framebuffer state so any meta function that binds its own framebuffer<br>
won't need to save the read buffer, right? This is the case for<br>
_mesa_meta_CopyImageSubData_uncompressed which binds both a read and<br>
write FBO so I think it shouldn't be needed at least for this patch<br>
series. Presumably the _mesa_meta_begin there can just pass 0 for the<br>
state to save because hardly any state affects glBlitFramebuffer and it<br>
is binding its own framebuffers.<br></blockquote><div><br></div><div>Neil,<br>Thanks for pointing that out.  Some how in all the confusion of GL API's, I missed the fact that it's implicitly bound to the framebuffer.  The docs for gl[Read/Draw]Buffer say nothing about that.  I'll drop this patch.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
If we do use this patch then I think MESA_META_READ_BUFFER would have to<br>
be removed from the state that is saved in<br>
copytexsubimage_using_blit_framebuffer because in that case we do want<br>
to read from whatever the application has chosen for the read buffer.<br>
There seems to be a number of other meta functions that explicitly<br>
disable MESA_META_DRAW_BUFFER even though they bind their own<br>
framebuffer. Maybe this is just for performance, but arguably these<br>
functions should also disable saving MESA_META_READ_BUFFER.<br>
<br>
Regards,<br>
- Neil<br>
_______________________________________________<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>