[Intel-gfx] [PATCH] drm/i915: Use true PPGTT in Gen8 if execlists are enabled

[Michel Thierry]

On 12/15/2014 10:11 AM, Daniel Vetter wrote:
> On Thu, Dec 11, 2014 at 12:07:18PM +0000, Michel Thierry wrote:
>> When execlists submission is enabled, try full ppgtt by default.
>>
>> Note, this patch considers that execlist support has been enabled by
>> default on Gen8.
>>
>> Signed-off-by: Michel Thierry <michel.thierry at intel.com>
>> ---
>>   drivers/gpu/drm/i915/i915_gem_gtt.c | 9 ++++++---
>>   1 file changed, 6 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
>> index 171f6ea..4ed3904 100644
>> --- a/drivers/gpu/drm/i915/i915_gem_gtt.c
>> +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
>> @@ -40,8 +40,8 @@ static int sanitize_enable_ppgtt(struct drm_device *dev, int enable_ppgtt)
>>   
>>   	has_aliasing_ppgtt = INTEL_INFO(dev)->gen >= 6;
>>   	has_full_ppgtt = INTEL_INFO(dev)->gen >= 7;
>> -	if (IS_GEN8(dev))
>> -		has_full_ppgtt = false; /* XXX why? */
>> +	if (IS_GEN8(dev) && !i915.enable_execlists)
>> +		has_full_ppgtt = false; /* Only enforce with execlists */
> Imo this has outlived it's usefulness - enable_ppgtt is an unsafe
> parameter so everyone setting it themselves gets what they need.
> Afair this was just because of the execlist depency on gen8 for ppgtt.
> -Daniel
Not sure if I'm following you up on this...
The aim was to change the default value to _full_ only when execlists 
are also enable (after Thomas' patch).  In gen8, we don't want to have 
full ppgtt with legacy ring submission.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 5510 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20141215/a0daa67c/attachment-0001.bin>