]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/xe/pf: Log only top level VF state changes
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Tue, 30 Sep 2025 23:35:20 +0000 (01:35 +0200)
committerMichal Wajdeczko <michal.wajdeczko@intel.com>
Thu, 2 Oct 2025 21:58:30 +0000 (23:58 +0200)
The user likely only care about top level VF state changes, any VF
state logs on the per-GT basis can be demoted to the debug level.

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: MichaƂ Winiarski <michal.winiarski@intel.com>
Link: https://lore.kernel.org/r/20250930233525.201263-3-michal.wajdeczko@intel.com
drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c
drivers/gpu/drm/xe/xe_sriov_pf_control.c

index 4f7fff892bc00b53c1021d99b3fc525eef8b7bae..e582f4d60015783872ec8260e8e6ea7d3e508d6c 100644 (file)
@@ -616,7 +616,7 @@ int xe_gt_sriov_pf_control_pause_vf(struct xe_gt *gt, unsigned int vfid)
        }
 
        if (pf_expect_vf_state(gt, vfid, XE_GT_SRIOV_STATE_PAUSED)) {
-               xe_gt_sriov_info(gt, "VF%u paused!\n", vfid);
+               xe_gt_sriov_dbg(gt, "VF%u paused!\n", vfid);
                return 0;
        }
 
@@ -755,7 +755,7 @@ int xe_gt_sriov_pf_control_resume_vf(struct xe_gt *gt, unsigned int vfid)
                return err;
 
        if (pf_expect_vf_state(gt, vfid, XE_GT_SRIOV_STATE_RESUMED)) {
-               xe_gt_sriov_info(gt, "VF%u resumed!\n", vfid);
+               xe_gt_sriov_dbg(gt, "VF%u resumed!\n", vfid);
                return 0;
        }
 
@@ -896,7 +896,7 @@ int xe_gt_sriov_pf_control_stop_vf(struct xe_gt *gt, unsigned int vfid)
                return err;
 
        if (pf_expect_vf_state(gt, vfid, XE_GT_SRIOV_STATE_STOPPED)) {
-               xe_gt_sriov_info(gt, "VF%u stopped!\n", vfid);
+               xe_gt_sriov_dbg(gt, "VF%u stopped!\n", vfid);
                return 0;
        }
 
index 56c0dd3822625bb3636aa179673f83475fd7e774..fdadd4fbe98561f805c1ecb9bd2b60c6d81fbd78 100644 (file)
@@ -6,6 +6,7 @@
 #include "xe_device.h"
 #include "xe_gt_sriov_pf_control.h"
 #include "xe_sriov_pf_control.h"
+#include "xe_sriov_printk.h"
 
 /**
  * xe_sriov_pf_control_pause_vf() - Pause a VF on all GTs.
@@ -28,7 +29,11 @@ int xe_sriov_pf_control_pause_vf(struct xe_device *xe, unsigned int vfid)
                result = result ? -EUCLEAN : err;
        }
 
-       return result;
+       if (result)
+               return result;
+
+       xe_sriov_info(xe, "VF%u paused!\n", vfid);
+       return 0;
 }
 
 /**
@@ -52,7 +57,11 @@ int xe_sriov_pf_control_resume_vf(struct xe_device *xe, unsigned int vfid)
                result = result ? -EUCLEAN : err;
        }
 
-       return result;
+       if (result)
+               return result;
+
+       xe_sriov_info(xe, "VF%u resumed!\n", vfid);
+       return 0;
 }
 
 /**
@@ -76,7 +85,11 @@ int xe_sriov_pf_control_stop_vf(struct xe_device *xe, unsigned int vfid)
                result = result ? -EUCLEAN : err;
        }
 
-       return result;
+       if (result)
+               return result;
+
+       xe_sriov_info(xe, "VF%u stopped!\n", vfid);
+       return 0;
 }
 
 /**