drm/i915/selftests: Fix dependency of some timeouts on HZ
authorJanusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Thu, 22 Feb 2024 11:32:40 +0000 (12:32 +0100)
committerAndi Shyti <andi.shyti@linux.intel.com>
Tue, 5 Mar 2024 17:04:48 +0000 (18:04 +0100)
commit6ee3f54b880c91ab2e244eb4ffd4bfed37832b25
tree2a1f4ef6468197a7f0c24428d776bd58c1c8ecd6
parent6616e048171da09bdcde12c6dcb30a2eea4b461b
drm/i915/selftests: Fix dependency of some timeouts on HZ

Third argument of i915_request_wait() accepts a timeout value in jiffies.
Most users pass either a simple HZ based expression, or a result of
msecs_to_jiffies(), or MAX_SCHEDULE_TIMEOUT, or a very small number not
exceeding 4 if applicable as that value.  However, there is one user --
intel_selftest_wait_for_rq() -- that passes a WAIT_FOR_RESET_TIME symbol,
defined as a large constant value that most probably represents a desired
timeout in ms.  While that usage results in the intended value of timeout
on usual x86_64 kernel configurations, it is not portable across different
architectures and custom kernel configs.

Rename the symbol to clearly indicate intended units and convert it to
jiffies before use.

Fixes: 3a4bfa091c46 ("drm/i915/selftest: Fix workarounds selftest for GuC submission")
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Cc: Rahul Kumar Singh <rahul.kumar.singh@intel.com>
Cc: John Harrison <John.C.Harrison@Intel.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com>
Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240222113347.648945-2-janusz.krzysztofik@linux.intel.com
drivers/gpu/drm/i915/selftests/intel_scheduler_helpers.c