All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early()
@ 2026-06-09 20:17 Gustavo Sousa
  2026-06-09 20:17 ` [PATCH 1/9] drm/xe: Add framework for info probing Gustavo Sousa
                   ` (11 more replies)
  0 siblings, 12 replies; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
  To: intel-xe; +Cc: Gustavo Sousa

Functions xe_info_init() and xe_info_init_early() are meant to
initialize fields of xe->info, but they also probe the hardware as
part of the process.  This mixed responsibilities issue makes kunit
testing need to rely on static stubbing in order to initialize the
fake device info.

To make the design cleaner, let's do the necessary refactors so that
those functions don't try to poke the hardware and receive and info
that needs to be probed as parameters instead.  The end result is that
functions xe_info_init() and xe_info_init_early() become better
reusable and we have a cleaner tests/xe_pci.c.

This will also be relevant for an upcoming series that is likely to
attempt to do fake device initialization outside of tests/xe_pci.c.
Nevertheless, even without it the refactor is arguably good in itself,
as observed by the end result of tests/xe_pci.c.

Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
Gustavo Sousa (9):
      drm/xe: Add framework for info probing
      drm/xe/step: Pass xe_step_info to xe_step_*_get() functions
      drm/xe: Add devid and revid to xe_probed_info
      drm/xe/step: Make xe_step_platform_get() independent from xe->info
      drm/xe: Add platform-level step info to xe_probed_info
      drm/xe/tests: Set non-GMDID graphics step in xe_pci_fake_device_init()
      drm/xe: Add graphics/media IPs and their step info to xe_probed_info
      drm/xe: Don't initialize tile_count in xe_info_init_early()
      drm/xe: Add tile_count to xe_probed_info

 drivers/gpu/drm/xe/tests/xe_pci.c     |  53 +++++------
 drivers/gpu/drm/xe/tests/xe_wa_test.c |   3 -
 drivers/gpu/drm/xe/xe_pci.c           | 160 +++++++++++++++++++++++-----------
 drivers/gpu/drm/xe/xe_step.c          |  38 ++++----
 drivers/gpu/drm/xe/xe_step.h          |   8 +-
 5 files changed, 164 insertions(+), 98 deletions(-)
---
base-commit: 3806de1666bcca0525d737e3849c602cbe98fb1a
change-id: 20260604-xe-probe-info-35f404748ce3

Best regards,
--  
Gustavo Sousa <gustavo.sousa@intel.com>


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

end of thread, other threads:[~2026-06-17 12:43 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
2026-06-09 20:17 ` [PATCH 1/9] drm/xe: Add framework for info probing Gustavo Sousa
2026-06-16 18:51   ` Violet Monti
2026-06-16 21:56   ` Matt Roper
2026-06-17 12:43     ` Gustavo Sousa
2026-06-09 20:17 ` [PATCH 2/9] drm/xe/step: Pass xe_step_info to xe_step_*_get() functions Gustavo Sousa
2026-06-16 19:31   ` Violet Monti
2026-06-09 20:17 ` [PATCH 3/9] drm/xe: Add devid and revid to xe_probed_info Gustavo Sousa
2026-06-16 19:33   ` Violet Monti
2026-06-09 20:17 ` [PATCH 4/9] drm/xe/step: Make xe_step_platform_get() independent from xe->info Gustavo Sousa
2026-06-16 19:37   ` Violet Monti
2026-06-09 20:17 ` [PATCH 5/9] drm/xe: Add platform-level step info to xe_probed_info Gustavo Sousa
2026-06-16 19:50   ` Violet Monti
2026-06-09 20:17 ` [PATCH 6/9] drm/xe/tests: Set non-GMDID graphics step in xe_pci_fake_device_init() Gustavo Sousa
2026-06-16 19:58   ` Violet Monti
2026-06-09 20:17 ` [PATCH 7/9] drm/xe: Add graphics/media IPs and their step info to xe_probed_info Gustavo Sousa
2026-06-16 20:22   ` Violet Monti
2026-06-09 20:17 ` [PATCH 8/9] drm/xe: Don't initialize tile_count in xe_info_init_early() Gustavo Sousa
2026-06-16 21:07   ` Violet Monti
2026-06-09 20:17 ` [PATCH 9/9] drm/xe: Add tile_count to xe_probed_info Gustavo Sousa
2026-06-16 21:16   ` Violet Monti
2026-06-09 20:58 ` ✓ CI.KUnit: success for drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Patchwork
2026-06-09 22:06 ` ✓ Xe.CI.BAT: " Patchwork
2026-06-10 12:22 ` ✗ Xe.CI.FULL: failure " 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.