Re: [Intel-gfx] [PATCH] drm/i915/cnl: Gen10 render context size.

2017-07-14 Thread Daniele Ceraolo Spurio
Hi, I'm a bit late to the party, but as I already mentioned at [1] the CNL render ctx size is smaller than the gen9 one according to the specs. The ctx format in the specs contains 16752 dwords, i.e. 17 pages, to which we need to add 1 page for the PPHWSP, so 18 pages total. If we re-use the

Re: [Intel-gfx] [PATCH] drm/i915/cnl: Gen10 render context size.

2017-07-07 Thread Rodrigo Vivi
patch merged to dinq. thanks for reviewing. On Thu, Jul 6, 2017 at 7:51 PM, Ben Widawsky wrote: > On 17-07-06 14:06:24, Vivi, Rodrigo wrote: >> >> No change on render context size is required for Gen10. >> >> So this patch doesn't change the default behaviour, >> but only avoid the missing_case m

Re: [Intel-gfx] [PATCH] drm/i915/cnl: Gen10 render context size.

2017-07-06 Thread Ben Widawsky
On 17-07-06 14:06:24, Vivi, Rodrigo wrote: No change on render context size is required for Gen10. So this patch doesn't change the default behaviour, but only avoid the missing_case message. Cc: Ben Widawsky Signed-off-by: Rodrigo Vivi Reviewed-by: Ben Widawsky [snip] -- Ben Widawsky,

[Intel-gfx] [PATCH] drm/i915/cnl: Gen10 render context size.

2017-07-06 Thread Rodrigo Vivi
No change on render context size is required for Gen10. So this patch doesn't change the default behaviour, but only avoid the missing_case message. Cc: Ben Widawsky Signed-off-by: Rodrigo Vivi --- drivers/gpu/drm/i915/intel_engine_cs.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drive