[2/2] drm/xe/pf: Print configuration KLVs using debug printer
Laguna, Lukasz
lukasz.laguna at intel.com
Tue Jun 10 06:11:02 UTC 2025
On 6/4/2025 21:00, Michal Wajdeczko wrote:
> While we print VF's configuration KLVs only under DEBUG_SRIOV
> config, we should be doing it at debug level, not info level.
>
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
> ---
> drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> index 3556c41c041b..3128d9adc10e 100644
> --- a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> +++ b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> @@ -104,13 +104,13 @@ static int pf_push_vf_buf_klvs(struct xe_gt *gt, unsigned int vfid, u32 num_klvs
> }
>
> if (IS_ENABLED(CONFIG_DRM_XE_DEBUG_SRIOV)) {
> - struct drm_printer p = xe_gt_info_printer(gt);
> + struct drm_printer p = xe_gt_dbg_printer(gt);
> void *klvs = xe_guc_buf_cpu_ptr(buf);
> char name[8];
>
> - xe_gt_sriov_info(gt, "pushed %s config with %u KLV%s:\n",
> - xe_sriov_function_name(vfid, name, sizeof(name)),
> - num_klvs, str_plural(num_klvs));
> + xe_gt_sriov_dbg(gt, "pushed %s config with %u KLV%s:\n",
> + xe_sriov_function_name(vfid, name, sizeof(name)),
> + num_klvs, str_plural(num_klvs));
> xe_guc_klv_print(klvs, num_dwords, &p);
> }
>
Shouldn't xe_gt_sriov_pf_config_restore() be updated in the same way on
the occasion?
More information about the Intel-xe
mailing list