• Matt Roper's avatar
    drm/i915/xehp: LNCF/LBCF workarounds should be on the GT list · 4583d6be
    Matt Roper authored
    
    
    Although registers in the L3 bank/node configuration ranges are marked
    as having "DEV" reset characteristics in the bspec, this appears to be a
    hold-over from pre-Xe_HP platforms.  In reality, these registers
    maintain their values across engine resets, meaning that workarounds
    and tuning settings targeting them should be placed on the GT
    workaround list rather than an engine workaround list.
    
    Note that an extra clue here is that these registers moved from the
    RENDER forcewake domain to the GT forcewake domain in Xe_HP; generally
    RCS/CCS engine resets should not lead to the reset of a register that
    lives outside the RENDER domain.
    
    Re-applying these registers on engine resets wouldn't actually hurt
    anything, but is unnecessary and just makes it more confusing to anyone
    trying to decipher how these registers really work.
    
    v2:
     - Also move DG2's Wa_14010648519 to the GT list.  (Gustavo)
    
    Cc: Gustavo Sousa <gustavo.sousa@intel.com>
    Signed-off-by: default avatarMatt Roper <matthew.d.roper@intel.com>
    Reviewed-by: default avatarGustavo Sousa <gustavo.sousa@intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20230209232228.859317-1-matthew.d.roper@intel.com
    4583d6be
intel_workarounds.c 97.7 KB