All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 0/8] drm/i915/selftests: Modernize printing in i915 selftests
@ 2026-08-28 10:03 Krzysztof Karas
  2026-08-28 10:03 ` [PATCH v3 1/8] drm/i915/selftest: Fix a typo in alloc_workqueue call Krzysztof Karas
                   ` (8 more replies)
  0 siblings, 9 replies; 25+ messages in thread
From: Krzysztof Karas @ 2026-08-28 10:03 UTC (permalink / raw)
  To: intel-gfx
  Cc: dri-devel, Jani Nikula, Andi Shyti, Janusz Krzysztofik,
	Sebastian Brzezinka, Krzysztof Niemiec, Krzysztof Karas

Currently, in many instances, i915 functions call pr_* macros
to write a message to the system log. This way of printing
lacks device context, which could be a valuable source of
information. Many files in DRM already use drm_* prints to
include that context, so it would be beneficial to do the same
for i915 as well. Start small and convert i915 selftests first.

I do not anticipate this series to pass BAT runs due to
filtering warnings and error messages in our CI. Since the
formatting is slightly different, expected messages will be
detected as new issues.

v3 (Janusz, sashiko):
- keep pr_* where no device exists;
- pass struct drm_device explicitly through selftest wrappers
  instead of casting void *data;
- fall back to pr_* when device is NULL, to avoid "NULL device"
  output;
- keep lines limited to 80 characters where possible;
- drop unrelated reformatting;
- introduce local drm_i915_private variable for multiple complex
  dereferences;

Krzysztof Karas (8):
  drm/i915/selftest: Fix a typo in alloc_workqueue call
  drm/i915/selftests: Use drm_* prints in live selftests
  drm/i915/selftests: Use drm_* prints in selftest helpers
  drm/i915/gt/selftests: Use drm_* prints in aggregated gt selftests
  drm/i915/gt/selftests: Use drm_* prints in gt standalone tests
  drm/i915/gt/selftests: Use drm_* prints in gt selftest helpers
  drm/i915/selftests: Use drm_* prints in selftest headers
  drm/i915/selftests: Add drm_device to selftest wrappers

 .../gpu/drm/i915/gem/selftests/huge_pages.c   |   4 +-
 .../i915/gem/selftests/i915_gem_client_blt.c  |   2 +-
 .../i915/gem/selftests/i915_gem_coherency.c   |   2 +-
 .../drm/i915/gem/selftests/i915_gem_context.c |   2 +-
 .../drm/i915/gem/selftests/i915_gem_dmabuf.c  |   4 +-
 .../drm/i915/gem/selftests/i915_gem_migrate.c |   2 +-
 .../drm/i915/gem/selftests/i915_gem_mman.c    |   4 +-
 .../drm/i915/gem/selftests/i915_gem_object.c  |   4 +-
 .../drm/i915/gem/selftests/i915_gem_phys.c    |   2 +-
 drivers/gpu/drm/i915/gt/selftest_context.c    |  35 +-
 drivers/gpu/drm/i915/gt/selftest_engine_cs.c  |  12 +-
 .../drm/i915/gt/selftest_engine_heartbeat.c   |  15 +-
 drivers/gpu/drm/i915/gt/selftest_engine_pm.c  |  60 +--
 drivers/gpu/drm/i915/gt/selftest_execlists.c  | 355 +++++++++++-------
 drivers/gpu/drm/i915/gt/selftest_gt_pm.c      |  36 +-
 drivers/gpu/drm/i915/gt/selftest_hangcheck.c  |   2 +-
 drivers/gpu/drm/i915/gt/selftest_llc.c        |  29 +-
 drivers/gpu/drm/i915/gt/selftest_lrc.c        |   2 +-
 drivers/gpu/drm/i915/gt/selftest_migrate.c    |  69 ++--
 drivers/gpu/drm/i915/gt/selftest_mocs.c       |  14 +-
 drivers/gpu/drm/i915/gt/selftest_rc6.c        |  59 +--
 drivers/gpu/drm/i915/gt/selftest_reset.c      |  30 +-
 drivers/gpu/drm/i915/gt/selftest_ring.c       |   2 +-
 .../drm/i915/gt/selftest_ring_submission.c    |   2 +-
 drivers/gpu/drm/i915/gt/selftest_rps.c        | 297 ++++++++-------
 drivers/gpu/drm/i915/gt/selftest_slpc.c       | 105 ++++--
 drivers/gpu/drm/i915/gt/selftest_timeline.c   |   4 +-
 drivers/gpu/drm/i915/gt/selftest_tlb.c        |  41 +-
 .../gpu/drm/i915/gt/selftest_workarounds.c    | 106 ++++--
 drivers/gpu/drm/i915/gt/st_shmem_utils.c      |   2 +-
 drivers/gpu/drm/i915/gt/uc/selftest_guc.c     |   2 +-
 .../drm/i915/gt/uc/selftest_guc_hangcheck.c   |   2 +-
 .../drm/i915/gt/uc/selftest_guc_multi_lrc.c   |   2 +-
 drivers/gpu/drm/i915/i915_selftest.h          |  16 +-
 drivers/gpu/drm/i915/selftests/i915_active.c  |  14 +-
 drivers/gpu/drm/i915/selftests/i915_gem.c     |   2 +-
 .../gpu/drm/i915/selftests/i915_gem_evict.c   |  79 ++--
 drivers/gpu/drm/i915/selftests/i915_gem_gtt.c | 227 ++++++-----
 drivers/gpu/drm/i915/selftests/i915_perf.c    |  31 +-
 drivers/gpu/drm/i915/selftests/i915_request.c | 307 ++++++++-------
 .../gpu/drm/i915/selftests/i915_selftest.c    |  71 +++-
 .../gpu/drm/i915/selftests/i915_sw_fence.c    |   4 +-
 drivers/gpu/drm/i915/selftests/i915_syncmap.c |   2 +-
 drivers/gpu/drm/i915/selftests/i915_vma.c     | 261 ++++++++-----
 .../gpu/drm/i915/selftests/igt_flush_test.c   |   7 +-
 .../gpu/drm/i915/selftests/igt_live_test.c    |   6 +-
 drivers/gpu/drm/i915/selftests/igt_reset.c    |   5 +-
 .../drm/i915/selftests/intel_memory_region.c  |   6 +-
 .../i915/selftests/intel_scheduler_helpers.c  |   8 +-
 drivers/gpu/drm/i915/selftests/intel_uncore.c |  49 +--
 .../gpu/drm/i915/selftests/mock_gem_device.c  |   4 +-
 drivers/gpu/drm/i915/selftests/scatterlist.c  |   2 +-
 52 files changed, 1455 insertions(+), 955 deletions(-)

-- 
2.34.1


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

end of thread, other threads:[~2026-08-28 18:08 UTC | newest]

Thread overview: 25+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-28 10:03 [PATCH v3 0/8] drm/i915/selftests: Modernize printing in i915 selftests Krzysztof Karas
2026-08-28 10:03 ` [PATCH v3 1/8] drm/i915/selftest: Fix a typo in alloc_workqueue call Krzysztof Karas
2026-08-28 14:24   ` Andi Shyti
2026-08-28 16:54   ` Andi Shyti
2026-08-28 10:03 ` [PATCH v3 2/8] drm/i915/selftests: Use drm_* prints in live selftests Krzysztof Karas
2026-08-28 14:34   ` Andi Shyti
2026-08-28 16:41   ` Krzysztof Niemiec
2026-08-28 10:03 ` [PATCH v3 3/8] drm/i915/selftests: Use drm_* prints in selftest helpers Krzysztof Karas
2026-08-28 15:58   ` Andi Shyti
2026-08-28 16:49   ` Krzysztof Niemiec
2026-08-28 10:03 ` [PATCH v3 4/8] drm/i915/gt/selftests: Use drm_* prints in aggregated gt selftests Krzysztof Karas
2026-08-28 16:18   ` Andi Shyti
2026-08-28 17:40   ` Krzysztof Niemiec
2026-08-28 10:03 ` [PATCH v3 5/8] drm/i915/gt/selftests: Use drm_* prints in gt standalone tests Krzysztof Karas
2026-08-28 16:23   ` Andi Shyti
2026-08-28 18:02   ` Krzysztof Niemiec
2026-08-28 10:03 ` [PATCH v3 6/8] drm/i915/gt/selftests: Use drm_* prints in gt selftest helpers Krzysztof Karas
2026-08-28 16:28   ` Andi Shyti
2026-08-28 18:06   ` Krzysztof Niemiec
2026-08-28 10:03 ` [PATCH v3 7/8] drm/i915/selftests: Use drm_* prints in selftest headers Krzysztof Karas
2026-08-28 16:30   ` Andi Shyti
2026-08-28 18:08   ` Krzysztof Niemiec
2026-08-28 10:03 ` [PATCH v3 8/8] drm/i915/selftests: Add drm_device to selftest wrappers Krzysztof Karas
2026-08-28 16:36   ` Andi Shyti
2026-08-28 11:31 ` ✗ i915.CI.BAT: failure for drm/i915/selftests: Modernize printing in i915 selftests (rev4) Patchwork

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.