Re: [igt-dev] [PATCH i-g-t 1/3] lib/igt_power: Modify igt_power library for xe

[Riana Tauro <riana.tauro@intel.com>]

Hi Kamil

On 8/18/2023 1:30 PM, Kamil Konieczny wrote:
> Hi Riana,
> 
> On 2023-08-17 at 11:22:59 +0530, Riana Tauro wrote:
>> modify igt_pm library for xe tests. To check if a
>    ^
> Start sentence from uppercase, s/modify/Modify/
> 
>> device is dgfx the igt_power open call needs to be modified for
>> xe.
>>
>> Signed-off-by: Riana Tauro <riana.tauro@intel.com>
>> ---
>>   lib/igt_power.c | 5 ++++-
>>   1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/lib/igt_power.c b/lib/igt_power.c
>> index 3b34be406..439d49fc3 100644
>> --- a/lib/igt_power.c
>> +++ b/lib/igt_power.c
>> @@ -12,6 +12,7 @@
>>   #include "igt_power.h"
>>   #include "igt_sysfs.h"
>>   
>> +#include "xe/xe_query.h"
> 
> Add newline.

Thanks for the review. Will fix these

Thanks
Riana Tauro
> 
> With this fixed,
> Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
> 
>>   static const char *rapl_domains[] = { "cpu", "gpu", "pkg", "ram" };
>>   
>>   static int rapl_parse(struct rapl *r, const char *str)
>> @@ -99,11 +100,13 @@ static inline void rapl_close(struct rapl *r)
>>   int igt_power_open(int fd, struct igt_power *p, const char *domain)
>>   {
>>   	int i;
>> +	bool is_dgfx;
>>   
>>   	p->hwmon_fd = -1;
>>   	p->rapl.fd = -1;
>>   
>> -	if (gem_has_lmem(fd)) {
>> +	is_dgfx = is_xe_device(fd) ? xe_has_vram(fd) : gem_has_lmem(fd);
>> +	if (is_dgfx) {
>>   		if (strncmp(domain, "gpu", strlen("gpu")) == 0) {
>>   			p->hwmon_fd = igt_hwmon_open(fd);
>>   			if (p->hwmon_fd >= 0)
>> -- 
>> 2.40.0
>>