[Intel-gfx] [PATCH] drm/i915: Let execlist_update_context() cover !FULL_PPGTT mode.

Zhi Wang zhi.a.wang at intel.com
Mon Feb 6 11:03:39 UTC 2017


Thanks for the merging. I guess that with your patches of keeping PD 
structure under aliasing PPGTT mode in 32bit page table, the amount of 
PDPs will not change anymore under aliasing PPGTT mode. :P

On 02/06/17 18:58, Chris Wilson wrote:
> On Mon, Feb 06, 2017 at 06:37:16PM +0800, Zhi Wang wrote:
>> execlist_update_context() will try to update PDPs in a context before a
>> ELSP submission only for full PPGTT mode, while PDPs was populated during
>> context initialization. Now the latter code path is removed. Let
>> execlist_update_context() also cover !FULL_PPGTT mode.
>>
>> Cc: Tvrtko Ursulin <tvrtko.ursulin at linux.intel.com>
>> Cc: Michal Winiarski <michal.winiarski at intel.com>
>> Cc: Michel Thierry <michel.thierry at intel.com>
>> Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
>> Cc: Chris Wilson <chris at chris-wilson.co.uk>
>> Cc: Zhenyu Wang <zhenyuw at linux.intel.com>
>> Cc: Zhiyuan Lv <zhiyuan.lv at intel.com>
>> Signed-off-by: Zhi Wang <zhi.a.wang at intel.com>
>
> My mistake from earlier missing the aliasing case.
>
> Fixes: 34869776c76b ("drm/i915: check ppgtt validity when init reg state")
> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
> -Chris
>


More information about the Intel-gfx mailing list