Re: [Mesa-dev] [PATCH 3/6] i965: always run the post-RA scheduler

2015-10-21 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand Let's add the perf numbers and get this pushed. On Sat, Oct 3, 2015 at 6:13 PM, Jason Ekstrand wrote: > On Sat, Oct 3, 2015 at 11:13 AM, Jason Ekstrand wrote: >> On Fri, Oct 2, 2015 at 2:37 PM, Connor Abbott wrote: >>> Before, we would only do scheduling after regi

Re: [Mesa-dev] [PATCH 3/6] i965: always run the post-RA scheduler

2015-10-03 Thread Jason Ekstrand
On Sat, Oct 3, 2015 at 11:13 AM, Jason Ekstrand wrote: > On Fri, Oct 2, 2015 at 2:37 PM, Connor Abbott 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

Re: [Mesa-dev] [PATCH 3/6] i965: always run the post-RA scheduler

2015-10-03 Thread Ilia Mirkin
On Sat, Oct 3, 2015 at 2:38 PM, Jason Ekstrand wrote: > On Sat, Oct 3, 2015 at 11:32 AM, Ilia Mirkin wrote: >> On Sat, Oct 3, 2015 at 2:28 PM, Jason Ekstrand wrote: >>> On Sat, Oct 3, 2015 at 11:26 AM, Ilia Mirkin wrote: On Sat, Oct 3, 2015 at 2:13 PM, Jason Ekstrand wrote: > On

Re: [Mesa-dev] [PATCH 3/6] i965: always run the post-RA scheduler

2015-10-03 Thread Jason Ekstrand
On Sat, Oct 3, 2015 at 11:32 AM, Ilia Mirkin wrote: > On Sat, Oct 3, 2015 at 2:28 PM, Jason Ekstrand wrote: >> On Sat, Oct 3, 2015 at 11:26 AM, Ilia Mirkin wrote: >>> On Sat, Oct 3, 2015 at 2:13 PM, Jason Ekstrand wrote: On Fri, Oct 2, 2015 at 2:37 PM, Connor Abbott wrote: > Before, w

Re: [Mesa-dev] [PATCH 3/6] i965: always run the post-RA scheduler

2015-10-03 Thread Ilia Mirkin
On Sat, Oct 3, 2015 at 2:28 PM, Jason Ekstrand wrote: > On Sat, Oct 3, 2015 at 11:26 AM, Ilia Mirkin wrote: >> On Sat, Oct 3, 2015 at 2:13 PM, Jason Ekstrand wrote: >>> On Fri, Oct 2, 2015 at 2:37 PM, Connor Abbott wrote: Before, we would only do scheduling after register allocation if we

Re: [Mesa-dev] [PATCH 3/6] i965: always run the post-RA scheduler

2015-10-03 Thread Jason Ekstrand
On Sat, Oct 3, 2015 at 11:26 AM, Ilia Mirkin wrote: > On Sat, Oct 3, 2015 at 2:13 PM, Jason Ekstrand wrote: >> On Fri, Oct 2, 2015 at 2:37 PM, Connor Abbott wrote: >>> Before, we would only do scheduling after register allocation if we >>> spilled, despite the fact that the pre-RA scheduler was

Re: [Mesa-dev] [PATCH 3/6] i965: always run the post-RA scheduler

2015-10-03 Thread Ilia Mirkin
On Sat, Oct 3, 2015 at 2:13 PM, Jason Ekstrand wrote: > On Fri, Oct 2, 2015 at 2:37 PM, Connor Abbott 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 l

Re: [Mesa-dev] [PATCH 3/6] i965: always run the post-RA scheduler

2015-10-03 Thread Jason Ekstrand
On Fri, Oct 2, 2015 at 2:37 PM, Connor Abbott 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

[Mesa-dev] [PATCH 3/6] i965: always run the post-RA scheduler

2015-10-02 Thread Connor Abbott
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 inst