Skip to content

Commit a82f1bb

Browse files
neosys007rodrigovivi
authored andcommitted
drm/i915/gt: use correct selftest config symbol
intel_engine_user.c checks CONFIG_DRM_I915_SELFTESTS before running the engine UABI isolation check. Kconfig defines DRM_I915_SELFTEST, without the trailing "S", and the rest of i915 uses CONFIG_DRM_I915_SELFTEST. Because CONFIG_DRM_I915_SELFTESTS is not backed by any Kconfig symbol, the IS_ENABLED() test is always false. Use the existing selftest symbol so the debug/selftest guarded path can be reached when selftests are enabled. This is a source-level fix. It does not claim dynamic hardware reproduction; the evidence is the Kconfig definition and the inconsistent guard in intel_engine_user.c. Fixes: 750e76b ("drm/i915/gt: Move the [class][inst] lookup for engines onto the GT") Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn> Signed-off-by: Tvrtko Ursulin <tursulin@ursulin.net> Link: https://lore.kernel.org/r/20260705080225.436-1-pengpeng@iscas.ac.cn (cherry picked from commit 14a2012) Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
1 parent a13c140 commit a82f1bb

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/gpu/drm/i915/gt/intel_engine_user.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ void intel_engines_driver_register(struct drm_i915_private *i915)
259259
p = &prev->rb_right;
260260
}
261261

262-
if (IS_ENABLED(CONFIG_DRM_I915_SELFTESTS) &&
262+
if (IS_ENABLED(CONFIG_DRM_I915_SELFTEST) &&
263263
IS_ENABLED(CONFIG_DRM_I915_DEBUG_GEM)) {
264264
struct intel_engine_cs *engine;
265265
unsigned int isolation;

0 commit comments

Comments
 (0)