Re: [Intel-gfx] [PATCH] drm/i915/cnl: Do not add an extra page for precaution in the Gen10 LRC size

2017-10-05 Thread Chris Wilson
Quoting Oscar Mateo (2017-10-04 16:39:52) > BSpec indicates exactly 16752 DWORDs (17 pages), plus one page for PPHWSP. > Please > notice that, when looking at the BSpec context image table, the right filter > has > to be applied (e.g. "CNL") as some rows are excluded for specific GENs. > > BSpec

[Intel-gfx] [PATCH] drm/i915/cnl: Do not add an extra page for precaution in the Gen10 LRC size

2017-10-04 Thread Oscar Mateo
BSpec indicates exactly 16752 DWORDs (17 pages), plus one page for PPHWSP. Please notice that, when looking at the BSpec context image table, the right filter has to be applied (e.g. "CNL") as some rows are excluded for specific GENs. BSpec: 1383 v2: Update count and add BSpec tag (Joonas) v3: W

Re: [Intel-gfx] [PATCH] drm/i915/cnl: Do not add an extra page for precaution in the Gen10 LRC size

2017-10-04 Thread Joonas Lahtinen
On Tue, 2017-10-03 at 08:46 -0700, Oscar Mateo wrote: > > On 10/03/2017 05:25 AM, Joonas Lahtinen wrote: > > On Mon, 2017-10-02 at 13:31 -0700, Oscar Mateo wrote: > > > BSpec indicates exactly 16750 DWORDs (17 pages), plus one page for PPHWSP. > > > > Assuming this is; > > > > Bspec: 1383 > > >

[Intel-gfx] [PATCH] drm/i915/cnl: Do not add an extra page for precaution in the Gen10 LRC size

2017-10-03 Thread Oscar Mateo
BSpec indicates exactly 16752 DWORDs (17 pages), plus one page for PPHWSP. BSpec: 1383 v2: Update count and add BSpec tag (Joonas) Suggested-by: Joonas Lahtinen Fixes: 7fd0b1a ("drm/i915/cnl: Add Gen10 LRC size") Signed-off-by: Oscar Mateo Cc: Rodrigo Vivi Cc: Daniele Ceraolo Spurio Cc: Ben

Re: [Intel-gfx] [PATCH] drm/i915/cnl: Do not add an extra page for precaution in the Gen10 LRC size

2017-10-03 Thread Oscar Mateo
On 10/03/2017 05:25 AM, Joonas Lahtinen wrote: On Mon, 2017-10-02 at 13:31 -0700, Oscar Mateo wrote: BSpec indicates exactly 16750 DWORDs (17 pages), plus one page for PPHWSP. Assuming this is; Bspec: 1383 I'm actually getting 16801 DWORDs by copy-pasting to spreadsheet and summing up. Are

Re: [Intel-gfx] [PATCH] drm/i915/cnl: Do not add an extra page for precaution in the Gen10 LRC size

2017-10-03 Thread Joonas Lahtinen
On Mon, 2017-10-02 at 13:31 -0700, Oscar Mateo wrote: > BSpec indicates exactly 16750 DWORDs (17 pages), plus one page for PPHWSP. Assuming this is; Bspec: 1383 I'm actually getting 16801 DWORDs by copy-pasting to spreadsheet and summing up. Are we looking from different places? Both lead to th

[Intel-gfx] [PATCH] drm/i915/cnl: Do not add an extra page for precaution in the Gen10 LRC size

2017-10-02 Thread Oscar Mateo
BSpec indicates exactly 16750 DWORDs (17 pages), plus one page for PPHWSP. Suggested-by: Joonas Lahtinen Fixes: 7fd0b1a ("drm/i915/cnl: Add Gen10 LRC size") Signed-off-by: Oscar Mateo Cc: Rodrigo Vivi Cc: Daniele Ceraolo Spurio Cc: Ben Widawsky --- drivers/gpu/drm/i915/intel_engine_cs.c | 2