Re: [PATCH v2 2/3] drm/xe/pcode: Improve PCODE logging using xe_log helper

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

On 12-08-2026 06:23 pm, Bhadane, Dnyaneshwar wrote:
>
>> -----Original Message-----
>> From: Koujalagi, Mallesh <mallesh.koujalagi@intel.com>
>> Sent: Wednesday, August 12, 2026 6:05 PM
>> To: intel-xe@lists.freedesktop.org; Vivi, Rodrigo <rodrigo.vivi@intel.com>; Brost,
>> Matthew <matthew.brost@intel.com>
>> Cc: Gupta, Anshuman <anshuman.gupta@intel.com>; Nilawar, Badal
>> <badal.nilawar@intel.com>; Belgaumkar, Vinay <vinay.belgaumkar@intel.com>;
>> Tauro, Riana <riana.tauro@intel.com>; Poosa, Karthik
>> <karthik.poosa@intel.com>; Anirban, Sk <sk.anirban@intel.com>; Jadav, Raag
>> <raag.jadav@intel.com>; Wajdeczko, Michal <michal.wajdeczko@intel.com>;
>> aravind.iddamsetty@linux.intel.com; Nerlige Ramappa, Umesh
>> <umesh.nerlige.ramappa@intel.com>; Bhadane, Dnyaneshwar
>> <dnyaneshwar.bhadane@intel.com>; Koujalagi, Mallesh
>> <mallesh.koujalagi@intel.com>
>> Subject: [PATCH v2 2/3] drm/xe/pcode: Improve PCODE logging using xe_log
>> helper
>>
>> Use xe_log helper for PCODE errors instead of drm_err(). This adds PCODE
>> component tag, making issues easier to detect and track with automated tools.
>>
>> Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
>> ---
>> v2:
>> - s/pcode/PCODE. (Michal)
>> - Drop PCODE in xe_log_err.
>> ---
>>   drivers/gpu/drm/xe/xe_pcode.c | 9 +++++----
>>   1 file changed, 5 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/xe/xe_pcode.c b/drivers/gpu/drm/xe/xe_pcode.c
>> index ccc3bdeed6bb..c69440ea477b 100644
>> --- a/drivers/gpu/drm/xe/xe_pcode.c
>> +++ b/drivers/gpu/drm/xe/xe_pcode.c
>> @@ -14,6 +14,7 @@
>>   #include "regs/xe_pmt.h"
>>   #include "xe_assert.h"
>>   #include "xe_device.h"
>> +#include "xe_log.h"
>>   #include "xe_mmio.h"
>>   #include "xe_pcode_api.h"
>>   #include "xe_pm.h"
>> @@ -219,8 +220,8 @@ int xe_pcode_request(struct xe_tile *tile, u32 mbox, u32
>> request,
>>   	 * requests, and for any quirks of the PCODE firmware that delays
>>   	 * the request completion.
>>   	 */
>> -	drm_err(&tile_to_xe(tile)->drm,
>> -		"PCODE timeout, retrying with preemption disabled\n");
>> +	xe_log_err(tile, PCODE, -ETIMEDOUT,
>> +		   "Timeout, retrying with preemption disabled\n");
>>   	preempt_disable();
>>   	ret = pcode_try_request(tile, mbox, request, reply_mask, reply, &status,
>>   				true, 50 * 1000, true);
>> @@ -320,8 +321,8 @@ int xe_pcode_ready(struct xe_device *xe, bool locked)
>>   		mutex_unlock(&tile->pcode.lock);
>>
>>   	if (ret)
>> -		drm_err(&xe->drm,
>> -			"PCODE initialization timedout after: 3 min\n");
>> +		xe_log_err(xe, PCODE, ret,
>> +			   "Initialization timedout after: 3 min\n");
>>
>>   	return ret;
>>   }
> Hello Mallesh,
> This is one more place inside of the pcode_mailbox_status().
> Could you please include that as well to this patch ?

That already covered in SIGID the patch series: 
https://patchwork.freedesktop.org/patch/743325/?series=171022&rev=3

Thanks,

-/Mallesh


> Dnyaneshwar
>
>> --
>> 2.48.1