Re: [PATCH v2 2/2] drm/xe/pf: Print configuration KLVs using debug printer

["Laguna, Lukasz" <lukasz.laguna@intel.com>]

On 7/3/2025 16:57, 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@intel.com>
> Cc: Lukasz Laguna <lukasz.laguna@intel.com>

Reviewed-by: Lukasz Laguna <lukasz.laguna@intel.com>

> ---
> v2: update xe_gt_sriov_pf_config_restore (Lukasz)
> ---
>   drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c | 10 +++++-----
>   1 file changed, 5 insertions(+), 5 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 d186f780885d..d0cf1d80be07 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);
>   	}
>   
> @@ -2349,7 +2349,7 @@ int xe_gt_sriov_pf_config_restore(struct xe_gt *gt, unsigned int vfid,
>   		return -EINVAL;
>   
>   	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);
>   
>   		drm_printf(&p, "restoring VF%u config:\n", vfid);
>   		xe_guc_klv_print(buf, size / sizeof(u32), &p);