Revert "drm/xe/xe2: Enable Indirect Ring State support for Xe2"
authorMatthew Brost <matthew.brost@intel.com>
Wed, 2 Jul 2025 03:58:46 +0000 (20:58 -0700)
committerLucas De Marchi <lucas.demarchi@intel.com>
Tue, 8 Jul 2025 03:57:17 +0000 (20:57 -0700)
This reverts commit fe0154cf8222d9e38c60ccc124adb2f9b5272371.

Seeing some unexplained random failures during LRC context switches with
indirect ring state enabled. The failures were always there, but the
repro rate increased with the addition of WA BB as a separate BO.
Commit 3a1edef8f4b5 ("drm/xe: Make WA BB part of LRC BO") helped to
reduce the issues in the context switches, but didn't eliminate them
completely.

Indirect ring state is not required for any current features, so disable
for now until failures can be root caused.

Cc: stable@vger.kernel.org
Fixes: fe0154cf8222 ("drm/xe/xe2: Enable Indirect Ring State support for Xe2")
Signed-off-by: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://lore.kernel.org/r/20250702035846.3178344-1-matthew.brost@intel.com
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
(cherry picked from commit 03d85ab36bcbcbe9dc962fccd3f8e54d7bb93b35)
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
drivers/gpu/drm/xe/xe_pci.c

index ac4beaed58ffb669128f1ec4fc28297f9bf8c4d3..278af53c74dc10c7cd2faade356c40e2e00493b1 100644 (file)
@@ -140,7 +140,6 @@ static const struct xe_graphics_desc graphics_xelpg = {
        .has_asid = 1, \
        .has_atomic_enable_pte_bit = 1, \
        .has_flat_ccs = 1, \
-       .has_indirect_ring_state = 1, \
        .has_range_tlb_invalidation = 1, \
        .has_usm = 1, \
        .has_64bit_timestamp = 1, \