Re: [PATCH] drm/xe: Move page fault init after topology init

["Summers, Stuart" <stuart.summers@intel.com>]

On Thu, 2025-07-10 at 12:12 -0700, Matthew Brost wrote:
> We need the topology to determine GT page fault queue size, move page
> fault init after topology init.
> 
> Cc: stable@vger.kernel.org
> Fixes: 3338e4f90c14 ("drm/xe: Use topology to determine page fault
> queue size")
> Signed-off-by: Matthew Brost <matthew.brost@intel.com>

Yep, good catch and thanks!

Reviewed-by: Stuart Summers <stuart.summers@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_gt.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
> index d397df056e4c..af03e19ef9be 100644
> --- a/drivers/gpu/drm/xe/xe_gt.c
> +++ b/drivers/gpu/drm/xe/xe_gt.c
> @@ -628,15 +628,15 @@ int xe_gt_init(struct xe_gt *gt)
>         if (err)
>                 return err;
>  
> -       err = xe_gt_pagefault_init(gt);
> +       err = xe_gt_sysfs_init(gt);
>         if (err)
>                 return err;
>  
> -       err = xe_gt_sysfs_init(gt);
> +       err = gt_init_with_gt_forcewake(gt);
>         if (err)
>                 return err;
>  
> -       err = gt_init_with_gt_forcewake(gt);
> +       err = xe_gt_pagefault_init(gt);
>         if (err)
>                 return err;
>