[igt-dev] [PATCH i-g-t v2 4/4] tests/xe: Add a test that validates residency during s2idle

[Gupta, Anshuman]

> -----Original Message-----
> From: Tauro, Riana <riana.tauro at intel.com>
> Sent: Friday, August 4, 2023 4:01 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Tauro, Riana <riana.tauro at intel.com>; Gupta, Anshuman
> <anshuman.gupta at intel.com>; Dixit, Ashutosh <ashutosh.dixit at intel.com>;
> Belgaumkar, Vinay <vinay.belgaumkar at intel.com>; Nilawar, Badal
> <badal.nilawar at intel.com>; Sundaresan, Sujaritha
> <sujaritha.sundaresan at intel.com>
> Subject: [PATCH i-g-t v2 4/4] tests/xe: Add a test that validates residency during
> s2idle
> 
> During s2idle, the device is powered up and GT C6 enabled so we expect the
> residency to increase across suspend.
> Add a test that validates the residency over a s2idle cycle is within the threshold.
> 
> v2: skip d3cold disable for igfx (Anshuman)
> 
> Signed-off-by: Riana Tauro <riana.tauro at intel.com>
> ---
>  tests/xe/xe_pm_residency.c | 56 ++++++++++++++++++++++++++++++++++----
>  1 file changed, 51 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/xe/xe_pm_residency.c b/tests/xe/xe_pm_residency.c index
> 4936de166..78d820aee 100644
> --- a/tests/xe/xe_pm_residency.c
> +++ b/tests/xe/xe_pm_residency.c
> @@ -10,8 +10,10 @@
>   * Functionality: GT C States
>   * Test category: functionality test
>   */
> +#include <limits.h>
> 
>  #include "igt.h"
> +#include "igt_device.h"
>  #include "igt_sysfs.h"
> 
>  #include "xe/xe_query.h"
> @@ -29,6 +31,11 @@ const double tolerance = 0.1;
>  		     (tol) * 100.0, (tol) * 100.0, \
>  		     (double)(ref))
> 
> +enum test_type {
> +	TEST_S2IDLE,
> +	TEST_SLEEP,
How about TEST_IDLE ?
> +};
> +
>  /**
>   * SUBTEST: gt-c6-on-idle
>   * Description: Validate GT C6 state on idle @@ -38,9 +45,22 @@ const double
> tolerance = 0.1;
>   * Description: basic residency test to validate idle residency
>   *		measured over a time interval is within the tolerance
>   * Run type: FULL
> + *
> + * SUBTEST: gt-c6-freeze
> + * Description: Validate idle residency measured over suspend(s2idle)
> + *              is within the tolerance
> + * Run type: FULL
>   */
>  IGT_TEST_DESCRIPTION("Tests for gtidle properties");
> 
> +static unsigned long walltime_ms(void)
> +{
> +	struct timespec ts;
> +
> +	clock_gettime(CLOCK_REALTIME, &ts);
> +	return ts.tv_sec * 1000 + ts.tv_nsec / 1000000; }
> +
>  static unsigned int measured_usleep(unsigned int usec)  {
>  	struct timespec ts = { };
> @@ -69,15 +89,25 @@ static unsigned long read_idle_residency(int fd, int gt)
>  	return residency;
>  }
> 
> -static void test_idle_residency(int fd, int gt)
> +static void test_idle_residency(int fd, int gt, enum test_type flag)
>  {
>  	unsigned long elapsed_ms, residency_start, residency_end;
> 
>  	igt_assert_f(igt_wait(xe_is_gt_in_c6(fd, gt), 1000, 1), "GT not in C6\n");
> 
> -	residency_start = read_idle_residency(fd, gt);
> -	elapsed_ms = measured_usleep(SLEEP_DURATION * 1000) / 1000;
> -	residency_end = read_idle_residency(fd, gt);
> +	if (flag == TEST_S2IDLE) {
> +		residency_start = read_idle_residency(fd, gt);
> +		elapsed_ms -= walltime_ms();
> +		igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE,
> SUSPEND_TEST_NONE);
> +		elapsed_ms += walltime_ms();
> +		residency_end = read_idle_residency(fd, gt);
> +	}
> +
> +	if (flag == TEST_SLEEP) {
> +		residency_start = read_idle_residency(fd, gt);
> +		elapsed_ms = measured_usleep(SLEEP_DURATION * 1000) /
> 1000;
> +		residency_end = read_idle_residency(fd, gt);
> +	}
> 
>  	igt_info("Measured %lums of idle residency in %lums\n",
>  		 residency_end - residency_start, elapsed_ms); @@ -88,6
> +118,8 @@ static void test_idle_residency(int fd, int gt)  igt_main  {
>  	int fd, gt;
Nit: Declare fd and gt after d3cold_allowed.
Thanks,
Anshuman. 
> +	char pci_slot_name[NAME_MAX];
> +	uint32_t d3cold_allowed;
> 
>  	igt_fixture {
>  		fd = drm_open_driver(DRIVER_XE);
> @@ -99,10 +131,24 @@ igt_main
>  		xe_for_each_gt(fd, gt)
>  			igt_assert_f(igt_wait(xe_is_gt_in_c6(fd, gt), 1000, 1),
> "GT not in C6\n");
> 
> +	igt_describe("Validate idle residency measured over suspend cycle is
> within the tolerance");
> +	igt_subtest("gt-c6-freeze") {
> +		if (xe_has_vram(fd)) {
> +			igt_device_get_pci_slot_name(fd, pci_slot_name);
> +			igt_pm_get_d3cold_allowed(pci_slot_name,
> &d3cold_allowed);
> +			igt_pm_set_d3cold_allowed(pci_slot_name, 0);
> +		}
> +		xe_for_each_gt(fd, gt)
> +			test_idle_residency(fd, gt, TEST_S2IDLE);
> +
> +		if (xe_has_vram(fd))
> +			igt_pm_set_d3cold_allowed(pci_slot_name,
> d3cold_allowed);
> +	}
> +
>  	igt_describe("Validate idle residency measured over a time interval is
> within the tolerance");
>  	igt_subtest("idle-residency")
>  		xe_for_each_gt(fd, gt)
> -			test_idle_residency(fd, gt);
> +			test_idle_residency(fd, gt, TEST_SLEEP);
> 
>  	igt_fixture {
>  		close(fd);
> --
> 2.40.0