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

Reply via email to