✓ CI.checkpatch: success for drm/xe: Set LRC addresses before guc load (rev2)

[Patchwork]

== Series Details ==

Series: drm/xe: Set LRC addresses before guc load (rev2)
URL   : https://patchwork.freedesktop.org/series/147505/
State : success

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
13a92ce9fd458ebd6064f23cec8c39c53d02ed26
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit b0df53f8f92c9fb6b951b3700e7683101fcb9edf
Author: Lucas De Marchi <lucas.demarchi at intel.com>
Date:   Wed Apr 9 21:59:34 2025 -0700

    drm/xe: Set LRC addresses before guc load
    
    The metadata saved in the ADS is read by GuC when it's initialized.
    Saving the addresses to the LRCs when they are populated is too late as
    GuC will keep using the old ones.
    
    This was causing GuC to use the RCS LRC for any engine class. It's not a
    big problem on a Linux-only scenario since the they are used by GuC only
    on media engines when the watchdog is triggered. However, in a
    virtualization scenario with Windows as the VF, it causes the wrong LRCs
    to be loaded as the watchdog is used for all engines.
    
    Fix it by letting guc_golden_lrc_init() initialize the metadata, like
    other *_init() functions, and later guc_golden_lrc_populate() to copy
    the LRCs to the right places. The former is called before the second GuC
    load, while the latter is called after LRCs have been recorded.
    
    Cc: Chee Yin Wong <chee.yin.wong at intel.com>
    Cc: John Harrison <john.c.harrison at intel.com>
    Cc: Matt Roper <matthew.d.roper at intel.com>
    Cc: Matthew Brost <matthew.brost at intel.com>
    Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
+ /mt/dim checkpatch 00a3cb4f4ab069fae5ccd844f89f2962cc0283bb drm-intel
b0df53f8f92c drm/xe: Set LRC addresses before guc load