From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by gabe.freedesktop.org (Postfix) with ESMTPS id 9631510E72C for ; Mon, 10 Oct 2022 21:42:19 +0000 (UTC) From: Umesh Nerlige Ramappa To: igt-dev@lists.freedesktop.org, Lionel G Landwerlin , Ashutosh Dixit Date: Mon, 10 Oct 2022 21:41:52 +0000 Message-Id: <20221010214215.5378-14-umesh.nerlige.ramappa@intel.com> In-Reply-To: <20221010214215.5378-1-umesh.nerlige.ramappa@intel.com> References: <20221010214215.5378-1-umesh.nerlige.ramappa@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t v6 13/36] i915/perf: Wait longer for rc6 residency in DG2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: igt@perf@rc6-disable waits for 50ms to check if we entered rc6. One some failures this seems to take longer to enter rc6, about 100ms. Tweak the test for DG2 to use a longer wait time. Signed-off-by: Umesh Nerlige Ramappa Acked-by: Lionel Landwerlin --- tests/i915/perf.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/tests/i915/perf.c b/tests/i915/perf.c index befae0da..7045c214 100644 --- a/tests/i915/perf.c +++ b/tests/i915/perf.c @@ -4272,7 +4272,10 @@ test_rc6_disable(void) .properties_ptr = to_user_pointer(properties), }; unsigned long rc6_start, rc6_end, rc6_enabled; + uint32_t rc6_duration; + /* On DG2, we wait longer to enter rc6 */ + rc6_duration = IS_DG2(devid) ? 100000 : 50000; rc6_enabled = 0; igt_sysfs_rps_scanf(sysfs, RC6_ENABLE, "%lu", &rc6_enabled); igt_require(rc6_enabled); @@ -4280,7 +4283,7 @@ test_rc6_disable(void) /* Verify rc6 is functional by measuring residency while idle */ gem_quiescent_gpu(drm_fd); rc6_start = rc6_residency_ms(); - usleep(50000); + usleep(rc6_duration); rc6_end = rc6_residency_ms(); igt_require(rc6_end != rc6_start); @@ -4288,7 +4291,7 @@ test_rc6_disable(void) stream_fd = __perf_open(drm_fd, ¶m, false); rc6_start = rc6_residency_ms(); - usleep(50000); + usleep(rc6_duration); rc6_end = rc6_residency_ms(); igt_assert_eq(rc6_end - rc6_start, 0); @@ -4297,7 +4300,7 @@ test_rc6_disable(void) /* But once OA is closed, we expect the device to sleep again */ rc6_start = rc6_residency_ms(); - usleep(50000); + usleep(rc6_duration); rc6_end = rc6_residency_ms(); igt_assert_neq(rc6_end - rc6_start, 0); } -- 2.25.1