Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/13] drm/i915/dpt: Optimize i915 DPT
@ 2026-11-11 11:19 Ville Syrjala
  2026-11-11 11:19 ` [PATCH 01/13] drm/i915: Bump surface alignment to 32k on MTL for async flips Ville Syrjala
                   ` (16 more replies)
  0 siblings, 17 replies; 20+ messages in thread
From: Ville Syrjala @ 2026-11-11 11:19 UTC (permalink / raw)
  To: intel-gfx; +Cc: intel-xe

From: Ville Syrjälä <ville.syrjala@linux.intel.com>

My original i915 DPT implementation has one annoying performance
flaw. It creates a new DPT for every new framebuffers, which
given your typical dumb userspace implementation happens every
single frame. The intel ddx might be the only userspace implementation
with a userspace side cache of framebuffers, but the intel ddx
does not even support any platform with DPT, so not relevant
here.

Take a page out of the i915 vma book and cache the DPTs in the 
gem object instead. This way the same DPT will get reused as
long as userspace keeps rotating through the same set of gem 
objects (which is the case).

As a bonus the whole DPT parent interface disappears from the
display side.

Ville Syrjälä (13):
  drm/i915: Bump surface alignment to 32k on MTL for async flips
  drm/i915/fb: Fix error handling onion in intel_framebuffer_init()
  drm/i915/dpt: Stop allocation DPTs from stolen
  drm/i915/fb_pin: Remove overzealous vma NULL checks
  drm/i915: intel_{remapped,rotation}_info_size() into
    intel/gtt_view_types.h
  drm/i915/dpt: Compute DPT size in the DPT code
  drm/i915/vma: Extract intel_gtt_view_compare()
  drm/i915/vma: Introduce i915_vma_lookup()
  drm/i915/dpt: Make i915_dpt_{suspend,resume}() internal to i915
  drm/i915/dpt: Cache DPTs in the gem object
  drm/i915/dpt: Hide the DPT even more from the display code
  drm/i915/dpt: Remove dpt->vma
  drm/i915/dpt: s/struct intel_dpt/struct i915_dpt/

 .../drm/i915/display/intel_display_driver.c   |   2 +-
 .../drm/i915/display/intel_display_types.h    |   3 -
 drivers/gpu/drm/i915/display/intel_dpt.c      |  58 -----
 drivers/gpu/drm/i915/display/intel_dpt.h      |   2 -
 drivers/gpu/drm/i915/display/intel_fb.c       |  64 +----
 drivers/gpu/drm/i915/display/intel_fb.h       |   3 -
 drivers/gpu/drm/i915/display/intel_parent.c   |  40 +--
 drivers/gpu/drm/i915/display/intel_parent.h   |  14 +-
 drivers/gpu/drm/i915/display/intel_plane.c    |  13 +-
 .../drm/i915/display/skl_universal_plane.c    |   9 +
 drivers/gpu/drm/i915/gem/i915_gem_object.c    |   6 +
 .../gpu/drm/i915/gem/i915_gem_object_types.h  |   6 +
 drivers/gpu/drm/i915/gt/intel_ggtt.c          |  10 +
 drivers/gpu/drm/i915/gt/intel_gtt.h           |   4 +
 drivers/gpu/drm/i915/i915_dpt.c               | 233 ++++++++++++++----
 drivers/gpu/drm/i915/i915_dpt.h               |  21 +-
 drivers/gpu/drm/i915/i915_driver.c            |   8 -
 drivers/gpu/drm/i915/i915_fb_pin.c            |  56 +++--
 drivers/gpu/drm/i915/i915_vma.c               |  42 +++-
 drivers/gpu/drm/i915/i915_vma.h               |  49 ++--
 drivers/gpu/drm/xe/display/xe_fb_pin.c        |  15 +-
 drivers/gpu/drm/xe/display/xe_initial_plane.c |   3 +-
 include/drm/intel/display_parent_interface.h  |  17 +-
 include/drm/intel/gtt_view_types.h            |  38 +++
 24 files changed, 400 insertions(+), 316 deletions(-)

-- 
2.56.0


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

end of thread, other threads:[~2026-11-11 20:29 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-11-11 11:19 [PATCH 00/13] drm/i915/dpt: Optimize i915 DPT Ville Syrjala
2026-11-11 11:19 ` [PATCH 01/13] drm/i915: Bump surface alignment to 32k on MTL for async flips Ville Syrjala
2026-11-11 11:19 ` [PATCH 02/13] drm/i915/fb: Fix error handling onion in intel_framebuffer_init() Ville Syrjala
2026-11-11 11:19 ` [PATCH 03/13] drm/i915/dpt: Stop allocation DPTs from stolen Ville Syrjala
2026-11-11 11:19 ` [PATCH 04/13] drm/i915/fb_pin: Remove overzealous vma NULL checks Ville Syrjala
2026-11-11 11:19 ` [PATCH 05/13] drm/i915: intel_{remapped, rotation}_info_size() into intel/gtt_view_types.h Ville Syrjala
2026-11-11 11:20 ` [PATCH 06/13] drm/i915/dpt: Compute DPT size in the DPT code Ville Syrjala
2026-11-11 11:31   ` sashiko-bot
2026-11-11 12:39     ` Ville Syrjälä
2026-11-11 11:20 ` [PATCH 07/13] drm/i915/vma: Extract intel_gtt_view_compare() Ville Syrjala
2026-11-11 11:20 ` [PATCH 08/13] drm/i915/vma: Introduce i915_vma_lookup() Ville Syrjala
2026-11-11 11:20 ` [PATCH 09/13] drm/i915/dpt: Make i915_dpt_{suspend, resume}() internal to i915 Ville Syrjala
2026-11-11 11:20 ` [PATCH 10/13] drm/i915/dpt: Cache DPTs in the gem object Ville Syrjala
2026-11-11 11:20 ` [PATCH 11/13] drm/i915/dpt: Hide the DPT even more from the display code Ville Syrjala
2026-11-11 11:20 ` [PATCH 12/13] drm/i915/dpt: Remove dpt->vma Ville Syrjala
2026-11-11 11:20 ` [PATCH 13/13] drm/i915/dpt: s/struct intel_dpt/struct i915_dpt/ Ville Syrjala
2026-11-11 12:28 ` ✗ CI.checkpatch: warning for drm/i915/dpt: Optimize i915 DPT Patchwork
2026-11-11 12:30 ` ✓ CI.KUnit: success " Patchwork
2026-11-11 13:05 ` ✓ Xe.CI.BAT: " Patchwork
2026-11-11 20:29 ` ✓ 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