<div class="gmail_quote">On 7 September 2012 09:27, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><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>
> Also add a clarifying comment for why the width/height doesn't need<br>
> adjustment for Gen7.<br>
<br>
</div><div class="im">> diff --git a/src/mesa/drivers/dri/i965/gen7_blorp.cpp b/src/mesa/drivers/dri/i965/gen7_blorp.cpp<br>
> index a65a975..ff11f3a 100644<br>
> --- a/src/mesa/drivers/dri/i965/gen7_blorp.cpp<br>
> +++ b/src/mesa/drivers/dri/i965/gen7_blorp.cpp<br>
> @@ -144,6 +144,11 @@ gen7_blorp_emit_surface_state(struct brw_context *brw,<br>
>     uint32_t wm_surf_offset;<br>
>     uint32_t width, height;<br>
>     surface->get_miplevel_dims(&width, &height);<br>
> +   /* Note: since gen7 uses INTEL_MSAA_LAYOUT_CMS or INTEL_MSAA_LAYOUT_IMS for<br>
<br>
</div>Is that supposed to be s/IMS/UMS/?<br>
</blockquote></div><br><div>Oops, yes.  I'll flx that before pushing.</div>