Re: [igt-dev] [i-g-t] i915/gem_exec_fair: Extend icl ctx timestamp w/a to gen11

[Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>]

On 25/06/2021 14:03, Tvrtko Ursulin wrote:
> 
> On 24/06/2021 13:22, Tejas Upadhyay wrote:
>> 80ns timestamp interval identified for icl also applies to
>> gen11 platforms. igt@gem_exec_fair@basic-deadline is
>> passing with the change on ehl and jsl.
>>
>> Signed-off-by: Tejas Upadhyay 
>> <tejaskumarx.surendrakumar.upadhyay@intel.com>
>> ---
>>   tests/i915/gem_exec_fair.c | 9 ++-------
>>   1 file changed, 2 insertions(+), 7 deletions(-)
>>
>> diff --git a/tests/i915/gem_exec_fair.c b/tests/i915/gem_exec_fair.c
>> index 628f4df2..45237574 100644
>> --- a/tests/i915/gem_exec_fair.c
>> +++ b/tests/i915/gem_exec_fair.c
>> @@ -98,16 +98,11 @@ static uint64_t div64_u64_round_up(uint64_t x, 
>> uint64_t y)
>>       return (x + y - 1) / y;
>>   }
>> -static bool is_icelake(int i915)
>> -{
>> -    return intel_get_device_info(intel_get_drm_devid(i915))->is_icelake;
>> -}
>> -
>>   static uint64_t ns_to_ctx_ticks(int i915, uint64_t ns)
>>   {
>>       int f = read_timestamp_frequency(i915);
>> -    if (is_icelake(i915))
>> -        f = 12500000; /* icl!!! are you feeling alright? CTX vs CS */
>> +    if (intel_gen(intel_get_drm_devid(i915)) == 11)
>> +        f = 12500000; /* gen11!!! are you feeling alright? CTX vs CS */
>>       return div64_u64_round_up(ns * f, NSEC64);
>>   }
>>
> 
> Same as the kernel patch, okay, I can push this one as well.

Okay CI results are not yet in.. please ping me or someone when they arrive.

Regards,

Tvrtko

> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> 
> Regards,
> 
> Tvrtko
> _______________________________________________
> igt-dev mailing list
> igt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev