[Bug 98815] [SKL/BDW GT2] large perf regression in TessMark

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=98815

Kenneth Graunke <kenneth at whitecape.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #4 from Kenneth Graunke <kenneth at whitecape.org> ---
Fixed by:

commit 5da84a7e120d1df848531c6e7eb60340ac4dc43c
Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Wed Nov 23 12:24:22 2016 -0800

    i965: Fix a mistake from porting the URB allocation code to arrays.

    Commit 6d416bcd846a49414f210cd761789156c37a7b3e (i965: Use arrays in
    Gen7+ URB code.) introduced a regression which caused us to fail to
    allocate all of our URB space.

       -         total_wants -= ds_wants;
       +         total_wants -= additional;

    The new line should have been total_wants -= wants[i].

    Fixes a large performance regression in TessMark.

    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98815
    Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
    Reviewed-by: Matt Turner <mattst88 at gmail.com>

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-3d-bugs/attachments/20161124/b8515e02/attachment.html>