From: Ville Syrjala <ville.syrjala@linux.intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: intel-xe@lists.freedesktop.org
Subject: [PATCH v2 00/19] drm/i915/overlay: Convert to parent interface
Date: Thu, 26 Feb 2026 12:07:19 +0200 [thread overview]
Message-ID: <20260226100738.29997-1-ville.syrjala@linux.intel.com> (raw)
From: Ville Syrjälä <ville.syrjala@linux.intel.com>
Split the overlay code into i915 vs. display parts and introduce
and new parent interface for the display->i915 calls.
The interface is rather verbose due to the interruptible ioctl
design of the overlay code. Ideally we'd implement the overlay
as a drm plane and tell it to flip via MMIO writes to OVADD,
at which point the parent interface could be limited to just
two functions to repartition the render cache as needed.
Maybe one day...
Smoke tested on i830 and i965gm.
v2: Deal with the IS_ERR() screwup
Rebase due to kmalloc_obj() & co.
Ville Syrjälä (19):
drm/i915/overlay: Remove GPU hang snapshot stuff
drm/i915/overlay: Track current frontbuffer_bits
drm/i915/overlay: Extract i915_overlay_is_active()
drm/i915/overlay: Remove redundant overlay->active
drm/i915/overlay: Relocate the underrun check
drm/i915/overlay: Introduce i915_overlay_obj_lookup()
drm/i915/overlay: Use struct drm_gem_object as the type
drm/i915/overlay: Extract i915_overlay_reset()
drm/i915/overlay: Extract i915_overlay_setup()
drm/i915/overlay: Extract i915_overlay_cleanup()
drm/i915/overlay: Abstract buffer (un)pinning
drm/i915/overlay: Rename low level i915 specific functions
drm/i915/overlay: Adjust i915 specific interfaces
drm/i915/overlay: Make i830_overlay_clock_gating() i915 specific
drm/i915/overlay: s/dev_priv/i915/
drm/i915/overlay: Split 'struct intel_overlay'
drm/i915/overlay: Don't use fetch_and_zero() in display code
drm/i915/overlay: Move i915 specific code into i915_overlay.c
drm/i915/overlay: Convert overlay to parent interface
drivers/gpu/drm/i915/Makefile | 1 +
.../gpu/drm/i915/display/intel_display_regs.h | 2 -
.../drm/i915/display/intel_display_snapshot.c | 4 -
drivers/gpu/drm/i915/display/intel_overlay.c | 584 ++----------------
drivers/gpu/drm/i915/display/intel_overlay.h | 48 --
drivers/gpu/drm/i915/display/intel_parent.c | 76 +++
drivers/gpu/drm/i915/display/intel_parent.h | 26 +
drivers/gpu/drm/i915/i915_driver.c | 2 +
drivers/gpu/drm/i915/i915_drv.h | 3 +
drivers/gpu/drm/i915/i915_overlay.c | 517 ++++++++++++++++
drivers/gpu/drm/i915/i915_overlay.h | 11 +
drivers/gpu/drm/i915/i915_reg.h | 4 +
drivers/gpu/drm/xe/Makefile | 1 +
include/drm/intel/display_parent_interface.h | 33 +
14 files changed, 726 insertions(+), 586 deletions(-)
create mode 100644 drivers/gpu/drm/i915/i915_overlay.c
create mode 100644 drivers/gpu/drm/i915/i915_overlay.h
--
2.52.0
next reply other threads:[~2026-02-26 10:07 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-26 10:07 Ville Syrjala [this message]
2026-02-26 10:07 ` [PATCH v2 01/19] drm/i915/overlay: Remove GPU hang snapshot stuff Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 02/19] drm/i915/overlay: Track current frontbuffer_bits Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 03/19] drm/i915/overlay: Extract i915_overlay_is_active() Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 04/19] drm/i915/overlay: Remove redundant overlay->active Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 05/19] drm/i915/overlay: Relocate the underrun check Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 06/19] drm/i915/overlay: Introduce i915_overlay_obj_lookup() Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 07/19] drm/i915/overlay: Use struct drm_gem_object as the type Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 08/19] drm/i915/overlay: Extract i915_overlay_reset() Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 09/19] drm/i915/overlay: Extract i915_overlay_setup() Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 10/19] drm/i915/overlay: Extract i915_overlay_cleanup() Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 11/19] drm/i915/overlay: Abstract buffer (un)pinning Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 12/19] drm/i915/overlay: Rename low level i915 specific functions Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 13/19] drm/i915/overlay: Adjust i915 specific interfaces Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 14/19] drm/i915/overlay: Make i830_overlay_clock_gating() i915 specific Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 15/19] drm/i915/overlay: s/dev_priv/i915/ Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 16/19] drm/i915/overlay: Split 'struct intel_overlay' Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 17/19] drm/i915/overlay: Don't use fetch_and_zero() in display code Ville Syrjala
2026-02-26 13:01 ` [PATCH v3 " Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 18/19] drm/i915/overlay: Move i915 specific code into i915_overlay.c Ville Syrjala
2026-02-26 13:38 ` [PATCH v3 " Ville Syrjala
2026-02-26 10:07 ` [PATCH v2 19/19] drm/i915/overlay: Convert overlay to parent interface Ville Syrjala
2026-02-26 13:01 ` [PATCH v3 " Ville Syrjala
2026-02-26 10:15 ` ✗ CI.checkpatch: warning for drm/i915/overlay: Convert to parent interface (rev2) Patchwork
2026-02-26 10:16 ` ✓ CI.KUnit: success " Patchwork
2026-02-26 11:03 ` ✗ Xe.CI.BAT: failure " Patchwork
2026-02-26 12:27 ` ✗ Xe.CI.FULL: " Patchwork
2026-02-26 13:45 ` ✗ CI.checkpatch: warning for drm/i915/overlay: Convert to parent interface (rev5) Patchwork
2026-02-26 13:47 ` ✓ CI.KUnit: success " Patchwork
2026-02-26 14:42 ` ✓ Xe.CI.BAT: " Patchwork
2026-02-26 16:14 ` ✓ Xe.CI.FULL: " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260226100738.29997-1-ville.syrjala@linux.intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox