Re: [PATCH 11/12] drm/xe/lmtt: Report invalidation errors using structured xe_log API

["Mallesh, Koujalagi" <mallesh.koujalagi@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1548 bytes --]


On 20-08-2026 12:44 am, Dnyaneshwar Bhadane wrote:
> Replace xe_tile_sriov_err() with xe_log_err() using the LMTT component
> tag for LMTT and MERT invalidation failures, preserving the SR-IOV
> prefix in the log message.
>
> Signed-off-by: Dnyaneshwar Bhadane<dnyaneshwar.bhadane@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_lmtt.c | 11 +++++++----
>   1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_lmtt.c b/drivers/gpu/drm/xe/xe_lmtt.c
> index 0c726eda9390..a9feeacbae55 100644
> --- a/drivers/gpu/drm/xe/xe_lmtt.c
> +++ b/drivers/gpu/drm/xe/xe_lmtt.c
> @@ -14,6 +14,7 @@
>   #include "xe_bo.h"
>   #include "xe_tlb_inval.h"
>   #include "xe_lmtt.h"
> +#include "xe_log.h"
>   #include "xe_map.h"
>   #include "xe_mert.h"
>   #include "xe_mmio.h"
> @@ -302,14 +303,16 @@ void xe_lmtt_invalidate_hw(struct xe_lmtt *lmtt)
>   
>   	err = lmtt_invalidate_hw(lmtt);
>   	if (err)
> -		xe_tile_sriov_err(tile, "LMTT invalidation failed (%pe)",
> -				  ERR_PTR(err));
> +		xe_log_err(tile, LMTT, -EIO,  "%s invalidation failed (%pe)",
Use err as errno.
> +			   xe_sriov_printk_prefix(xe),
> +			   ERR_PTR(err));
nit: drop ERR_PTR(err.
>   
>   	if (xe_device_has_mert(xe) && xe_tile_is_root(tile)) {
>   		err = xe_mert_invalidate_lmtt(xe);
>   		if (err)
> -			xe_tile_sriov_err(tile, "MERT LMTT invalidation failed (%pe)",
> -					  ERR_PTR(err));
> +			xe_log_err(tile, LMTT, -EIO,  "%s MERT invalidation failed (%pe)",
> +				   xe_sriov_printk_prefix(xe),
> +				   ERR_PTR(err));
ditto
>   	}
>   }
>   

[-- Attachment #2: Type: text/html, Size: 2570 bytes --]