Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v8 00/10] drm/xe/sysctrl: Clean up error handling in
@ 2026-09-21 13:39 Mallesh Koujalagi
  2026-09-21 13:39 ` [PATCH v8 01/10] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear() Mallesh Koujalagi
                   ` (12 more replies)
  0 siblings, 13 replies; 19+ messages in thread
From: Mallesh Koujalagi @ 2026-09-21 13:39 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

The patch series improves error handling in the System Controller mailbox
driver by making error propagation more consistent and using
xe_log_err() for structured error reporting.

It also removes a few redundant checks and simplifies helper
interfaces where no error path exists.

v2:
- Use -EOVERFLOW.
- Drop sysctrl in xe_log_err. (Michal)
- Drop redundant %pe.
- Print response->count.

v3:
- Rebase.

v4:
- Replace xe_warn to xe_log_err_info for. (Michal)
- Use xe_assert().
- Change sysctrl_wait_bit_* to return errno.
- Make sysctrl_write_frame return type as void.
- Log mismatched header info.

v5:
- Fix align issue. (Michal)
- Use %#x after REG_FIELD_GET change.
- Use REG_FIELD_GET instead of FIELD_GET.
- Modify logging message.
- Change Error as EXFULL.
- Update logging message.

v6:
- Move REG_FIELD_GET to different patch. (Michal)
- Update response message.
- Move all REG_FIELD_GET to single patch.
- Update log message. (Michal)
- Change correct errno. (Michal)

v7:
- Update FIELD_PREP to REG_FIELD_PREP. (Michal/Sashiko)
- Update FIELD_GET to REG_FIELD_GET.
- Remove le32_to_cpu. (Michal)

v8:
- Rebase.
- Rephrase commit message. (Michal)

Mallesh Koujalagi (10):
  drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear()
  drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set()
  drm/xe/sysctrl: Make sysctrl_write_frame() void
  drm/xe/sysctrl: Make sysctrl_read_frame() void
  drm/xe/sysctrl: Use xe_assert() for payload size validation
  drm/xe/sysctrl: simplify mailbox header handling
  drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET()
  drm/xe/sysctrl: Improve firmware response error logging
  drm/xe/sysctrl: Log group and command ID on mailbox failure
  drm/xe/sysctrl: Report 'System Controller event' error using SIGID

 drivers/gpu/drm/xe/xe_sysctrl_event.c   |  14 +--
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 110 +++++++++++-------------
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.h |   6 +-
 3 files changed, 61 insertions(+), 69 deletions(-)

-- 
2.48.1


^ permalink raw reply	[flat|nested] 19+ messages in thread

end of thread, other threads:[~2026-09-24  5:34 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-21 13:39 [PATCH v8 00/10] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
2026-09-21 13:39 ` [PATCH v8 01/10] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear() Mallesh Koujalagi
2026-09-21 13:39 ` [PATCH v8 02/10] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set() Mallesh Koujalagi
2026-09-21 13:39 ` [PATCH v8 03/10] drm/xe/sysctrl: Make sysctrl_write_frame() void Mallesh Koujalagi
2026-09-21 13:39 ` [PATCH v8 04/10] drm/xe/sysctrl: Make sysctrl_read_frame() void Mallesh Koujalagi
2026-09-21 13:39 ` [PATCH v8 05/10] drm/xe/sysctrl: Use xe_assert() for payload size validation Mallesh Koujalagi
2026-09-21 13:39 ` [PATCH v8 06/10] drm/xe/sysctrl: simplify mailbox header handling Mallesh Koujalagi
2026-09-22 11:17   ` Michal Wajdeczko
2026-09-23 10:54     ` Mallesh, Koujalagi
2026-09-24  5:34     ` Anoop Vijay
2026-09-23 23:52   ` Umesh Nerlige Ramappa
2026-09-21 13:39 ` [PATCH v8 07/10] drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET() Mallesh Koujalagi
2026-09-22 11:11   ` Michal Wajdeczko
2026-09-21 13:39 ` [PATCH v8 08/10] drm/xe/sysctrl: Improve firmware response error logging Mallesh Koujalagi
2026-09-21 13:39 ` [PATCH v8 09/10] drm/xe/sysctrl: Log group and command ID on mailbox failure Mallesh Koujalagi
2026-09-21 13:39 ` [PATCH v8 10/10] drm/xe/sysctrl: Report 'System Controller event' error using SIGID Mallesh Koujalagi
2026-09-21 13:48 ` ✓ CI.KUnit: success for drm/xe/sysctrl: Clean up error handling in (rev6) Patchwork
2026-09-21 14:54 ` ✗ Xe.CI.BAT: failure " Patchwork
2026-09-21 20:02 ` ✗ Xe.CI.FULL: " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox