<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [IVB/HSW/BDW/BSW Bisected] GLB2.7 Egypt, GfxBench3.0 T-Rex & ALU and many SynMark cases performance reduced by 10-23%"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=90895#c9">Comment # 9</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [IVB/HSW/BDW/BSW Bisected] GLB2.7 Egypt, GfxBench3.0 T-Rex & ALU and many SynMark cases performance reduced by 10-23%"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=90895">bug 90895</a>
              from <span class="vcard"><a class="email" href="mailto:eero.t.tamminen@intel.com" title="Eero Tamminen <eero.t.tamminen@intel.com>"> <span class="fn">Eero Tamminen</span></a>
</span></b>
        <pre>Did some quick tests on this with GfxBench ALU test, on HSW GT3e.  Weird bug...

Reverting Ben's commit fixes the regression, although on HSW the cost per frame
should only be couple of ifs (for gen8) and skipping over perf output code on
unused branch.

Just replacing the perf_debug() calls with puts() calls improves perf, so maybe
the issue is somehow related to code inside the ifs (size of it?), although
that code doesn't get executed.

Adding missing unlikely() to the if in perf_debug() macro gave similar
improvement.  Additionally removing gen8 ifs didn't change perf.

Then I tested most of the indicated tests on BSW, HSW GT3e and HSW GT2. Just
adding unlikely() to the second if in perf_debug() macro improved the
performance several percents in the tests on all these platforms.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>