Hi Sebastian, The series looks good to me, so: Reviewed-by: Krzysztof Karas <[email protected]> on all patches.
> This series cleans up the placement of several context-specific workarounds > in the i915 driver by moving them from rcs_engine_wa_init() to the > appropriate *_ctx_workarounds_init() functions. This improves clarity and > ensures that platform-specific workarounds are applied during context > initialization, as intended. > > It also aligns with the hardware documentation, which mandates that certain > CACHE_MODE_0 and CACHE_MODE_1 register settings be applied during context > initialization to ensure correct GPU behavior. > > If some workarounds remain in rcs_engine_wa_init(), it's due to issues > encountered while attempting to move them, such as the workaround not > sticking when applied in context, or other platform-specific constraints > that require further investigation. > > Sebastian Brzezinka (3): > drm/i915/gt: Relocate compression repacking WA for JSL/EHL > drm/i915/gt: Relocate Gen7 context-specific workarounds > drm/i915/gt: Relocate Gen6 context-specific workaround > > drivers/gpu/drm/i915/gt/intel_workarounds.c | 49 +++++++++++---------- > 1 file changed, 25 insertions(+), 24 deletions(-) > > -- > 2.34.1 > -- Best Regards, Krzysztof
