From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 622ACC27C52 for ; Thu, 6 Jun 2024 05:59:53 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id F15FF10E545; Thu, 6 Jun 2024 05:59:52 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="ZCUZ0xMu"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.10]) by gabe.freedesktop.org (Postfix) with ESMTPS id D80E010E545 for ; Thu, 6 Jun 2024 05:59:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1717653592; x=1749189592; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=LBjrET4hBLb+kdD2f2IJ3XrD8uc4x5Qk3ogJL9LywrE=; b=ZCUZ0xMuy+Oz5tcgJMfOuCqGWr3X4M/EVcWmWdRGGnstT9TPGdYtwqZe 2fCZGpdj3axiqCNvjcx/sT9s3uMc+KNV62mCYlDQKbcYR3vDduvLQmdxb nzeW+PhbxL9BdyhsBwIcACNRKwF3GFVgdJau+ZVPDd+iL4HMCRUbeQMBs /O4DUvIdwKhZzTm4Jh6h3mk1+CB2p5XXl+9mDMloFi1DzMVAyJbVJ2kt/ yU9lGdGlU2qucBdiqmtGroLzILv4QNrK8+4tZSuTueHbZlPJfC8NVbZKI HXAU+Go/lulsbGszHpnWMaMCyg3lGJMmE2LkTlPAleXWNxsbNVcm4Tk86 Q==; X-CSE-ConnectionGUID: E4xTt6rxSXys5FS+kw+v+A== X-CSE-MsgGUID: bqf4EBjTQDecNJ3npCGnNQ== X-IronPort-AV: E=McAfee;i="6600,9927,11094"; a="25698758" X-IronPort-AV: E=Sophos;i="6.08,218,1712646000"; d="scan'208";a="25698758" Received: from fmviesa002.fm.intel.com ([10.60.135.142]) by fmvoesa104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Jun 2024 22:59:51 -0700 X-CSE-ConnectionGUID: 6EbWhHhdQ0W2Zvhnb34K9g== X-CSE-MsgGUID: ymPc4KxMRJShYQr6i5bcCw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.08,218,1712646000"; d="scan'208";a="61050731" Received: from bhanu-nuclab.iind.intel.com ([10.145.169.172]) by fmviesa002-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Jun 2024 22:59:50 -0700 From: Bhanuprakash Modem To: igt-dev@lists.freedesktop.org Cc: Bhanuprakash Modem , Manasi Navare Subject: [PATCH i-g-t 2/3] tests/kms_vrr: Fix hardcoded step size Date: Thu, 6 Jun 2024 11:22:04 +0530 Message-ID: <20240606055205.3296434-3-bhanuprakash.modem@intel.com> X-Mailer: git-send-email 2.43.2 In-Reply-To: <20240606055205.3296434-1-bhanuprakash.modem@intel.com> References: <20240606055205.3296434-1-bhanuprakash.modem@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: igt-dev@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development mailing list for IGT GPU Tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" In seamless-virtual subtest we are trying different Vrefresh values between Vmin & Vmax by incrementing the Vrefresh by by 10 Hz (hardcoded). With this approach: - We'll achieve less no. of refresh rates if VRR range is low Ex: We can achieve only 1 step between 40 to 60 Hz. - Execution time will increase in case of high VRR range. Ex: We need 30 steps to reach 60 Hz to 360 Hz. Instead of hardcoding the step size (10 Hz), allow test to execute fixed number of steps (say 5 steps to reach Vmin to Vmax) will guarantee to test (5) different refresh rates between Vmin & Vmax irrespective of the VRR range. Cc: Manasi Navare Signed-off-by: Bhanuprakash Modem --- tests/kms_vrr.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/tests/kms_vrr.c b/tests/kms_vrr.c index 9245305f8..dec49eb12 100644 --- a/tests/kms_vrr.c +++ b/tests/kms_vrr.c @@ -665,6 +665,7 @@ test_seamless_virtual_rr_basic(data_t *data, enum pipe pipe, igt_output_t *outpu uint32_t result; unsigned int vrefresh; uint64_t rate[] = {0}; + uint32_t step_size; igt_info("Use HIGH_RR Mode as default\n"); kmstest_dump_mode(&data->switch_modes[HIGH_RR_MODE]); @@ -684,8 +685,14 @@ test_seamless_virtual_rr_basic(data_t *data, enum pipe pipe, igt_output_t *outpu "Refresh rate (%u Hz) %"PRIu64"ns: Target threshold not reached, result was %u%%\n", data->switch_modes[HIGH_RR_MODE].vrefresh, rate[0], result); + /* + * Calculate step size by considering the no. of steps required to + * reach Vmin to Vmax as 5. + */ + step_size = (data->range.max - data->range.min) / 5; + /* Switch to Virtual RR */ - for (vrefresh = data->range.min + 10; vrefresh < data->range.max; vrefresh += 10) { + for (vrefresh = data->range.min + step_size; vrefresh < data->range.max; vrefresh += step_size) { drmModeModeInfo virtual_mode = virtual_rr_vrr_range_mode(output, vrefresh); igt_info("Requesting Virtual Mode with Refresh Rate (%u Hz): \n", vrefresh); -- 2.43.2