Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/2] Cleaning up code related to VRAM regions and its initialization - part 1
@ 2025-02-03 13:53 Piórkowski, Piotr
  2025-02-03 13:53 ` [PATCH v2 1/2] drm/xe: Rename struct xe_mem_region to struct xe_vram_region Piórkowski, Piotr
                   ` (9 more replies)
  0 siblings, 10 replies; 13+ messages in thread
From: Piórkowski, Piotr @ 2025-02-03 13:53 UTC (permalink / raw)
  To: intel-xe
  Cc: Piotr Piórkowski, Thomas Hellström, Matthew Brost,
	Matthew Auld, Farah Kassabri, Rodrigo Vivi

From: Piotr Piórkowski <piotr.piorkowski@intel.com>

I would like to reorganize the code in the Xe driver regarding VRAM regions,
in order to allow in the future easier extension with more VRAM regions and
other than per tile ways to manage allocations in VRAM.
I would like to start with such simple changes as renaming the structure
xe_mem_region to xe_vram_region, to stop pretending that we use it for
something other than VRAM regions, and in the next patch move to this
structure, the field responsible for the TTM manager - this will allow us
to keep the things needed to manage allocations in VRAM in one place.

In future patches, I would like to make some changes in the initialization
of VRAM regions, and allow for dynamic allocation of these regions
depending on whether or not we need them in a particular case. Finally,
I would like to introduce the possibility of a separate VRAM region which
would be dedicated only for kernel structures, which may be necessary in
the case of SRIOV for future platforms.

Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Cc: Matthew Auld <matthew.auld@intel.com>
Cc: Farah Kassabri <farah.kassabri@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>

v2:
 - remove xe_vram_region pointer from xe_ttm_vram_mgr
 - stop use dynamic alloaction for xe_ttm_vram_mgr in xe_vram_region
 - rename struct xe_ttm_vram_mgr vram_mgr to ttm

Piotr Piórkowski (2):
  drm/xe: Rename struct xe_mem_region to struct xe_vram_region
  drm/xe: Move VRAM manager to struct xe_vram_region

 drivers/gpu/drm/xe/xe_bo.c                 | 22 ++++++++++++++--------
 drivers/gpu/drm/xe/xe_device_types.h       | 14 +++++++-------
 drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c |  2 +-
 drivers/gpu/drm/xe/xe_tile.c               |  6 +-----
 drivers/gpu/drm/xe/xe_ttm_vram_mgr.c       |  3 +--
 drivers/gpu/drm/xe/xe_ttm_vram_mgr_types.h |  4 ----
 6 files changed, 24 insertions(+), 27 deletions(-)

-- 
2.34.1


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

end of thread, other threads:[~2025-02-06 20:53 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-03 13:53 [PATCH v2 0/2] Cleaning up code related to VRAM regions and its initialization - part 1 Piórkowski, Piotr
2025-02-03 13:53 ` [PATCH v2 1/2] drm/xe: Rename struct xe_mem_region to struct xe_vram_region Piórkowski, Piotr
2025-02-06 20:52   ` Rodrigo Vivi
2025-02-03 13:53 ` [PATCH v2 2/2] drm/xe: Move VRAM manager " Piórkowski, Piotr
2025-02-06 20:52   ` Rodrigo Vivi
2025-02-03 15:25 ` ✓ CI.Patch_applied: success for Cleaning up code related to VRAM regions and its initialization - part 1 (rev2) Patchwork
2025-02-03 15:25 ` ✓ CI.checkpatch: " Patchwork
2025-02-03 15:26 ` ✓ CI.KUnit: " Patchwork
2025-02-03 15:42 ` ✓ CI.Build: " Patchwork
2025-02-03 15:45 ` ✗ CI.Hooks: failure " Patchwork
2025-02-03 15:46 ` ✓ CI.checksparse: success " Patchwork
2025-02-03 16:06 ` ✓ Xe.CI.BAT: " Patchwork
2025-02-03 17:09 ` ✗ Xe.CI.Full: failure " Patchwork

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