<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 20, 2016 at 2:47 PM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat 18 Jun 2016, Jason Ekstrand wrote:<br>
> The docs specify that this only matters for render targets and surfaces<br>
> used with typed dataport messages.  On some platforms (gen4-6) the Depth<br>
> field has more bits than RenderTargetViewExtent so we can have textures<br>
> with more levels than we can render to.<br>
<br>
<br>
</span>I really want to see the commit message as a comment in the code,<br>
because it's subtle and surprising. Without such a comment, it's<br>
a complete mystery why the assignment is guarded by the usage flags.<br>
After all, the PRMs say RenderTargetViewExtent is ignored when the<br>
surface is neither a render target nor dataport surface; so<br>
(rhetorically) why doesn't set RenderTargetViewExtent unconditionally?<br></blockquote><div><br></div><div>Fair enough.  I'll add a comment<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Insert some variant of the commit message as comment, and this is<br>
Reviewed-by: Chad Versace <<a href="mailto:chad.versace@intel.com">chad.versace@intel.com</a>><br></blockquote><div><br></div><div>Thanks!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also, how did you find this subtle problem? Did a pack function throw an<br>
assertion failure?</blockquote><div><br></div><div>Ian wanted me to leave the max size of 3-D textures at 2048 on gen4-5 (since they don't do layered rendering) and it would have had problems in that case.  How did I find it?  I knew it was a problem from when I found the 3-D texture size issue. <br></div></div><br></div></div>