Re: [Mesa-dev] [PATCH 3/8] i965: Store the atoms directly in the context

2014-12-22 Thread Jason Ekstrand
On Dec 22, 2014 2:49 PM, "Ian Romanick" wrote: > > On 12/19/2014 06:20 PM, Jason Ekstrand wrote: > > I wrote basically this same patch when I was trying to speed up > > basically the same test. Unfortunately, I wasn't able to get much of an > > improvement out of it. Other than the one nit below

Re: [Mesa-dev] [PATCH 3/8] i965: Store the atoms directly in the context

2014-12-22 Thread Ian Romanick
On 12/19/2014 06:20 PM, Jason Ekstrand wrote: > I wrote basically this same patch when I was trying to speed up > basically the same test. Unfortunately, I wasn't able to get much of an > improvement out of it. Other than the one nit below, > Reviewed-by: Jason Ekstrand

Re: [Mesa-dev] [PATCH 3/8] i965: Store the atoms directly in the context

2014-12-22 Thread Ian Romanick
On 12/21/2014 09:11 PM, Kenneth Graunke wrote: > On Friday, December 19, 2014 02:20:54 PM Ian Romanick wrote: >> From: Ian Romanick >> >> Instead of having an extra pointer indirection in one of the hottest >> loops in the driver. >> >> On Bay Trail-D using Fedora 20 compile flags (-m64 -O2 -mtune

Re: [Mesa-dev] [PATCH 3/8] i965: Store the atoms directly in the context

2014-12-21 Thread Kenneth Graunke
On Sunday, December 21, 2014 09:19:53 PM Jason Ekstrand wrote: > On Sun, Dec 21, 2014 at 9:11 PM, Kenneth Graunke > wrote: > > > > On Friday, December 19, 2014 02:20:54 PM Ian Romanick wrote: > > > From: Ian Romanick > > > > > > Instead of having an extra pointer indirection in one of the hottest

Re: [Mesa-dev] [PATCH 3/8] i965: Store the atoms directly in the context

2014-12-21 Thread Jason Ekstrand
On Sun, Dec 21, 2014 at 9:11 PM, Kenneth Graunke wrote: > > On Friday, December 19, 2014 02:20:54 PM Ian Romanick wrote: > > From: Ian Romanick > > > > Instead of having an extra pointer indirection in one of the hottest > > loops in the driver. > > > > On Bay Trail-D using Fedora 20 compile flag

Re: [Mesa-dev] [PATCH 3/8] i965: Store the atoms directly in the context

2014-12-21 Thread Kenneth Graunke
On Friday, December 19, 2014 02:20:54 PM Ian Romanick wrote: > From: Ian Romanick > > Instead of having an extra pointer indirection in one of the hottest > loops in the driver. > > On Bay Trail-D using Fedora 20 compile flags (-m64 -O2 -mtune=generic > for 64-bit and -m32 -march=i686 -mtune=ato

Re: [Mesa-dev] [PATCH 3/8] i965: Store the atoms directly in the context

2014-12-19 Thread Jason Ekstrand
I wrote basically this same patch when I was trying to speed up basically the same test. Unfortunately, I wasn't able to get much of an improvement out of it. Other than the one nit below, Reviewed-by: Jason Ekstrand On Fri, Dec 19, 2014 at 2:20 PM, Ian Romanick wrote: > > From: Ian Romanick

[Mesa-dev] [PATCH 3/8] i965: Store the atoms directly in the context

2014-12-19 Thread Ian Romanick
From: Ian Romanick Instead of having an extra pointer indirection in one of the hottest loops in the driver. On Bay Trail-D using Fedora 20 compile flags (-m64 -O2 -mtune=generic for 64-bit and -m32 -march=i686 -mtune=atom for 32-bit), affects Gl32Batch7: 32-bit: Difference at 95.0% confidence