<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 11, 2017 at 10:36 AM, Scott D Phillips <span dir="ltr"><<a href="mailto:scott.d.phillips@intel.com" target="_blank">scott.d.phillips@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="">Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> writes:<br>
<br>
> oof...  Have you run this through Jenkins?  It should be ok, but<br>
> it will be a functional change.  It's a good change, but it is a<br>
> change.<br>
<br>
</span>I ran the patch through jenkins, where the only problem it<br>
reported was a gpu hang on skl gt2 in:<br>
<br>
ES2-CTS.functional.state_<wbr>query.integers.stencil_back_<wbr>fail_separate_both_getfloat<br>
<br>
Although the test succeeded and I didn't see a hang in several<br>
thousand runs on my own skl gt2. Maybe the hang was somebody<br>
else's fault?<span class=""><br></span></blockquote><div><br></div><div>Yeah, I doubt this caused the hang.  Wouldn't be a bad idea to run it again just to be sure.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> Also, this should probably get CCd to stable.<br>
<br>
</span>I was under the impression that a Fixes tag somehow magically got<br>
it to the right stable branches. If that's not the case then it<br>
looks like it should be:<br>
<br>
Cc: "17.2" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br><div class="HOEnZb"><div class="h5"></div></div></blockquote><div><br></div><div>Yup, that's right.<br><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
> --Jason<br>
><br>
> On Wed, Aug 9, 2017 at 3:52 PM, Scott D Phillips <<a href="mailto:scott.d.phillips@intel.com">scott.d.phillips@intel.com</a><br>
>> wrote:<br>
><br>
>> intel_miptree_texture_aux_<wbr>usage() takes an isl_format, but we are<br>
>> passing a mesa_format. clang warns:<br>
>><br>
>>  brw_blorp.c:305:52: warning: implicit conversion from enumeration<br>
>>     type 'mesa_format' to different enumeration type<br>
>>     'enum isl_format' [-Wenum-conversion]<br>
>>        intel_miptree_texture_aux_<wbr>usage(brw, src_mt, src_format);<br>
>>        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<wbr>~              ^~~~~~~~~~<br>
>><br>
>> Fixes: fc1639e46d ("i965/blorp: Use texture/render_aux_usage for blits")<br>
>> ---<br>
>>  src/mesa/drivers/dri/i965/brw_<wbr>blorp.c | 3 ++-<br>
>>  1 file changed, 2 insertions(+), 1 deletion(-)<br>
>><br>
>> diff --git a/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
>> b/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
>> index 446f507619..d8e48064e3 100644<br>
>> --- a/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
>> +++ b/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
>> @@ -301,8 +301,9 @@ brw_blorp_blit_miptrees(struct brw_context *brw,<br>
>>        src_format = dst_format = MESA_FORMAT_R_FLOAT32;<br>
>>     }<br>
>><br>
>> +   enum isl_format src_isl_format = brw_isl_format_for_mesa_<br>
>> format(src_format);<br>
>>     enum isl_aux_usage src_aux_usage =<br>
>> -      intel_miptree_texture_aux_<wbr>usage(brw, src_mt, src_format);<br>
>> +      intel_miptree_texture_aux_<wbr>usage(brw, src_mt, src_isl_format);<br>
>>     /* We do format workarounds for some depth formats so we can't reliably<br>
>>      * sample with HiZ.  One of these days, we should fix that.<br>
>>      */<br>
>> --<br>
>> 2.13.3<br>
>><br>
>> ______________________________<wbr>_________________<br>
>> mesa-dev mailing list<br>
>> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
>> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
>><br>
</div></div></blockquote></div><br></div></div>