Re: [Mesa-dev] [PATCH 5/5] anv/pipeline: Be smarter about depth/stencil state

2017-02-10 Thread Nanley Chery
On Thu, Feb 09, 2017 at 05:32:55PM -0800, Nanley Chery wrote: > On Wed, Feb 01, 2017 at 05:43:55PM -0800, Jason Ekstrand wrote: > > This seemed to help Dota 2 by a percent or two. > > I wasn't able to reproduce this on SKL, could you qualify this with "on > BDW"? > > > --- > > src/intel/vulkan/g

Re: [Mesa-dev] [PATCH 5/5] anv/pipeline: Be smarter about depth/stencil state

2017-02-10 Thread Nanley Chery
On Fri, Feb 10, 2017 at 10:59:38AM -0800, Jason Ekstrand wrote: > On Thu, Feb 9, 2017 at 5:32 PM, Nanley Chery wrote: > > > On Wed, Feb 01, 2017 at 05:43:55PM -0800, Jason Ekstrand wrote: > > > This seemed to help Dota 2 by a percent or two. > > > > I wasn't able to reproduce this on SKL, could y

Re: [Mesa-dev] [PATCH 5/5] anv/pipeline: Be smarter about depth/stencil state

2017-02-10 Thread Nanley Chery
On Fri, Feb 10, 2017 at 10:59:38AM -0800, Jason Ekstrand wrote: > On Thu, Feb 9, 2017 at 5:32 PM, Nanley Chery wrote: > > > On Wed, Feb 01, 2017 at 05:43:55PM -0800, Jason Ekstrand wrote: > > > This seemed to help Dota 2 by a percent or two. > > > > I wasn't able to reproduce this on SKL, could y

Re: [Mesa-dev] [PATCH 5/5] anv/pipeline: Be smarter about depth/stencil state

2017-02-10 Thread Jason Ekstrand
On Thu, Feb 9, 2017 at 5:32 PM, Nanley Chery wrote: > On Wed, Feb 01, 2017 at 05:43:55PM -0800, Jason Ekstrand wrote: > > This seemed to help Dota 2 by a percent or two. > > I wasn't able to reproduce this on SKL, could you qualify this with "on > BDW"? > I've changed it to: It's a bit hard

Re: [Mesa-dev] [PATCH 5/5] anv/pipeline: Be smarter about depth/stencil state

2017-02-09 Thread Nanley Chery
On Wed, Feb 01, 2017 at 05:43:55PM -0800, Jason Ekstrand wrote: > This seemed to help Dota 2 by a percent or two. I wasn't able to reproduce this on SKL, could you qualify this with "on BDW"? > --- > src/intel/vulkan/genX_pipeline.c | 133 > +-- > 1 file chan

Re: [Mesa-dev] [PATCH 5/5] anv/pipeline: Be smarter about depth/stencil state

2017-02-02 Thread Lionel Landwerlin
Thanks, that's pretty clean :) Reviewed-by: Lionel Landwerlin On 02/02/17 01:43, Jason Ekstrand wrote: This seemed to help Dota 2 by a percent or two. --- src/intel/vulkan/genX_pipeline.c | 133 +-- 1 file changed, 99 insertions(+), 34 deletions(-) diff

[Mesa-dev] [PATCH 5/5] anv/pipeline: Be smarter about depth/stencil state

2017-02-01 Thread Jason Ekstrand
This seemed to help Dota 2 by a percent or two. --- src/intel/vulkan/genX_pipeline.c | 133 +-- 1 file changed, 99 insertions(+), 34 deletions(-) diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c index f6940d2..1961874 100644 ---