<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body><span class="vcard"><a class="email" href="mailto:nanleychery@gmail.com" title="Nanley Chery <nanleychery@gmail.com>"> <span class="fn">Nanley Chery</span></a>
</span> changed
          <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Calculating the scissors fields when the y is flipped (0 on top) can generate negative numbers that will cause assertion failure later on."
   href="https://bugs.freedesktop.org/show_bug.cgi?id=108999">bug 108999</a>
          <br>
             <table border="1" cellspacing="0" cellpadding="8">
          <tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style="text-align:right;">Resolution</td>
           <td>---
           </td>
           <td>FIXED
           </td>
         </tr>

         <tr>
           <td style="text-align:right;">Status</td>
           <td>NEW
           </td>
           <td>RESOLVED
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Calculating the scissors fields when the y is flipped (0 on top) can generate negative numbers that will cause assertion failure later on."
   href="https://bugs.freedesktop.org/show_bug.cgi?id=108999#c3">Comment # 3</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Calculating the scissors fields when the y is flipped (0 on top) can generate negative numbers that will cause assertion failure later on."
   href="https://bugs.freedesktop.org/show_bug.cgi?id=108999">bug 108999</a>
              from <span class="vcard"><a class="email" href="mailto:nanleychery@gmail.com" title="Nanley Chery <nanleychery@gmail.com>"> <span class="fn">Nanley Chery</span></a>
</span></b>
        <pre>This bug has been fixed by:

commit fd37a19ac4c8b2ebff330b2a06a7f311f7d478e3
Author: Eleni Maria Stea <<a href="mailto:estea@igalia.com">estea@igalia.com</a>>
Date:   Fri Feb 22 23:02:30 2019 +0200

    i965: fixed clamping in set_scissor_bits when the y is flipped

    Calculating the scissor rectangle fields with the y flipped (0 on top)
    can generate negative values that will cause assertion failure later on
    as the scissor fields are all unsigned. We must clamp the bbox values
    again to make sure they don't exceed the fb_height. Also fixed a
    calculation error.

    Bugzilla: <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Calculating the scissors fields when the y is flipped (0 on top) can generate negative numbers that will cause assertion failure later on."
   href="show_bug.cgi?id=108999">https://bugs.freedesktop.org/show_bug.cgi?id=108999</a>
              <a class="bz_bug_link 
          bz_status_REOPENED "
   title="REOPENED - totem assert failure: totem: src/intel/genxml/gen9_pack.h:72: __gen_uint: La declaraciĆ³n `v <= max' no se cumple."
   href="show_bug.cgi?id=109594">https://bugs.freedesktop.org/show_bug.cgi?id=109594</a>

    v2:
       - I initially clamped the values inside the if (Y is flipped) case
       and I made a mistake in the calculation: the clamp of the bbox[2] should
       be a check if (bbox[2] >= fbheight) bbox[2] = fbheight - 1 instead and I
       shouldn't have changed the ScissorRectangleYMax calculation. As the
       fixed code is equivalent with using CLAMP instead of MAX2 at the top of
       the function when bbox[2] and bbox[3] are calculated, and the 2nd is
more
       clear, I replaced it. (Nanley Chery)

    v3:
       - Reversed the CLAMP change in bbox[3] as the API guarantees that the
       viewport height is positive. (Nanley Chery)

    v4:
      - Added nomination for the mesa-stable branch and the link to the second
      bugzilla bug (Nanley Chery)

    CC: <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.org</a>>
    Tested-by: Paul Chelombitko <<a href="mailto:qamonstergl@gmail.com">qamonstergl@gmail.com</a>>
    Reviewed-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>></pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the QA Contact for the bug.</li>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>