<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body><span class="vcard"><a class="email" href="mailto:itoral@igalia.com" title="Iago Toral <itoral@igalia.com>"> <span class="fn">Iago Toral</span></a>
</span> changed
              <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - glTexImage2D regression by texstore_rgba switch to _mesa_format_convert"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=89068">bug 89068</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;">CC</td>
           <td>
                
           </td>
           <td>jason@jlekstrand.net
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - glTexImage2D regression by texstore_rgba switch to _mesa_format_convert"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=89068#c7">Comment # 7</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - glTexImage2D regression by texstore_rgba switch to _mesa_format_convert"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=89068">bug 89068</a>
              from <span class="vcard"><a class="email" href="mailto:itoral@igalia.com" title="Iago Toral <itoral@igalia.com>"> <span class="fn">Iago Toral</span></a>
</span></b>
        <pre>After testing with a binary provided by the reporter and comparing with the old
code it looks like the problem comes from the fact that they are using
transferOps for color scaling (which is effectively scaling 50% alpha values to
100%)... and we are not dealing with that in texstore_rgba.

If we need to apply transferOps (we can call _mesa_texstore_needs_transfer_ops
to check this) then we have to convert from the src format to RGBA_FLOAT, so we
can call calling mesa_apply_rgba_transfer_ops(), then convert the result from
RGBA_FLOAT to the dst format...

I think we should do this after we have handled swaping scenarios and before
the current call to _mesa_format_convert.

Jason, if you agree with this I can write a patch.</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>