i965: Try not to reverse-schedule things when doing LIFO scheduling.
authorEric Anholt <eric@anholt.net>
Tue, 22 Oct 2013 19:27:06 +0000 (12:27 -0700)
committerEric Anholt <eric@anholt.net>
Fri, 25 Oct 2013 23:45:30 +0000 (16:45 -0700)
commitc3c9a8c85758796a26b48e484286e6b6f5a5299a
treefc60f6acd8ac69450039872b84c10a34ef736813
parenta7ce1fef27ccfe388b4c11017f64f4fe83ca6f6d
i965: Try not to reverse-schedule things when doing LIFO scheduling.

The LIFO plan was simple: Take the most recently made available
instructions, and pick those first.

But because of the order we were pushing things onto our list of
available-to-schedule instructions, it meant that when a set of
instructions was made available at the same time (for example, everything
at the start of the program that didn't depend on other instructions) we'd
schedule them in reverse order.

If you had 10 texture calls in a row in your program, each with
independent argument setup, we'd set up the last texture call's args and
execute it first, even though we wouldn't be able to consume its results
until we'd finished the other 9 texture calls (assuming consumption of
texture results happens near each texture call, and combines it with
another texture result, which is normal for a convolution shader).

To fix this, walk the list for doing LIFO in the order that instructions
were originally generated in the program, but choose to push
newly-made-available instructions to the other end of the list instead.

total instructions in shared programs: 1587242 -> 1586290 (-0.06%)
instructions in affected programs:     7801 -> 6849 (-12.20%)
GAINED:                                76
LOST:                                  67

Thanks to Chia-I Wu for pointing out the bug in my first version of the
patch that made it a huge loss.

Reviewed-by: Matt Turner <mattst88@gmail.com>
src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp