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

[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 gen9 size we end up wasting 4 pages per lrc.

I don't have access to a CNL to test this but I've been told that a 
setup using 19 pages (1 more for safety) has been tested successfully on 
CNL.

-Daniele

[1] https://lists.freedesktop.org/archives/intel-gfx/2017-April/125321.html

On 07/07/17 09:38, Rodrigo Vivi wrote:
> patch merged to dinq. thanks for reviewing.
> 
> On Thu, Jul 6, 2017 at 7:51 PM, Ben Widawsky
> <benjamin.widawsky at intel.com> 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 message.
>>>
>>> Cc: Ben Widawsky <benjamin.widawsky at intel.com>
>>> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
>>
>>
>> Reviewed-by: Ben Widawsky <ben at bwidawsk.net>
>>
>> [snip]
>>
>>
>> --
>> Ben Widawsky, Intel Open Source Technology Center
>>
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
> 
>