Re: [Intel-gfx] [PATCH 2/4] drm/i915/skl+: calculate ddb minimum allocation (v6)

2016-06-01 Thread Matt Roper
On Wed, Jun 01, 2016 at 07:55:18PM +0530, Mahesh Kumar wrote: > Reviewed-by: Kumar Mahesh Merged to dinq; thanks for the review (and thanks for doing the initial work this series was based on). Matt > > On Tuesday 31 May 2016 10:28 PM, Matt Roper wrote: > >From: "Kumar, Mahesh" > > > >don't

Re: [Intel-gfx] [PATCH 2/4] drm/i915/skl+: calculate ddb minimum allocation (v6)

2016-06-01 Thread Mahesh Kumar
Reviewed-by: Kumar Mahesh On Tuesday 31 May 2016 10:28 PM, Matt Roper wrote: From: "Kumar, Mahesh" don't always use 8 ddb as minimum, instead calculate using proper algorithm. v2: optimizations as per Matt's comments. v3 (by Matt): - Fix boolean logic for !fb test in skl_ddb_min_alloc()

[Intel-gfx] [PATCH 2/4] drm/i915/skl+: calculate ddb minimum allocation (v6)

2016-05-31 Thread Matt Roper
From: "Kumar, Mahesh" don't always use 8 ddb as minimum, instead calculate using proper algorithm. v2: optimizations as per Matt's comments. v3 (by Matt): - Fix boolean logic for !fb test in skl_ddb_min_alloc() - Adjust negative tiling format comparisons in skl_ddb_min_alloc() to improve r