From ac43294e8ec2227eb5fa3c0576f9ec07742eabb2 Mon Sep 17 00:00:00 2001 From: Michal Wajdeczko Date: Wed, 1 Oct 2025 01:35:20 +0200 Subject: [PATCH] drm/xe/pf: Log only top level VF state changes MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit 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 Reviewed-by: Michał Winiarski Link: https://lore.kernel.org/r/20250930233525.201263-3-michal.wajdeczko@intel.com --- drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c | 6 +++--- drivers/gpu/drm/xe/xe_sriov_pf_control.c | 19 ++++++++++++++++--- 2 files changed, 19 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c b/drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c index 4f7fff892bc00..e582f4d600157 100644 --- a/drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c +++ b/drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c @@ -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; } diff --git a/drivers/gpu/drm/xe/xe_sriov_pf_control.c b/drivers/gpu/drm/xe/xe_sriov_pf_control.c index 56c0dd3822625..fdadd4fbe9856 100644 --- a/drivers/gpu/drm/xe/xe_sriov_pf_control.c +++ b/drivers/gpu/drm/xe/xe_sriov_pf_control.c @@ -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; } /** -- 2.47.3