]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
drm/xe/pf: Resend PF provisioning after GT reset
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Fri, 11 Jul 2025 19:33:12 +0000 (21:33 +0200)
committerLucas De Marchi <lucas.demarchi@intel.com>
Thu, 17 Jul 2025 12:51:51 +0000 (09:51 -0300)
If we reload the GuC due to suspend/resume or GT reset then we
have to resend not only any VFs provisioning data, but also PF
configuration, like scheduling parameters (EQ, PT), as otherwise
GuC will continue to use default values.

Fixes: 411220808cee ("drm/xe/pf: Restart VFs provisioning after GT reset")
Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: Piotr Piórkowski <piotr.piorkowski@intel.com>
Link: https://lore.kernel.org/r/20250711193316.1920-3-michal.wajdeczko@intel.com
(cherry picked from commit 1c38dd6afa4a8ecce28e94da794fd1d205c30f51)
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c

index 2420a548cacc1c558a9f79d741cede94834d4851..53a44702c04afddac44da10017eedc74f1de2ad2 100644 (file)
@@ -2364,6 +2364,21 @@ int xe_gt_sriov_pf_config_restore(struct xe_gt *gt, unsigned int vfid,
        return err;
 }
 
+static int pf_push_self_config(struct xe_gt *gt)
+{
+       int err;
+
+       err = pf_push_full_vf_config(gt, PFID);
+       if (err) {
+               xe_gt_sriov_err(gt, "Failed to push self configuration (%pe)\n",
+                               ERR_PTR(err));
+               return err;
+       }
+
+       xe_gt_sriov_dbg_verbose(gt, "self configuration completed\n");
+       return 0;
+}
+
 static void fini_config(void *arg)
 {
        struct xe_gt *gt = arg;
@@ -2387,9 +2402,17 @@ static void fini_config(void *arg)
 int xe_gt_sriov_pf_config_init(struct xe_gt *gt)
 {
        struct xe_device *xe = gt_to_xe(gt);
+       int err;
 
        xe_gt_assert(gt, IS_SRIOV_PF(xe));
 
+       mutex_lock(xe_gt_sriov_pf_master_mutex(gt));
+       err = pf_push_self_config(gt);
+       mutex_unlock(xe_gt_sriov_pf_master_mutex(gt));
+
+       if (err)
+               return err;
+
        return devm_add_action_or_reset(xe->drm.dev, fini_config, gt);
 }
 
@@ -2407,6 +2430,10 @@ void xe_gt_sriov_pf_config_restart(struct xe_gt *gt)
        unsigned int n, total_vfs = xe_sriov_pf_get_totalvfs(gt_to_xe(gt));
        unsigned int fail = 0, skip = 0;
 
+       mutex_lock(xe_gt_sriov_pf_master_mutex(gt));
+       pf_push_self_config(gt);
+       mutex_unlock(xe_gt_sriov_pf_master_mutex(gt));
+
        for (n = 1; n <= total_vfs; n++) {
                if (xe_gt_sriov_pf_config_is_empty(gt, n))
                        skip++;