From: "Michael J. Ruhl" <michael.j.ruhl@intel.com>
To: platform-driver-x86@vger.kernel.org,
intel-xe@lists.freedesktop.org, hansg@kernel.org,
ilpo.jarvinen@linux.intel.com, matthew.brost@intel.com,
rodrigo.vivi@intel.com, thomas.hellstrom@linux.intel.com,
airlied@gmail.com, simona@ffwll.ch, david.e.box@linux.intel.com,
anoop.c.vijay@intel.com, badal.nilawar@intel.com,
matthew.d.roper@intel.com, james.ausmus@intel.com,
karthik.poosa@intel.com
Subject: [PATCH v9 00/20] Crescent Island PMT support
Date: Wed, 16 Sep 2026 13:11:38 -0700 [thread overview]
Message-ID: <20260916201139.2374531-22-michael.j.ruhl@intel.com> (raw)
The Crescent Island (CRI) GPU supports PMT via Xe driver
registration.
The CRI PMT MMIO mapping is shared for each PMT feature and it is
necessary to set an index register value to access the correct memory
space.
The PMT driver has a callback infrastructure to access data areas with
driver specific access. It is does not support the PMT control access.
CRI discovery/control spaces are part of the access window so a driver
callback is necessary to allow for the correct window to be exposed.
Add control path callback support to the PMT driver.
Add CRI PMT discovery information.
Update the Xe driver to support the CRI PMT access.
The crashlog access needs to be done with power enabled (telem is only
available when the device is powered).
Determine which access is requested, and do the appropriate power
request.
CRI FW is loaded at device power. Support the late-binding API to
wait for FW readiness. (pending sysctl patch set implements the API).
v2:
Address AI feedback:
- add missing entry->cb check
- address SRIOV VF concerns
- use xe_pm_runtime_put instead of _get in error path
Address review comments:
- redo power/lock sequencing
- support device hotplug events
v3:
- cleanup power manangement flow
- fixed an out of order power management locking issue
- add a GUID caching mechanism, for GUID read
- removed unused parameter (mmio)
v4:
- address various review comments
- refactor crashlog to allow for return values
- for callback usage, decouple PMT remap of data areas
- use IS_SRIOV_VF() rather than expose soc-remapper internals
- separate locking fixes intos distinct steps
- refactor define update patches to seperate BMG and CRI updates
- separate internal GUID into separate steps
v5:
- removed callback helpers
- do not return error message with sysfs
- fix missed return code check in trigger_store
- return error value not false
- use DEFINE_RES_MEM
- document when base is unused
- return error on invalid GUID request
v6:
- fix enabled/disabled inversion
- fix missing DOC reference
- change memcpy_to/fromio to readl/writel for register read
- guid cache comment and update on valid guid
v7:
- reordered to have BMG updates followed by CRI support
- separate base_offset usage for clarity
- added EAGAIN usage to block uninitialized use of the API
v8:
- fix punit crashlog offset (should be 0xE60 not 0xE78)
- remove EAGAIN support and re-order probe steps to initialize
VSEC before other features use it
v9:
- added appropriate reviewed-bys
- use dev_err instead of pr_err
- enforce defined base usage when callbacks are in plcace
- move SRIOV check to get_platform_info
- assert SRIOV for API check
- use scoped_guard() where possible
- update to use correct sysctrl helper
Michael J. Ruhl (18):
platform/x86/intel/pmt: complete pcidev to device update
platform/x86/intel/pmt: refactor rmw with a return value
platform/x86/intel/pmt: refactor rc with a return value
platform/x86/intel/pmt: Add register access callbacks
platform/x86/intel/pmt: Do not remap when using callbacks
drm/xe/vsec: Do not register BMG PMT for VF
drm/xe/vsec: Correct locking order
drm/xe/vsec: Use correct pm state get
drm/xe/vsec: Add DOC text for VSEC
drm/xe/vsec: Support possible hotplug exit
drm/xe/vsec: Refactor BattleMage PMT defines
drm/xe/vsec: Update VSEC probe order
drm/xe/vsec: Add base_offset to allow for more flexibilty
drm/xe/vsec: Support Crescent Island PMT
drm/xe/vsec: Crescent Island PMT decode
drm/xe/vsec: Crescent Island PMT callbacks
drm/xe/vsec: Support late bind fw information
drm/xe/vsec: Add PMT GUID internal access
drm/xe/vsec: Update PMT internal access
drm/xe/vsec: Refactor platform check
Documentation/gpu/xe/index.rst | 1 +
Documentation/gpu/xe/xe_vsec.rst | 14 +
drivers/gpu/drm/xe/regs/xe_pmt.h | 12 +-
drivers/gpu/drm/xe/xe_debugfs.c | 45 +-
drivers/gpu/drm/xe/xe_device.c | 6 +-
drivers/gpu/drm/xe/xe_device_types.h | 9 +
drivers/gpu/drm/xe/xe_hwmon.c | 10 +-
drivers/gpu/drm/xe/xe_pcode.c | 10 +-
drivers/gpu/drm/xe/xe_vsec.c | 552 +++++++++++++++++++--
drivers/gpu/drm/xe/xe_vsec.h | 3 +-
drivers/platform/x86/intel/pmt/class.c | 26 +-
drivers/platform/x86/intel/pmt/class.h | 5 +-
drivers/platform/x86/intel/pmt/crashlog.c | 237 ++++++---
drivers/platform/x86/intel/pmt/discovery.c | 2 +-
drivers/platform/x86/intel/pmt/telemetry.c | 3 +
include/linux/intel_vsec.h | 14 +-
16 files changed, 782 insertions(+), 167 deletions(-)
create mode 100644 Documentation/gpu/xe/xe_vsec.rst
--
2.43.0
next reply other threads:[~2026-09-16 20:11 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-16 20:11 Michael J. Ruhl [this message]
2026-09-16 20:11 ` [PATCH v9 01/20] platform/x86/intel/pmt: complete pcidev to device update Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 02/20] platform/x86/intel/pmt: refactor rmw with a return value Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 03/20] platform/x86/intel/pmt: refactor rc " Michael J. Ruhl
2026-09-16 20:20 ` sashiko-bot
2026-09-17 9:55 ` Ilpo Järvinen
2026-09-16 20:11 ` [PATCH v9 04/20] platform/x86/intel/pmt: Add register access callbacks Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 05/20] platform/x86/intel/pmt: Do not remap when using callbacks Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 06/20] drm/xe/vsec: Do not register BMG PMT for VF Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 07/20] drm/xe/vsec: Correct locking order Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 08/20] drm/xe/vsec: Use correct pm state get Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 09/20] drm/xe/vsec: Add DOC text for VSEC Michael J. Ruhl
2026-09-16 20:29 ` sashiko-bot
2026-09-17 9:56 ` Ilpo Järvinen
2026-09-17 9:57 ` Ilpo Järvinen
2026-09-17 12:55 ` Ruhl, Michael J
2026-09-16 20:11 ` [PATCH v9 10/20] drm/xe/vsec: Support possible hotplug exit Michael J. Ruhl
2026-09-16 20:21 ` sashiko-bot
2026-09-16 20:11 ` [PATCH v9 11/20] drm/xe/vsec: Refactor BattleMage PMT defines Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 12/20] drm/xe/vsec: Update VSEC probe order Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 13/20] drm/xe/vsec: Add base_offset to allow for more flexibilty Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 14/20] drm/xe/vsec: Support Crescent Island PMT Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 15/20] drm/xe/vsec: Crescent Island PMT decode Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 16/20] drm/xe/vsec: Crescent Island PMT callbacks Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 17/20] drm/xe/vsec: Support late bind fw information Michael J. Ruhl
2026-09-16 20:29 ` sashiko-bot
2026-09-17 12:30 ` Ruhl, Michael J
2026-09-16 20:11 ` [PATCH v9 18/20] drm/xe/vsec: Add PMT GUID internal access Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 19/20] drm/xe/vsec: Update PMT " Michael J. Ruhl
2026-09-16 20:11 ` [PATCH v9 20/20] drm/xe/vsec: Refactor platform check Michael J. Ruhl
2026-09-16 20:44 ` ✗ CI.checkpatch: warning for Crescent Island PMT support (rev11) Patchwork
2026-09-16 20:46 ` ✓ CI.KUnit: success " Patchwork
2026-09-16 22:00 ` ✗ Xe.CI.BAT: failure " Patchwork
2026-09-17 1:39 ` ✗ 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=20260916201139.2374531-22-michael.j.ruhl@intel.com \
--to=michael.j.ruhl@intel.com \
--cc=airlied@gmail.com \
--cc=anoop.c.vijay@intel.com \
--cc=badal.nilawar@intel.com \
--cc=david.e.box@linux.intel.com \
--cc=hansg@kernel.org \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=intel-xe@lists.freedesktop.org \
--cc=james.ausmus@intel.com \
--cc=karthik.poosa@intel.com \
--cc=matthew.brost@intel.com \
--cc=matthew.d.roper@intel.com \
--cc=platform-driver-x86@vger.kernel.org \
--cc=rodrigo.vivi@intel.com \
--cc=simona@ffwll.ch \
--cc=thomas.hellstrom@linux.intel.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.