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@xxxxxxxxx> 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@xxxxxxxxx>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
Reviewed-by: Ben Widawsky <ben@xxxxxxxxxxxx>
[snip]
--
Ben Widawsky, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
