<div dir="ltr">Hi Rafael,<div><br></div><div>Thanks a lot for reviewing the patch! Hopefully no one will have any objections to push this.</div><div><br></div><div>Thanks,</div><div>Vadym</div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-07-26 18:11 GMT+03:00 Rafael Antognolli <span dir="ltr"><<a href="mailto:rafael.antognolli@intel.com" target="_blank">rafael.antognolli@intel.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Vadym,<br>
<br>
Ken and Ian explained a bit the situation on this one to me, and it<br>
looks like neither of them are really against this patch. So unless<br>
someone else raise any concern, I'll ack and push the patch later today.<br>
<br>
Thanks for fixing this.<br>
<br>
Rafael<br>
<br>
On Thu, Jul 26, 2018 at 04:04:29PM +0300, Vadym Shovkoplias wrote:<br>
> ping<br>
<div class="HOEnZb"><div class="h5">> <br>
> On Tue, Jul 3, 2018 at 5:09 PM, Vadim Shovkoplias <<a href="mailto:vadim.shovkoplias@gmail.com">vadim.shovkoplias@gmail.com</a>><br>
> wrote:<br>
> <br>
>     Hi mesa devs,<br>
> <br>
>     Can anyone please review this ? <br>
>     This patch fixes following bugs:<br>
> <br>
>     Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=104388" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=104388</a><br>
>     Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=106158" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=106158</a><br>
>     Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=106667" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=106667</a><br>
> <br>
>    <br>
>     2018-06-07 18:27 GMT+03:00 Vadim Shovkoplias <<a href="mailto:vadim.shovkoplias@gmail.com">vadim.shovkoplias@gmail.com</a>>:<br>
> <br>
>         Hi Kenneth,<br>
> <br>
>         Can you please look at this patch ?<br>
> <br>
>         2018-06-07 15:30 GMT+03:00 Den <<a href="mailto:den.kos363@gmail.com">den.kos363@gmail.com</a>>:<br>
> <br>
>             Hello. Found out that this patch also fixes 2 new issues:<br>
> <br>
>             Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=106158" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=106158</a><br>
> <br>
>             Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=106667" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=106667</a><br>
> <br>
>             Tested-by: Denis <<a href="mailto:denys.kostin@globallogic.com">denys.kostin@globallogic.com</a>><br>
> <br>
> <br>
> <br>
>             On 24.05.18 14:16, vadym.shovkoplias wrote:<br>
> <br>
>                 Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=104388" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=104388</a><br>
>                 Signed-off-by: Andriy Khulap <<a href="mailto:andriy.khulap@globallogic.com">andriy.khulap@globallogic.com</a><wbr>><br>
>                 ---<br>
>                   src/mesa/drivers/dri/i965/<wbr>genX_state_upload.c | 11<br>
>                 +++++++++++<br>
>                   1 file changed, 11 insertions(+)<br>
> <br>
>                 diff --git a/src/mesa/drivers/dri/i965/<wbr>genX_state_upload.c b/<br>
>                 src/mesa/drivers/dri/i965/<wbr>genX_state_upload.c<br>
>                 index b485e2c..5aa8033 100644<br>
>                 --- a/src/mesa/drivers/dri/i965/<wbr>genX_state_upload.c<br>
>                 +++ b/src/mesa/drivers/dri/i965/<wbr>genX_state_upload.c<br>
>                 @@ -2473,6 +2473,17 @@ brw_calculate_guardband_size(<wbr>uint32_t<br>
>                 fb_width, uint32_t fb_height,<br>
>                       */<br>
>                      const float gb_size = GEN_GEN >= 7 ? 16384.0f : 8192.0f;<br>
>                   +   /* Workaround: prevent gpu hangs on SandyBridge<br>
>                 +    * by disabling guardband clipping for odd dimensions.<br>
>                 +    */<br>
>                 +   if (GEN_GEN == 6 && (fb_width & 1 || fb_height & 1)) {<br>
>                 +      *xmin = -1.0f;<br>
>                 +      *xmax =  1.0f;<br>
>                 +      *ymin = -1.0f;<br>
>                 +      *ymax =  1.0f;<br>
>                 +      return;<br>
>                 +   }<br>
>                 +<br>
>                      if (m00 != 0 && m11 != 0) {<br>
>                         /* First, we compute the screen-space render area */<br>
>                         const float ss_ra_xmin = MIN3(        0, m30 + m00, m30<br>
>                 - m00);<br>
> <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>
> <br>
> <br>
> <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>
> <br>
> <br>
> <br>
> <br>
</div></div><span class="im HOEnZb">> --<br>
> <br>
> Vadym Shovkoplias | Senior Software Engineer<br>
> GlobalLogic<br>
> P +380.57.766.7667  M +3.8050.931.7304  S vadym.shovkoplias<br>
> <a href="http://www.globallogic.com" rel="noreferrer" target="_blank">www.globallogic.com</a><br>
>  <br>
> <a href="http://www.globallogic.com/email_disclaimer.txt" rel="noreferrer" target="_blank">http://www.globallogic.com/<wbr>email_disclaimer.txt</a><br>
</span><div class="HOEnZb"><div class="h5">______________________________<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>
</div></div></blockquote></div><br></div>