<div dir="ltr">On 21 January 2013 00:55, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> writes:<br>
<br>
> On 19 January 2013 11:06, Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> wrote:<br>
</div><div class="im">>> +   //intel_renderbuffer_set_needs_downsample(dst_irb);<br>
>><br>
><br>
> I'm ok leaving this out--at the moment there's no way this code can be hit<br>
> for a multisampled destination image.  However, if we ever get around to<br>
> implementing the GLES extension EXT_multisampled_render_to_texture, we'll<br>
> have to have a hard think about what the correct behaviour should be.<br>
> Perhaps we should put a comment here saying something like:<br>
><br>
> /* Note: there is no need to call intel_renderbuffer_set_needs_downsample()<br>
> on the destination buffer, since automatic downsample is only needed for<br>
> multisampled window system framebuffers, which are not allowed as<br>
> destinations for CopyTexSubImage.  However, this will need to be revisited<br>
> if we ever support EXT_multisampled_render_to_texture. */<br>
<br>
</div>Isn't that extension all about multisampled user framebuffers, not<br>
window system framebuffers?<br>
</blockquote></div><br></div><div class="gmail_extra" style>It's about multisampled textures that need to get automatically downsampled in the same way that multisampled window system framebuffers do.  That's why we'll need to revisit the relationship between CopyTexSubImage and intel_renderbuffer_set_needs_downsample() when we implement it, because we'll need to translate the renderbuffer's needs_downsample flag into a more general texture needs_downsample flag, and make sure it gets set at the proper times.</div>
</div>