Revision 26d2b757fff02bbe971abc39071e263aa0cab924 authored by Janusz Krzysztofik on 22 February 2024, 11:32:40 UTC, committed by Joonas Lahtinen on 06 March 2024, 13:33:57 UTC
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 (cherry picked from commit 6ee3f54b880c91ab2e244eb4ffd4bfed37832b25) Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
1 parent 0b385be
File | Mode | Size |
---|---|---|
Kconfig | -rw-r--r-- | 1.1 KB |
Makefile | -rw-r--r-- | 197 bytes |
adfs.h | -rw-r--r-- | 6.5 KB |
dir.c | -rw-r--r-- | 9.5 KB |
dir_f.c | -rw-r--r-- | 7.2 KB |
dir_f.h | -rw-r--r-- | 1.0 KB |
dir_fplus.c | -rw-r--r-- | 7.1 KB |
dir_fplus.h | -rw-r--r-- | 976 bytes |
file.c | -rw-r--r-- | 841 bytes |
inode.c | -rw-r--r-- | 9.7 KB |
map.c | -rw-r--r-- | 10.1 KB |
super.c | -rw-r--r-- | 11.4 KB |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...