From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id 812736E821 for ; Thu, 16 Sep 2021 18:08:28 +0000 (UTC) From: Matthew Brost Date: Thu, 16 Sep 2021 11:03:01 -0700 Message-Id: <20210916180301.6791-4-matthew.brost@intel.com> In-Reply-To: <20210916180301.6791-1-matthew.brost@intel.com> References: <20210916180301.6791-1-matthew.brost@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t 3/3] i915/sysfs_preempt_timeout: Update test to work with GuC submission List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: igt-dev@lists.freedesktop.org Cc: john.c.harrison@intel.com, daniele.ceraolospurio@intel.com List-ID: Increase reset timeout as resets can take a bit longer with GuC submission because an error capture is done and with a large GuC log (16 MB) these take a while. Don't run 'off' section as with GuC submission we don't handle dynamically changing the preemption timeout from 'off' to 'on' on a currently running context. This is not bug in GuC submission rather an architectural decision to not implement this as there is no user aside from IGTs. We don't run this section on any gen12+ platforms as we assume GuC submission on these platforms. Signed-off-by: Matthew Brost --- tests/i915/sysfs_preempt_timeout.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/i915/sysfs_preempt_timeout.c b/tests/i915/sysfs_preempt_timeout.c index d176ae72e..fe6b30236 100644 --- a/tests/i915/sysfs_preempt_timeout.c +++ b/tests/i915/sysfs_preempt_timeout.c @@ -29,6 +29,7 @@ #include #include +#include "igt.h" #include "igt_params.h" #include "drmtest.h" #include "i915/gem.h" @@ -41,7 +42,7 @@ #include "sw_sync.h" #define ATTR "preempt_timeout_ms" -#define RESET_TIMEOUT 50 /* milliseconds, at least one jiffie for kworker */ +#define RESET_TIMEOUT 1000 /* milliseconds, at long enough for an error capture */ static bool __enable_hangcheck(int dir, bool state) { @@ -254,6 +255,9 @@ static void test_off(int i915, int engine) gem_quiescent_gpu(i915); igt_require(enable_hangcheck(i915, false)); + /* Not a supported behavior for GuC enabled platforms */ + igt_require(intel_gen(intel_get_drm_devid(i915)) < 12); + igt_assert(igt_sysfs_scanf(engine, "class", "%u", &class) == 1); igt_assert(igt_sysfs_scanf(engine, "instance", "%u", &inst) == 1); -- 2.32.0