Re: [igt-dev] [PATCH i-g-t 1/3] lib/igt_gt: Add a method to detect ggtt bind

[Nirmoy Das <nirmoy.das@linux.intel.com>]

Hi Kamil,

On 9/14/2023 3:31 PM, Kamil Konieczny wrote:
> Hi Nirmoy,
>
> On 2023-09-13 at 11:42:50 +0200, Nirmoy Das wrote:
>> On MTL GGTT updates happens through MI_UPDATE_GGTT command.
>> Add a method to detect that.
>>
>> Signed-off-by: Nirmoy Das <nirmoy.das@intel.com>
>> ---
>>   lib/igt_gt.c | 5 +++++
>>   lib/igt_gt.h | 1 +
>>   2 files changed, 6 insertions(+)
>>
>> diff --git a/lib/igt_gt.c b/lib/igt_gt.c
>> index a24a566c7..6895964a9 100644
>> --- a/lib/igt_gt.c
>> +++ b/lib/igt_gt.c
>> @@ -659,3 +659,8 @@ int gem_execbuf_flags_to_engine_class(unsigned int flags)
>>   		igt_assert(0);
>>   	}
>>   }
>> +
> Add description to public function.

Makes sense, will resend with doc.


Nirmoy

>
> Regards,
> Kamil
>
>> +bool gem_has_ggtt_bind(int fd)
>> +{
>> +	return IS_METEORLAKE(intel_get_drm_devid(fd));
>> +}
>> diff --git a/lib/igt_gt.h b/lib/igt_gt.h
>> index 3d10349e4..7d35a209d 100644
>> --- a/lib/igt_gt.h
>> +++ b/lib/igt_gt.h
>> @@ -83,5 +83,6 @@ extern const struct intel_execution_engine2 {
>>   } intel_execution_engines2[];
>>   
>>   int gem_execbuf_flags_to_engine_class(unsigned int flags);
>> +bool gem_has_ggtt_bind(int fd);
>>   
>>   #endif /* IGT_GT_H */
>> -- 
>> 2.41.0
>>