[Bug 34156] r600g performance regression between 780c183b and 862ebb41

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Fri Feb 18 11:21:39 PST 2011


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

--- Comment #5 from Dave Witbrodt <dawitbro at sbcglobal.net> 2011-02-18 11:21:39 PST ---
In comment 3, I made two naive attempts to revert the problem commit.  Both
failed, but I am only able to work on this stuff in my spare time -- and I have
not been able to look at this more carefully until today.

I discovered that several commits following the problem commit touch the same
code in 'src/gallium/drivers/r600/r600_shader.c', so today I decided to create
a branch from Mesa master at the last of those commits, revert them in reverse
order, then merge master onto the branch.  I expected to have the benefits of
code added since Feb. 7 without the performance regression identified in this
bug.

The commits involved were:

    $ git log a77e813d^..HEAD src/gallium/drivers/r600/r600_shader.c

    commit 077c448d184799e0d9ec962013ec784c6a5c1807
    Author: Henri Verbeet <hverbeet at gmail.com>
    Date:   Mon Feb 7 15:22:08 2011 +0100

        r600g: Add support for relative addressing on constant buffers.

    commit 7687eabaa0470261e059a2d6502628fffd209345
    Author: Henri Verbeet <hverbeet at gmail.com>
    Date:   Mon Feb 7 15:22:07 2011 +0100

        r600g: Split constants in r600_shader_from_tgsi().

    commit 1fa95c7f9e7f1b63364b1f9c6289690418cf6313
    Author: Henri Verbeet <hverbeet at gmail.com>
    Date:   Mon Feb 7 15:22:07 2011 +0100

        r600g: Do the tgsi_full_src_register to r600_shader_src conversion in
    r600_s

    commit a77e813de32643ae2dfffd7ad12abed596172cab
    Author: Henri Verbeet <hverbeet at gmail.com>
    Date:   Mon Feb 7 15:22:07 2011 +0100

        r600g: Split r600_bc_alu_src.

I branched from 077c448d, reverted each of these, then merged master onto the
branch.  It merged cleanly.

Unfortunately, this did not restore performance.  From that, I am concluding
that later code changes introduced at least one other regression.  I will be
busy with other tasks for the rest of the day, but tomorrow I will try some
more involved attempts to find out what is going on.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the dri-devel mailing list