Re: [PATCH v2 1/2] tests/intel/xe_gt_freq: Modify test to support new throttle reasons sysfs

[Raag Jadav <raag.jadav@intel.com>]

On Thu, Dec 04, 2025 at 11:54:30AM +0530, Sk Anirban wrote:
> Change throttle_basic_api to read the new atomic reasons attribute
> instead of checking status and individual reason files separately.
> This validates the kernel's TOCTOU-free throttling interface.

...

> +	char *throttle_reasons;
> +	int sysfs_fd;
> +
> +	sysfs_fd = xe_sysfs_gt_open(fd, gt_id);

This is actually gt_open so let's use gt_fd for consistency.

> +	igt_assert_lte(0, sysfs_fd);
> +
> +	throttle_reasons = igt_sysfs_get(sysfs_fd, "freq0/throttle/reasons");
> +	igt_assert(throttle_reasons);
> +
> +	igt_debug("gt%d/freq0/throttle/reasons: %s\n", gt_id, throttle_reasons);
> +
> +	if (strcmp(throttle_reasons, "none"))
> +		igt_info("GT %d is being throttled due to reasons: %s\n", gt_id, throttle_reasons);

I'm wondering if we should print the results explicitly and drop the
duplicate igt_debug() above? I'll let you decide which message to use :)

Raag

> +	free(throttle_reasons);
> +	close(sysfs_fd);
>  }
>  
>  /**
> -- 
> 2.43.0
>