On Sat, Oct 3, 2015 at 11:26 AM, Ilia Mirkin <imir...@alum.mit.edu> wrote: > On Sat, Oct 3, 2015 at 2:13 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote: >> On Fri, Oct 2, 2015 at 2:37 PM, Connor Abbott <cwabbo...@gmail.com> wrote: >>> Before, we would only do scheduling after register allocation if we >>> spilled, despite the fact that the pre-RA scheduler was only supposed to >>> be for register pressure and set the latencies of every instruction to >>> 1. This meant that unless we spilled, which we rarely do, then we never >>> considered instruction latencies at all, and we usually never bothered >>> to try and hide texture fetch latency. Although a later commit removes >>> the setting the latency to 1 part, we still want to always run the >>> post-RA scheduler since it's able to take the false dependencies that >>> the register allocator creates into account, and it can be more >>> aggressive than the pre-RA scheduler since it doesn't have to worry >>> about register pressure at all. >>> >>> XXX perf data >> >> Test master post-ra-sched diff %diff >> bench_OglPSBump2 396.730 402.386 5.656 +1.400% >> bench_OglPSBump8 244.370 247.591 3.221 +1.300% >> bench_OglPSPhong 241.117 242.002 0.885 +0.300% >> bench_OglPSPom 59.555 59.725 0.170 +0.200% >> bench_OglShMapPcf 86.149 102.346 16.197 +18.800% >> bench_OglVSTangent 388.849 395.489 6.640 +1.700% >> bench_trex 65.471 65.862 0.390 +0.500% >> bench_trexoff 69.562 70.150 0.588 +0.800% >> >> Unfortunately, neither of the unigin benchmarks (heaven or vally) >> seemed to render correctly. I just got white on both master and your >> branch. Not sure if we have a bug or if they just weren't running >> right. In any case, ministat didn't notice any difference in them. > > I believe they're called "features" :) Try with > disable_blend_func_extended=true
I pulled in a more recent drirc and am re-running those two. >> >>> Signed-off-by: Connor Abbott <cwabbo...@gmail.com> >>> --- >>> src/mesa/drivers/dri/i965/brw_fs.cpp | 3 +-- >>> 1 file changed, 1 insertion(+), 2 deletions(-) >>> >>> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp >>> b/src/mesa/drivers/dri/i965/brw_fs.cpp >>> index b269ade..14a9fdf 100644 >>> --- a/src/mesa/drivers/dri/i965/brw_fs.cpp >>> +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp >>> @@ -4981,8 +4981,7 @@ fs_visitor::allocate_registers() >>> if (failed) >>> return; >>> >>> - if (!allocated_without_spills) >>> - schedule_instructions(SCHEDULE_POST); >>> + schedule_instructions(SCHEDULE_POST); >>> >>> if (last_scratch > 0) >>> prog_data->total_scratch = brw_get_scratch_size(last_scratch); >>> -- >>> 2.1.0 >>> >>> _______________________________________________ >>> mesa-dev mailing list >>> mesa-dev@lists.freedesktop.org >>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev >> _______________________________________________ >> mesa-dev mailing list >> mesa-dev@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev