All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org
Cc: jani.nikula@intel.com, Jocelyn Falempe <jfalempe@redhat.com>,
	Maarten Lankhorst <dev@lankhorst.se>
Subject: [PATCH 0/8] drm/{i915, xe}/panic: refactor framebuffer allocation etc.
Date: Tue,  2 Sep 2025 20:51:46 +0300	[thread overview]
Message-ID: <cover.1756835342.git.jani.nikula@intel.com> (raw)

Split out intel_panic functionality separate from intel_bo abstraction,
and lift the framebuffer allocation back to where it belongs in
intel_fb.c.

There's more that should be improved, ideally making struct
intel_framebuffer opaque towards i915 and xe, but this already goes
along way in adjusting where the abstractions should happen.

BR,
Jani.

Cc: Jocelyn Falempe <jfalempe@redhat.com>
Cc: Maarten Lankhorst <dev@lankhorst.se>


Jani Nikula (8):
  drm/i915/fb: add intel_framebuffer_alloc()
  drm/{i915,xe}/panic: split out intel_panic.[ch]
  drm/{i915,xe}/panic: rename intel_bo_panic_*() to intel_panic_*()
  drm/{i915,xe}/fb: add panic pointer member to struct intel_framebuffer
  drm/{i915,xe}/panic: rename struct {i915,xe}_panic_data to struct
    intel_panic
  drm/{i915,xe}/panic: move framebuffer allocation where it belongs
  drm/{i915,xe}/panic: convert intel_panic_finish() to struct
    intel_panic
  drm/{i915,xe}/panic: pass struct intel_panic to intel_panic_setup()

 drivers/gpu/drm/i915/Makefile                 |  1 +
 drivers/gpu/drm/i915/display/i9xx_plane.c     |  4 +-
 drivers/gpu/drm/i915/display/intel_bo.c       | 15 ---
 drivers/gpu/drm/i915/display/intel_bo.h       |  3 -
 .../drm/i915/display/intel_display_types.h    |  2 +
 drivers/gpu/drm/i915/display/intel_fb.c       | 23 ++++-
 drivers/gpu/drm/i915/display/intel_fb.h       |  3 +
 drivers/gpu/drm/i915/display/intel_panic.c    | 27 ++++++
 drivers/gpu/drm/i915/display/intel_panic.h    | 14 +++
 drivers/gpu/drm/i915/display/intel_plane.c    |  6 +-
 .../drm/i915/display/skl_universal_plane.c    |  3 +-
 drivers/gpu/drm/i915/gem/i915_gem_object.h    | 11 ++-
 drivers/gpu/drm/i915/gem/i915_gem_pages.c     | 40 +++-----
 drivers/gpu/drm/xe/Makefile                   |  1 +
 drivers/gpu/drm/xe/display/intel_bo.c         | 91 -------------------
 drivers/gpu/drm/xe/display/xe_panic.c         | 80 ++++++++++++++++
 16 files changed, 176 insertions(+), 148 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/display/intel_panic.c
 create mode 100644 drivers/gpu/drm/i915/display/intel_panic.h
 create mode 100644 drivers/gpu/drm/xe/display/xe_panic.c

-- 
2.47.2


             reply	other threads:[~2025-09-02 17:52 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-02 17:51 Jani Nikula [this message]
2025-09-02 17:51 ` [PATCH 1/8] drm/i915/fb: add intel_framebuffer_alloc() Jani Nikula
2025-09-02 17:51 ` [PATCH 2/8] drm/{i915,xe}/panic: split out intel_panic.[ch] Jani Nikula
2025-09-02 17:51 ` [PATCH 3/8] drm/{i915, xe}/panic: rename intel_bo_panic_*() to intel_panic_*() Jani Nikula
2025-09-02 17:51 ` [PATCH 4/8] drm/{i915, xe}/fb: add panic pointer member to struct intel_framebuffer Jani Nikula
2025-09-02 17:51 ` [PATCH 5/8] drm/{i915, xe}/panic: rename struct {i915, xe}_panic_data to struct intel_panic Jani Nikula
2025-09-02 17:51 ` [PATCH 6/8] drm/{i915, xe}/panic: move framebuffer allocation where it belongs Jani Nikula
2025-10-01 15:04   ` [PATCH 6/8] drm/{i915,xe}/panic: " Jani Nikula
2025-10-01 16:37     ` Ville Syrjälä
2025-10-01 17:28       ` Jani Nikula
2025-10-01 17:43         ` Ville Syrjälä
2025-09-02 17:51 ` [PATCH 7/8] drm/{i915, xe}/panic: convert intel_panic_finish() to struct intel_panic Jani Nikula
2025-09-02 17:51 ` [PATCH 8/8] drm/{i915, xe}/panic: pass struct intel_panic to intel_panic_setup() Jani Nikula
2025-09-02 18:48 ` ✗ CI.checkpatch: warning for drm/{i915, xe}/panic: refactor framebuffer allocation etc Patchwork
2025-09-02 18:50 ` ✓ CI.KUnit: success " Patchwork
2025-09-02 19:05 ` ✗ CI.checksparse: warning " Patchwork
2025-09-02 19:25 ` ✓ Xe.CI.BAT: success " Patchwork
2025-09-02 20:51 ` ✓ i915.CI.BAT: " Patchwork
2025-09-03  2:28 ` ✗ Xe.CI.Full: failure " Patchwork
2025-09-03  7:05 ` ✗ i915.CI.Full: " Patchwork
2025-09-04 11:46 ` [PATCH 0/8] " Jocelyn Falempe
2025-09-08 11:29   ` [PATCH 0/8] drm/{i915,xe}/panic: " Jani Nikula

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=cover.1756835342.git.jani.nikula@intel.com \
    --to=jani.nikula@intel.com \
    --cc=dev@lankhorst.se \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=jfalempe@redhat.com \
    /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 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.