Re: [PATCH 0/5] drm/xe: Fix early probe order

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Thu, Apr 25, 2024 at 11:24:05AM -0700, Lucas De Marchi wrote:
> ( It looks like our initialization functions are in a less than optimal
>   order, with additional functions being added in random places. We
>   probably need to rename some to follow a pattern so this doesn't happen
>   again )
> 
> What this series does is to fix the mostly blatant mistakes in the gt
> initialization part: since the addition of hwconfig, the WA processing
> is not done as early as it should. Future workarounds will require
> XE_WA() matching much earlier. Move initialization around and drop dead
> code to cover that.


Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>

It is likely a good idea to wait for feedback from SRIOV side, since
they had tuned the init sequence with that in mind.

---


Cc: Riana Tauro <riana.tauro@intel.com>

who is also working to ensure sanity on our init sequences.

> 
> Lucas De Marchi (5):
>   drm/xe: Drop __engine_mask
>   drm/xe: Drop useless forcewake get/put
>   drm/xe: Move xe_gt_init_early() where it belongs
>   drm/xe: Move xe_force_wake_init_gt() inside gt initialization
>   drm/xe: Move sw-only pcode initialization
> 
>  drivers/gpu/drm/xe/xe_device.c   | 16 +++++-----------
>  drivers/gpu/drm/xe/xe_gt.c       | 15 ++++-----------
>  drivers/gpu/drm/xe/xe_gt_types.h |  6 ------
>  drivers/gpu/drm/xe/xe_migrate.c  |  3 +--
>  drivers/gpu/drm/xe/xe_pci.c      |  6 +++---
>  5 files changed, 13 insertions(+), 33 deletions(-)
> 
> -- 
> 2.43.0
>