* [PULL] drm-intel-gt-next
@ 2023-12-08 16:18 Joonas Lahtinen
0 siblings, 0 replies; 24+ messages in thread
From: Joonas Lahtinen @ 2023-12-08 16:18 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: dim-tools, dri-devel, Maxime Ripard, Thomas Zimmermann,
Rodrigo Vivi, intel-gfx
Hi Dave & Sima,
A rather late first drm-intel-gt-next PR towards v6.8.
As most significant change we have addition of the DRM fdinfo memory stats
functionality. Then DG2 and MTL workaround additions and fixes and a few
for older platforms as well. PMU WARN_ON splat cleanup.
The rest is mostly code cleanups and fixes for odd corner cases.
Best Regards, Joonas
***
drm-intel-gt-next-2023-12-08:
UAPI Changes:
- drm/i915: Implement fdinfo memory stats printing
Use the newly added drm_print_memory_stats helper to show memory
utilisation of our objects in drm/driver specific fdinfo output.
To collect the stats we walk the per memory regions object lists
and accumulate object size into the respective drm_memory_stats
categories.
Cross-subsystem Changes:
- Backmerge of drm-next (to bring drm-intel-next for PXP changes)
Driver Changes:
- Wa_18028616096 now applies to all DG2 (Matt R)
- Drop Wa_22014600077 on all DG2 (Matt R)
- Add new ATS-M device ID (Haridhar)
- More Meteorlake (MTL) workarounds (Matt R, Dnyaneshwar, Jonathan,
Gustavo, Radhakrishna)
- PMU WARN_ON cleanup on driver unbind (Umesh)
- Limit GGTT WC flushing workaround to pre BXT/ICL platforms
- Complement implementation for Wa_16018031267 / Wa_16018063123
(Andrzej, Jonathan, Nirmoy, Chris)
- Properly print internal GSC engine in trace logs (Tvrtko)
- Track gt pm wakerefs (Andrzej)
- Fix null deref bugs on perf code when perf is disabled (Harshit,
Tvrtko)
- Fix __i915_request_create memory leak on driver unbind (Andrzej)
- Remove spurious unsupported HuC message on MTL (Daniele)
- Read a shadowed mmio register for ggtt flush (Vinay)
- Add missing new-line to GT_TRACE (Andrzej)
- Add drm_dbgs for critical PXP events (Alan)
- Skip pxp init if gt is wedged (Zhanjun)
- Replace custom intel runtime_pm tracker with ref_tracker library
(Andrzej)
- Compiler warning/static checker/coding style cleanups (Arnd, Nirmoy,
Soumya, Gilbert, Dorcas, Kunwu, Sam, Tvrtko)
- Code structure and helper cleanups (Jani, Tvrtko, Andi)
- Selftest improvements (John, Tvrtko, Andrzej)
The following changes since commit 11ae5eb516b656e8a0e4efbea90ea24c152a346d:
Merge tag 'topic/vmemdup-user-array-2023-10-24-1' of git://anongit.freedesktop.org/drm/drm into drm-next (2023-10-24 11:13:29 +1000)
are available in the Git repository at:
git://anongit.freedesktop.org/drm/drm-intel tags/drm-intel-gt-next-2023-12-08
for you to fetch changes up to be5bcc4be9d9d3ae294072441a66fe39b74e5bba:
drm/i915/guc: Create the guc_to_i915() wrapper (2023-12-08 12:31:01 +0100)
----------------------------------------------------------------
UAPI Changes:
- drm/i915: Implement fdinfo memory stats printing
Use the newly added drm_print_memory_stats helper to show memory
utilisation of our objects in drm/driver specific fdinfo output.
To collect the stats we walk the per memory regions object lists
and accumulate object size into the respective drm_memory_stats
categories.
Cross-subsystem Changes:
- Backmerge of drm-next (to bring drm-intel-next for PXP changes)
Driver Changes:
- Wa_18028616096 now applies to all DG2 (Matt R)
- Drop Wa_22014600077 on all DG2 (Matt R)
- Add new ATS-M device ID (Haridhar)
- More Meteorlake (MTL) workarounds (Matt R, Dnyaneshwar, Jonathan,
Gustavo, Radhakrishna)
- PMU WARN_ON cleanup on driver unbind (Umesh)
- Limit GGTT WC flushing workaround to pre BXT/ICL platforms
- Complement implementation for Wa_16018031267 / Wa_16018063123
(Andrzej, Jonathan, Nirmoy, Chris)
- Properly print internal GSC engine in trace logs (Tvrtko)
- Track gt pm wakerefs (Andrzej)
- Fix null deref bugs on perf code when perf is disabled (Harshit,
Tvrtko)
- Fix __i915_request_create memory leak on driver unbind (Andrzej)
- Remove spurious unsupported HuC message on MTL (Daniele)
- Read a shadowed mmio register for ggtt flush (Vinay)
- Add missing new-line to GT_TRACE (Andrzej)
- Add drm_dbgs for critical PXP events (Alan)
- Skip pxp init if gt is wedged (Zhanjun)
- Replace custom intel runtime_pm tracker with ref_tracker library
(Andrzej)
- Compiler warning/static checker/coding style cleanups (Arnd, Nirmoy,
Soumya, Gilbert, Dorcas, Kunwu, Sam, Tvrtko)
- Code structure and helper cleanups (Jani, Tvrtko, Andi)
- Selftest improvements (John, Tvrtko, Andrzej)
----------------------------------------------------------------
Alan Previn (1):
drm/i915/pxp: Add drm_dbgs for critical PXP events.
Andi Shyti (1):
drm/i915/guc: Create the guc_to_i915() wrapper
Andrzej Hajda (8):
drm/i915: Reserve some kernel space per vm
drm/i915: Add WABB blit for Wa_16018031267 / Wa_16018063123
drm/i915/gt: add selftest to exercise WABB
drm/i915/gt: add missing new-line to GT_TRACE
drm/i915: do not clean GT table on error path
drm/i915: Replace custom intel runtime_pm tracker with ref_tracker library
drm/i915: Track gt pm wakerefs
drm/i915/selftests: wait for active idle event in i915_active_unlock_wait
Arnd Bergmann (1):
drm/i915/mtl: avoid stringop-overflow warning
Daniele Ceraolo Spurio (1):
drm/i915/huc: Stop printing about unsupported HuC on MTL
Dnyaneshwar Bhadane (2):
drm/i915/mtl: Add Wa_22016670082
drm/i915/mtl: Add Wa_14019821291
Dorcas AnonoLitunya (1):
drm/i915/gt: Remove prohibited space after opening parenthesis
Gilbert Adikankwu (1):
drm/i915/gt: Remove unncessary {} from if-else
Gustavo Sousa (1):
drm/i915/xelpmp: Add Wa_16021867713
Haridhar Kalvala (1):
drm/i915: ATS-M device ID update
Harshit Mogalapalli (1):
i915/perf: Fix NULL deref bugs with drm_dbg() calls
Jani Nikula (6):
drm/i915/pmu: add pmu_to_i915() helper
drm/i915/pmu: add event_to_pmu() helper
drm/i915/pmu: rearrange hrtimer pointer chasing
drm/i915: make some error capture functions static
drm/i915: move gpu error debugfs to i915_gpu_error.c
drm/i915: move gpu error sysfs to i915_gpu_error.c
John Harrison (2):
drm/i915/guc: Fix for potential false positives in GuC hang selftest
drm/i915/guc: Add a selftest for FAST_REQUEST errors
Jonathan Cavitt (2):
drm/i915: Set copy engine arbitration for Wa_16018031267 / Wa_16018063123
drm/i915/gt: Temporarily disable CPU caching into DMA for MTL
Kunwu Chan (1):
drm/i915: Fix potential spectre vulnerability
Matt Roper (3):
drm/i915/mcr: Hold GT forcewake during steering operations
drm/i915/dg2: Wa_18028616096 now applies to all DG2
drm/i915/dg2: Drop Wa_22014600077
Nirmoy Das (2):
drm/i915/gt: Use proper priority enum instead of 0
drm/i915: Flush WC GGTT only on required platforms
Radhakrishna Sripada (1):
drm/i915/mtl: Update Wa_22018931422
Sam James (1):
drm: i915: Adapt to -Walloc-size
Soumya Negi (1):
drm/i915/gt: Remove {} from if-else
Tvrtko Ursulin (12):
Merge drm/drm-next into drm-intel-gt-next
drm/i915: Add ability for tracking buffer objects per client
drm/i915: Record which client owns a VM
drm/i915: Track page table backing store usage
drm/i915: Account ring buffer and context state storage
drm/i915: Add stable memory region names
drm/i915: Implement fdinfo memory stats printing
drm/i915: Remove return type from i915_drm_client_remove_object
drm/i915: Add __rcu annotation to cursor when iterating client objects
drm/i915/gsc: Mark internal GSC engine with reserved uabi class
drm/i915/selftests: Fix engine reset count storage for multi-tile
drm/i915: Use internal class when counting engine resets
Umesh Nerlige Ramappa (1):
drm/i915/pmu: Check if pmu is closed before stopping event
Vinay Belgaumkar (1):
drm/i915: Read a shadowed mmio register for ggtt flush
Zhanjun Dong (1):
drm/i915: Skip pxp init if gt is wedged
drivers/gpu/drm/i915/Kconfig.debug | 18 ++
drivers/gpu/drm/i915/display/intel_display_power.c | 2 +-
drivers/gpu/drm/i915/gem/i915_gem_context.c | 12 +-
drivers/gpu/drm/i915/gem/i915_gem_context_types.h | 3 +
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 16 +-
drivers/gpu/drm/i915/gem/i915_gem_object.c | 13 +-
drivers/gpu/drm/i915/gem/i915_gem_object_types.h | 12 ++
drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 21 ++
.../drm/i915/gem/selftests/i915_gem_coherency.c | 10 +-
drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c | 14 +-
drivers/gpu/drm/i915/gem/selftests/mock_context.c | 4 +-
drivers/gpu/drm/i915/gt/gen8_ppgtt.c | 43 ++++
drivers/gpu/drm/i915/gt/intel_breadcrumbs.c | 13 +-
drivers/gpu/drm/i915/gt/intel_breadcrumbs_types.h | 3 +-
drivers/gpu/drm/i915/gt/intel_context.c | 14 ++
drivers/gpu/drm/i915/gt/intel_context.h | 4 +-
drivers/gpu/drm/i915/gt/intel_context_types.h | 2 +
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +-
drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c | 2 +-
drivers/gpu/drm/i915/gt/intel_engine_pm.c | 7 +-
drivers/gpu/drm/i915/gt/intel_engine_regs.h | 8 +
drivers/gpu/drm/i915/gt/intel_engine_types.h | 2 +
drivers/gpu/drm/i915/gt/intel_engine_user.c | 39 ++--
.../gpu/drm/i915/gt/intel_execlists_submission.c | 2 +-
drivers/gpu/drm/i915/gt/intel_ggtt.c | 58 ++++--
drivers/gpu/drm/i915/gt/intel_gt.c | 13 +-
drivers/gpu/drm/i915/gt/intel_gt.h | 9 +
drivers/gpu/drm/i915/gt/intel_gt_mcr.c | 23 ++-
drivers/gpu/drm/i915/gt/intel_gt_pm.c | 14 +-
drivers/gpu/drm/i915/gt/intel_gt_pm.h | 38 +++-
drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c | 4 +-
drivers/gpu/drm/i915/gt/intel_gt_regs.h | 3 +
drivers/gpu/drm/i915/gt/intel_gtt.c | 26 +++
drivers/gpu/drm/i915/gt/intel_gtt.h | 5 +
drivers/gpu/drm/i915/gt/intel_lrc.c | 100 +++++++++-
drivers/gpu/drm/i915/gt/intel_rc6.c | 16 +-
drivers/gpu/drm/i915/gt/intel_reset.c | 2 +-
drivers/gpu/drm/i915/gt/intel_sseu.c | 7 +-
drivers/gpu/drm/i915/gt/intel_workarounds.c | 38 ++--
drivers/gpu/drm/i915/gt/selftest_engine_cs.c | 20 +-
.../gpu/drm/i915/gt/selftest_engine_heartbeat.c | 2 +-
drivers/gpu/drm/i915/gt/selftest_gt_pm.c | 5 +-
drivers/gpu/drm/i915/gt/selftest_lrc.c | 65 ++++--
drivers/gpu/drm/i915/gt/selftest_reset.c | 10 +-
drivers/gpu/drm/i915/gt/selftest_rps.c | 17 +-
drivers/gpu/drm/i915/gt/selftest_slpc.c | 5 +-
drivers/gpu/drm/i915/gt/uc/intel_gsc_proxy.c | 2 +
drivers/gpu/drm/i915/gt/uc/intel_guc.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_guc.h | 4 +
drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 11 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 10 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_rc.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 23 ++-
drivers/gpu/drm/i915/gt/uc/intel_uc.c | 5 -
drivers/gpu/drm/i915/gt/uc/selftest_guc.c | 115 +++++++++++
.../gpu/drm/i915/gt/uc/selftest_guc_hangcheck.c | 2 +-
drivers/gpu/drm/i915/i915_debugfs.c | 108 +---------
drivers/gpu/drm/i915/i915_driver.c | 10 +-
drivers/gpu/drm/i915/i915_drm_client.c | 108 ++++++++++
drivers/gpu/drm/i915/i915_drm_client.h | 42 ++++
drivers/gpu/drm/i915/i915_gpu_error.c | 194 +++++++++++++++++-
drivers/gpu/drm/i915/i915_gpu_error.h | 44 ++--
drivers/gpu/drm/i915/i915_perf.c | 15 +-
drivers/gpu/drm/i915/i915_pmu.c | 86 ++++----
drivers/gpu/drm/i915/i915_sysfs.c | 79 +-------
drivers/gpu/drm/i915/intel_memory_region.c | 19 ++
drivers/gpu/drm/i915/intel_memory_region.h | 1 +
drivers/gpu/drm/i915/intel_runtime_pm.c | 221 ++-------------------
drivers/gpu/drm/i915/intel_runtime_pm.h | 11 +-
drivers/gpu/drm/i915/intel_wakeref.c | 35 +++-
drivers/gpu/drm/i915/intel_wakeref.h | 73 ++++++-
drivers/gpu/drm/i915/pxp/intel_pxp.c | 18 +-
drivers/gpu/drm/i915/pxp/intel_pxp_irq.c | 5 +-
drivers/gpu/drm/i915/pxp/intel_pxp_session.c | 6 +-
drivers/gpu/drm/i915/pxp/intel_pxp_types.h | 1 +
drivers/gpu/drm/i915/selftests/igt_live_test.c | 9 +-
drivers/gpu/drm/i915/selftests/igt_live_test.h | 3 +-
include/drm/i915_pciids.h | 3 +-
80 files changed, 1322 insertions(+), 695 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2023-12-15 12:11 Joonas Lahtinen
0 siblings, 0 replies; 24+ messages in thread
From: Joonas Lahtinen @ 2023-12-15 12:11 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: dim-tools, dri-devel, Maxime Ripard, Thomas Zimmermann,
Rodrigo Vivi, intel-gfx
Hi Dave & Sima,
Final drm-intel-gt-next PR for v6.8.
Elimination of kmap_atomic() from the driver to allow kernel wide
cleanup. One new DG2 W/A and static checker/spelling fixes.
Best Regards,
Joonas
***
drm-intel-gt-next-2023-12-15:
Driver Changes:
- Eliminate use of kmap_atomic() in i915 (Zhao)
- Add Wa_14019877138 for DG2 (Haridhar)
- Static checker and spelling fixes (Colin, Karthik, Randy)
-
The following changes since commit be5bcc4be9d9d3ae294072441a66fe39b74e5bba:
drm/i915/guc: Create the guc_to_i915() wrapper (2023-12-08 12:31:01 +0100)
are available in the Git repository at:
git://anongit.freedesktop.org/drm/drm-intel tags/drm-intel-gt-next-2023-12-15
for you to fetch changes up to 31accc37eaee98a90b25809ed58c6ee4956ab642:
drm/i915: Use kmap_local_page() in gem/i915_gem_execbuffer.c (2023-12-15 09:34:31 +0000)
----------------------------------------------------------------
Driver Changes:
- Eliminate use of kmap_atomic() in i915 (Zhao)
- Add Wa_14019877138 for DG2 (Haridhar)
- Static checker and spelling fixes (Colin, Karthik, Randy)
-
----------------------------------------------------------------
Colin Ian King (1):
drm/i915/selftests: Fix spelling mistake "initialiased" -> "initialised"
Haridhar Kalvala (1):
drm/i915: Add Wa_14019877138
Karthik Poosa (1):
drm/i915/hwmon: Fix static analysis tool reported issues
Randy Dunlap (1):
drm/i915/uapi: fix typos/spellos and punctuation
Zhao Liu (9):
drm/i915: Use kmap_local_page() in gem/i915_gem_object.c
drm/i915: Use memcpy_[from/to]_page() in gem/i915_gem_pyhs.c
drm/i915: Use kmap_local_page() in gem/i915_gem_shmem.c
drm/i915: Use kmap_local_page() in gem/selftests/huge_pages.c
drm/i915: Use kmap_local_page() in gem/selftests/i915_gem_coherency.c
drm/i915: Use kmap_local_page() in gem/selftests/i915_gem_context.c
drm/i915: Use memcpy_from_page() in gt/uc/intel_uc_fw.c
drm/i915: Use kmap_local_page() in i915_cmd_parser.c
drm/i915: Use kmap_local_page() in gem/i915_gem_execbuffer.c
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 10 +++++-----
drivers/gpu/drm/i915/gem/i915_gem_object.c | 8 +++-----
drivers/gpu/drm/i915/gem/i915_gem_phys.c | 10 ++--------
drivers/gpu/drm/i915/gem/i915_gem_shmem.c | 6 ++++--
drivers/gpu/drm/i915/gem/selftests/huge_pages.c | 6 +++---
drivers/gpu/drm/i915/gem/selftests/i915_gem_coherency.c | 12 ++++--------
drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c | 8 ++++----
drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c | 2 +-
drivers/gpu/drm/i915/gt/intel_gt_regs.h | 3 +++
drivers/gpu/drm/i915/gt/intel_workarounds.c | 3 +++
drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 5 +----
drivers/gpu/drm/i915/i915_cmd_parser.c | 4 ++--
drivers/gpu/drm/i915/i915_hwmon.c | 4 ++--
include/uapi/drm/i915_drm.h | 12 ++++++------
14 files changed, 43 insertions(+), 50 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2024-02-15 10:06 Tvrtko Ursulin
2024-02-16 2:58 ` Dave Airlie
0 siblings, 1 reply; 24+ messages in thread
From: Tvrtko Ursulin @ 2024-02-15 10:06 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, dim-tools
Hi Dave, Daniel,
First pull request for 6.9 with probably one more coming in one to two
weeks.
Nothing to interesting in this one, mostly a sprinkle of small fixes in
GuC, HuC, Perf/OA, a tiny bit of prep work for future platforms and some
code cleanups.
One new uapi in the form of a GuC submission version query which Mesa
wants for implementing Vulkan async compute queues.
Regards,
Tvrtko
drm-intel-gt-next-2024-02-15:
UAPI Changes:
- Add GuC submission interface version query (Tvrtko Ursulin)
Driver Changes:
Fixes/improvements/new stuff:
- Atomically invalidate userptr on mmu-notifier (Jonathan Cavitt)
- Update handling of MMIO triggered reports (Umesh Nerlige Ramappa)
- Don't make assumptions about intel_wakeref_t type (Jani Nikula)
- Add workaround 14019877138 [xelpg] (Tejas Upadhyay)
- Allow for very slow HuC loading [huc] (John Harrison)
- Flush context destruction worker at suspend [guc] (Alan Previn)
- Close deregister-context race against CT-loss [guc] (Alan Previn)
- Avoid circular locking issue on busyness flush [guc] (John Harrison)
- Use rc6.supported flag from intel_gt for rc6_enable sysfs (Juan Escamilla)
- Reflect the true and current status of rc6_enable (Juan Escamilla)
- Wake GT before sending H2G message [mtl] (Vinay Belgaumkar)
- Restart the heartbeat timer when forcing a pulse (John Harrison)
Future platform enablement:
- Extend driver code of Xe_LPG to Xe_LPG+ [xelpg] (Harish Chegondi)
- Extend some workarounds/tuning to gfx version 12.74 [xelpg] (Matt Roper)
Miscellaneous:
- Reconcile Excess struct member kernel-doc warnings (Randy Dunlap)
- Change wa and EU_PERF_CNTL registers to MCR type [guc] (Shuicheng Lin)
- Add flex arrays to struct i915_syncmap (Erick Archer)
- Increasing the sleep time for live_rc6_manual [selftests] (Anirban Sk)
The following changes since commit 31accc37eaee98a90b25809ed58c6ee4956ab642:
drm/i915: Use kmap_local_page() in gem/i915_gem_execbuffer.c (2023-12-15 09:34:31 +0000)
are available in the Git repository at:
git://anongit.freedesktop.org/drm/drm-intel tags/drm-intel-gt-next-2024-02-15
for you to fetch changes up to eb927f01dfb6309c8a184593c2c0618c4000c481:
drm/i915/gt: Restart the heartbeat timer when forcing a pulse (2024-02-14 17:17:35 -0800)
----------------------------------------------------------------
UAPI Changes:
- Add GuC submission interface version query (Tvrtko Ursulin)
Driver Changes:
Fixes/improvements/new stuff:
- Atomically invalidate userptr on mmu-notifier (Jonathan Cavitt)
- Update handling of MMIO triggered reports (Umesh Nerlige Ramappa)
- Don't make assumptions about intel_wakeref_t type (Jani Nikula)
- Add workaround 14019877138 [xelpg] (Tejas Upadhyay)
- Allow for very slow HuC loading [huc] (John Harrison)
- Flush context destruction worker at suspend [guc] (Alan Previn)
- Close deregister-context race against CT-loss [guc] (Alan Previn)
- Avoid circular locking issue on busyness flush [guc] (John Harrison)
- Use rc6.supported flag from intel_gt for rc6_enable sysfs (Juan Escamilla)
- Reflect the true and current status of rc6_enable (Juan Escamilla)
- Wake GT before sending H2G message [mtl] (Vinay Belgaumkar)
- Restart the heartbeat timer when forcing a pulse (John Harrison)
Future platform enablement:
- Extend driver code of Xe_LPG to Xe_LPG+ [xelpg] (Harish Chegondi)
- Extend some workarounds/tuning to gfx version 12.74 [xelpg] (Matt Roper)
Miscellaneous:
- Reconcile Excess struct member kernel-doc warnings (Randy Dunlap)
- Change wa and EU_PERF_CNTL registers to MCR type [guc] (Shuicheng Lin)
- Add flex arrays to struct i915_syncmap (Erick Archer)
- Increasing the sleep time for live_rc6_manual [selftests] (Anirban Sk)
----------------------------------------------------------------
Alan Previn (2):
drm/i915/guc: Flush context destruction worker at suspend
drm/i915/guc: Close deregister-context race against CT-loss
Anirban Sk (1):
drm/i915/selftests: Increasing the sleep time for live_rc6_manual
Erick Archer (1):
drm/i915: Add flex arrays to struct i915_syncmap
Harish Chegondi (1):
drm/i915/xelpg: Extend driver code of Xe_LPG to Xe_LPG+
Jani Nikula (1):
drm/i915: don't make assumptions about intel_wakeref_t type
John Harrison (3):
drm/i915/huc: Allow for very slow HuC loading
drm/i915/guc: Avoid circular locking issue on busyness flush
drm/i915/gt: Restart the heartbeat timer when forcing a pulse
Jonathan Cavitt (1):
drm/i915/gem: Atomically invalidate userptr on mmu-notifier
Juan Escamilla (2):
drm/i915/gt: Use rc6.supported flag from intel_gt for rc6_enable sysfs
drm/i915/gt: Reflect the true and current status of rc6_enable
Matt Roper (1):
drm/i915/xelpg: Extend some workarounds/tuning to gfx version 12.74
Randy Dunlap (4):
drm/i915/gem: reconcile Excess struct member kernel-doc warnings
drm/i915/gt: reconcile Excess struct member kernel-doc warnings
drm/i915/guc: reconcile Excess struct member kernel-doc warnings
drm/i915/perf: reconcile Excess struct member kernel-doc warnings
Shuicheng Lin (1):
drm/i915/guc: Change wa and EU_PERF_CNTL registers to MCR type
Tejas Upadhyay (1):
drm/i915/xelpg: Add workaround 14019877138
Tvrtko Ursulin (1):
drm/i915: Add GuC submission interface version query
Umesh Nerlige Ramappa (1):
drm/i915/perf: Update handling of MMIO triggered reports
Vinay Belgaumkar (1):
drm/i915/mtl: Wake GT before sending H2G message
drivers/gpu/drm/i915/display/intel_display_power.c | 4 +-
drivers/gpu/drm/i915/gem/i915_gem_context_types.h | 4 +-
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 8 --
drivers/gpu/drm/i915/gem/i915_gem_pm.c | 10 ++
drivers/gpu/drm/i915/gem/i915_gem_userptr.c | 42 -------
drivers/gpu/drm/i915/gem/i915_gem_userptr.h | 14 ---
drivers/gpu/drm/i915/gt/gen8_engine_cs.c | 4 +-
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 3 +-
drivers/gpu/drm/i915/gt/intel_engine_heartbeat.c | 3 +
drivers/gpu/drm/i915/gt/intel_gsc.h | 7 +-
drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c | 18 +--
drivers/gpu/drm/i915/gt/intel_mocs.c | 2 +-
drivers/gpu/drm/i915/gt/intel_rc6.c | 2 +-
drivers/gpu/drm/i915/gt/intel_workarounds.c | 27 +++--
drivers/gpu/drm/i915/gt/selftest_rc6.c | 4 +-
drivers/gpu/drm/i915/gt/uc/intel_guc.h | 75 ++++++------
drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 21 ++--
drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c | 10 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 126 ++++++++++++++++++++-
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.h | 2 +
drivers/gpu/drm/i915/gt/uc/intel_huc.c | 64 ++++++++++-
drivers/gpu/drm/i915/gt/uc/intel_uc.c | 4 +-
drivers/gpu/drm/i915/i915_debugfs.c | 2 +-
drivers/gpu/drm/i915/i915_drv.h | 8 --
drivers/gpu/drm/i915/i915_gem.c | 5 -
drivers/gpu/drm/i915/i915_perf.c | 41 ++++++-
drivers/gpu/drm/i915/i915_perf_types.h | 9 +-
drivers/gpu/drm/i915/i915_query.c | 33 ++++++
drivers/gpu/drm/i915/i915_syncmap.c | 19 ++--
drivers/gpu/drm/i915/intel_uncore.c | 5 +-
include/uapi/drm/i915_drm.h | 12 ++
31 files changed, 392 insertions(+), 196 deletions(-)
delete mode 100644 drivers/gpu/drm/i915/gem/i915_gem_userptr.h
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PULL] drm-intel-gt-next
2024-02-15 10:06 Tvrtko Ursulin
@ 2024-02-16 2:58 ` Dave Airlie
2024-02-16 9:31 ` Thomas Hellström
2024-02-16 9:41 ` Joonas Lahtinen
0 siblings, 2 replies; 24+ messages in thread
From: Dave Airlie @ 2024-02-16 2:58 UTC (permalink / raw)
To: Tvrtko Ursulin
Cc: Daniel Vetter, Jani Nikula, Joonas Lahtinen, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, dim-tools
On Thu, 15 Feb 2024 at 20:06, Tvrtko Ursulin
<tvrtko.ursulin@linux.intel.com> wrote:
>
> Hi Dave, Daniel,
>
> First pull request for 6.9 with probably one more coming in one to two
> weeks.
>
> Nothing to interesting in this one, mostly a sprinkle of small fixes in
> GuC, HuC, Perf/OA, a tiny bit of prep work for future platforms and some
> code cleanups.
>
> One new uapi in the form of a GuC submission version query which Mesa
> wants for implementing Vulkan async compute queues.
>
> Regards,
>
> Tvrtko
>
> drm-intel-gt-next-2024-02-15:
> UAPI Changes:
>
> - Add GuC submission interface version query (Tvrtko Ursulin)
>
> Driver Changes:
>
> Fixes/improvements/new stuff:
>
> - Atomically invalidate userptr on mmu-notifier (Jonathan Cavitt)
I've pulled this, but the above patch is triggering my this seems
wrong spider sense.
This and probably the preceeding patch that this references seem to
move i915 to a long term pinning of userptr in memory with what I can
see no accounting, and away from what the desired behaviour for
drivers should be.
It also feels like the authorship on this might be lies which also worries me.
Dave.
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PULL] drm-intel-gt-next
2024-02-16 2:58 ` Dave Airlie
@ 2024-02-16 9:31 ` Thomas Hellström
2024-02-16 9:33 ` Thomas Hellström
2024-02-16 9:41 ` Joonas Lahtinen
1 sibling, 1 reply; 24+ messages in thread
From: Thomas Hellström @ 2024-02-16 9:31 UTC (permalink / raw)
To: Dave Airlie, Tvrtko Ursulin
Cc: Daniel Vetter, Jani Nikula, Joonas Lahtinen, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard, Oded Gabbay,
Lucas De Marchi, dri-devel, intel-gfx, dim-tools
Hi, Dave
On Fri, 2024-02-16 at 12:58 +1000, Dave Airlie wrote:
> On Thu, 15 Feb 2024 at 20:06, Tvrtko Ursulin
> <tvrtko.ursulin@linux.intel.com> wrote:
> >
> > Hi Dave, Daniel,
> >
> > First pull request for 6.9 with probably one more coming in one to
> > two
> > weeks.
> >
> > Nothing to interesting in this one, mostly a sprinkle of small
> > fixes in
> > GuC, HuC, Perf/OA, a tiny bit of prep work for future platforms and
> > some
> > code cleanups.
> >
> > One new uapi in the form of a GuC submission version query which
> > Mesa
> > wants for implementing Vulkan async compute queues.
> >
> > Regards,
> >
> > Tvrtko
> >
> > drm-intel-gt-next-2024-02-15:
> > UAPI Changes:
> >
> > - Add GuC submission interface version query (Tvrtko Ursulin)
> >
> > Driver Changes:
> >
> > Fixes/improvements/new stuff:
> >
> > - Atomically invalidate userptr on mmu-notifier (Jonathan Cavitt)
>
> I've pulled this, but the above patch is triggering my this seems
> wrong spider sense.
>
> This and probably the preceeding patch that this references seem to
> move i915 to a long term pinning of userptr in memory with what I can
> see no accounting, and away from what the desired behaviour for
> drivers should be.
I can only answer for the first patch there, It was some time ago it
was written, but at that point the pinning was held both by get_pages()
and by submission. I removed the submission pinning and instead moved
get_pages() to start of submission. So no significant change in pinning
time there. For some reason I can't clearly remember the submission
pinning got in the way of the vm_bind implementation. That said, the
pinning AFAIR is released in the gem shrinker. And it's different from
what other drivers are doing. i915 never got to the point where it
completely dropped the pinning after the binding.
/Thomas
>
> It also feels like the authorship on this might be lies which also
> worries me.
>
> Dave.
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PULL] drm-intel-gt-next
2024-02-16 9:31 ` Thomas Hellström
@ 2024-02-16 9:33 ` Thomas Hellström
0 siblings, 0 replies; 24+ messages in thread
From: Thomas Hellström @ 2024-02-16 9:33 UTC (permalink / raw)
To: Dave Airlie, Tvrtko Ursulin
Cc: Daniel Vetter, Jani Nikula, Joonas Lahtinen, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard, Oded Gabbay,
Lucas De Marchi, dri-devel, intel-gfx, dim-tools
On Fri, 2024-02-16 at 10:31 +0100, Thomas Hellström wrote:
> Hi, Dave
>
> On Fri, 2024-02-16 at 12:58 +1000, Dave Airlie wrote:
> > On Thu, 15 Feb 2024 at 20:06, Tvrtko Ursulin
> > <tvrtko.ursulin@linux.intel.com> wrote:
> > >
> > > Hi Dave, Daniel,
> > >
> > > First pull request for 6.9 with probably one more coming in one
> > > to
> > > two
> > > weeks.
> > >
> > > Nothing to interesting in this one, mostly a sprinkle of small
> > > fixes in
> > > GuC, HuC, Perf/OA, a tiny bit of prep work for future platforms
> > > and
> > > some
> > > code cleanups.
> > >
> > > One new uapi in the form of a GuC submission version query which
> > > Mesa
> > > wants for implementing Vulkan async compute queues.
> > >
> > > Regards,
> > >
> > > Tvrtko
> > >
> > > drm-intel-gt-next-2024-02-15:
> > > UAPI Changes:
> > >
> > > - Add GuC submission interface version query (Tvrtko Ursulin)
> > >
> > > Driver Changes:
> > >
> > > Fixes/improvements/new stuff:
> > >
> > > - Atomically invalidate userptr on mmu-notifier (Jonathan Cavitt)
> >
> > I've pulled this, but the above patch is triggering my this seems
> > wrong spider sense.
> >
> > This and probably the preceeding patch that this references seem to
> > move i915 to a long term pinning of userptr in memory with what I
> > can
> > see no accounting, and away from what the desired behaviour for
> > drivers should be.
>
> I can only answer for the first patch there, It was some time ago it
> was written, but at that point the pinning was held both by
> get_pages()
> and by submission. I removed the submission pinning and instead moved
> get_pages() to start of submission. So no significant change in
> pinning
> time there. For some reason I can't clearly remember the submission
> pinning got in the way of the vm_bind implementation. That said, the
> pinning AFAIR is released in the gem shrinker. And it's different
> from
> what other drivers are doing. i915 never got to the point where it
> completely dropped the pinning after the binding.
(And with the first patch I mean "Simplify userptr locking")
/Thomas
>
> /Thomas
>
>
> >
> > It also feels like the authorship on this might be lies which also
> > worries me.
> >
> > Dave.
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PULL] drm-intel-gt-next
2024-02-16 2:58 ` Dave Airlie
2024-02-16 9:31 ` Thomas Hellström
@ 2024-02-16 9:41 ` Joonas Lahtinen
2024-02-20 15:14 ` Joonas Lahtinen
1 sibling, 1 reply; 24+ messages in thread
From: Joonas Lahtinen @ 2024-02-16 9:41 UTC (permalink / raw)
To: Dave Airlie, Tvrtko Ursulin
Cc: Daniel Vetter, Jani Nikula, Rodrigo Vivi, Thomas Zimmermann,
Maarten Lankhorst, Maxime Ripard, Thomas Hellström,
Oded Gabbay, Lucas De Marchi, dri-devel, intel-gfx, dim-tools,
Jonathan Cavitt
(+ Jonathan)
Quoting Dave Airlie (2024-02-16 04:58:03)
> On Thu, 15 Feb 2024 at 20:06, Tvrtko Ursulin
> <tvrtko.ursulin@linux.intel.com> wrote:
> >
> > Hi Dave, Daniel,
> >
> > First pull request for 6.9 with probably one more coming in one to two
> > weeks.
> >
> > Nothing to interesting in this one, mostly a sprinkle of small fixes in
> > GuC, HuC, Perf/OA, a tiny bit of prep work for future platforms and some
> > code cleanups.
> >
> > One new uapi in the form of a GuC submission version query which Mesa
> > wants for implementing Vulkan async compute queues.
> >
> > Regards,
> >
> > Tvrtko
> >
> > drm-intel-gt-next-2024-02-15:
> > UAPI Changes:
> >
> > - Add GuC submission interface version query (Tvrtko Ursulin)
> >
> > Driver Changes:
> >
> > Fixes/improvements/new stuff:
> >
> > - Atomically invalidate userptr on mmu-notifier (Jonathan Cavitt)
>
> I've pulled this, but the above patch is triggering my this seems
> wrong spider sense.
>
> This and probably the preceeding patch that this references seem to
> move i915 to a long term pinning of userptr in memory with what I can
> see no accounting, and away from what the desired behaviour for
> drivers should be.
I asked Thomas to take a more detailed look. Jonathan, Thomas really should
have been Cc'd in the original patch as the patch was explicitly referred
in the text even.
> It also feels like the authorship on this might be lies which also worries me.
Fear not. This can probably be blamed on the i915 maintainers.
When we have an internal patch which has many revisions and is then
essentially rewritten for upstreaming, we specifically asked NOT to keep
the "From:" line intact, but instead swap in person who rewrote the patch[1].
To document credits/involvement of the original author we've recommended
to keep the Signed-off-by line however. "Co-developed-by" does not really
express the situation correctly. "Based on patch by" style pure textual
credit reference was also discussed but is hard to grep.
Discussed with Sima who suggested if we should consider something like
"Original-patch-by:" tag to better express this situation?
Regards, Joonas
[1] If the "From: " line is not updated, it sometimes leads to
situation where you can see a patch with "From:" pointing to you, that
doesn't contain a single unmodified line anymore.
>
> Dave.
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PULL] drm-intel-gt-next
2024-02-16 9:41 ` Joonas Lahtinen
@ 2024-02-20 15:14 ` Joonas Lahtinen
0 siblings, 0 replies; 24+ messages in thread
From: Joonas Lahtinen @ 2024-02-20 15:14 UTC (permalink / raw)
To: Dave Airlie, Tvrtko Ursulin
Cc: Daniel Vetter, Jani Nikula, Rodrigo Vivi, Thomas Zimmermann,
Maarten Lankhorst, Maxime Ripard, Thomas Hellstrom, Oded Gabbay,
Lucas De Marchi, dri-devel, intel-gfx, dim-tools, Jonathan Cavitt
Quoting Joonas Lahtinen (2024-02-16 11:41:44)
> (+ Jonathan)
>
> Quoting Dave Airlie (2024-02-16 04:58:03)
> > On Thu, 15 Feb 2024 at 20:06, Tvrtko Ursulin
> > <tvrtko.ursulin@linux.intel.com> wrote:
> > >
> > > Hi Dave, Daniel,
> > >
> > > First pull request for 6.9 with probably one more coming in one to two
> > > weeks.
> > >
> > > Nothing to interesting in this one, mostly a sprinkle of small fixes in
> > > GuC, HuC, Perf/OA, a tiny bit of prep work for future platforms and some
> > > code cleanups.
> > >
> > > One new uapi in the form of a GuC submission version query which Mesa
> > > wants for implementing Vulkan async compute queues.
> > >
> > > Regards,
> > >
> > > Tvrtko
> > >
> > > drm-intel-gt-next-2024-02-15:
> > > UAPI Changes:
> > >
> > > - Add GuC submission interface version query (Tvrtko Ursulin)
> > >
> > > Driver Changes:
> > >
> > > Fixes/improvements/new stuff:
> > >
> > > - Atomically invalidate userptr on mmu-notifier (Jonathan Cavitt)
> >
> > I've pulled this, but the above patch is triggering my this seems
> > wrong spider sense.
> >
> > This and probably the preceeding patch that this references seem to
> > move i915 to a long term pinning of userptr in memory with what I can
> > see no accounting, and away from what the desired behaviour for
> > drivers should be.
>
> I asked Thomas to take a more detailed look. Jonathan, Thomas really should
> have been Cc'd in the original patch as the patch was explicitly referred
> in the text even.
>
> > It also feels like the authorship on this might be lies which also worries me.
>
> Fear not. This can probably be blamed on the i915 maintainers.
>
> When we have an internal patch which has many revisions and is then
> essentially rewritten for upstreaming, we specifically asked NOT to keep
> the "From:" line intact, but instead swap in person who rewrote the patch[1].
Just to state the obvious for the public record:
This should never be done lightly or without reaching out to the
original author. This should only be for the exceptional cases where the
patch has significantly changed.
This was just the explanation why it's not an immediate red flag to see
such a patch. Based on the discussion around the topic we should be more
explicit if such a case has happened or if there simply has been an error
in the patch handling.
So we'll work on clarifying the instructions here.
Regards, Joonas
> To document credits/involvement of the original author we've recommended
> to keep the Signed-off-by line however. "Co-developed-by" does not really
> express the situation correctly. "Based on patch by" style pure textual
> credit reference was also discussed but is hard to grep.
>
> Discussed with Sima who suggested if we should consider something like
> "Original-patch-by:" tag to better express this situation?
>
> Regards, Joonas
>
> [1] If the "From: " line is not updated, it sometimes leads to
> situation where you can see a patch with "From:" pointing to you, that
> doesn't contain a single unmodified line anymore.
>
> >
> > Dave.
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2024-02-28 14:02 Tvrtko Ursulin
0 siblings, 0 replies; 24+ messages in thread
From: Tvrtko Ursulin @ 2024-02-28 14:02 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, dim-tools
Hi Dave, Sima,
Last drm-intel-gt-next pull request for 6.9.
There are only two small fixes in there so could also wait for the
-next-fixes round if so would be preferred. One fix is for a kerneldoc
warning and other for a very unlikely userptr object creation failure
where cleanup would oops.
Regards,
Tvrtko
drm-intel-gt-next-2024-02-28:
Driver Changes:
Fixes:
- Add some boring kerneldoc (Tvrtko Ursulin)
- Check before removing mm notifier (Nirmoy
The following changes since commit eb927f01dfb6309c8a184593c2c0618c4000c481:
drm/i915/gt: Restart the heartbeat timer when forcing a pulse (2024-02-14 17:17:35 -0800)
are available in the Git repository at:
git://anongit.freedesktop.org/drm/drm-intel tags/drm-intel-gt-next-2024-02-28
for you to fetch changes up to db7bbd13f08774cde0332c705f042e327fe21e73:
drm/i915: Check before removing mm notifier (2024-02-28 13:11:32 +0000)
----------------------------------------------------------------
Driver Changes:
Fixes:
- Add some boring kerneldoc (Tvrtko Ursulin)
- Check before removing mm notifier (Nirmoy
----------------------------------------------------------------
Nirmoy Das (1):
drm/i915: Check before removing mm notifier
Tvrtko Ursulin (1):
drm/i915: Add some boring kerneldoc
drivers/gpu/drm/i915/gem/i915_gem_userptr.c | 3 +++
include/uapi/drm/i915_drm.h | 4 ++++
2 files changed, 7 insertions(+)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2024-04-26 7:17 Joonas Lahtinen
0 siblings, 0 replies; 24+ messages in thread
From: Joonas Lahtinen @ 2024-04-26 7:17 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave & Sima,
Here's the drm-intel-gt-next PR for v6.10 in one shot.
We are adding a new uAPI for Mesa to request higher GT frequency for
compute contexts on GuC platform.
Then there is a W/A for DG2 to move to fixed CCS load balancing and
make all DG2 SKUs appear with single CCS with all the EUs attached by
default. Read more below under "UAPI Changes". There is one reported
regression against it which we're working on resolving, so expect to
see -next-fixes shortly once that happens.
Beyond that we have a bunch of workaround updates/fixes, fix for UAF
that has been hunted down for a while, GT reset fix for platforms that
load GuC but don't submit via it, fix for execlists priority submission,
proper capture of EIR register on hang.
THe rest is usual code cleanups/refactoring and selftest fixes.
Regards, Joonas
***
drm-intel-gt-next-2024-04-26:
UAPI Changes:
- drm/i915/guc: Use context hints for GT frequency
Allow user to provide a low latency context hint. When set, KMD
sends a hint to GuC which results in special handling for this
context. SLPC will ramp the GT frequency aggressively every time
it switches to this context. The down freq threshold will also be
lower so GuC will ramp down the GT freq for this context more slowly.
We also disable waitboost for this context as that will interfere with
the strategy.
We need to enable the use of SLPC Compute strategy during init, but
it will apply only to contexts that set this bit during context
creation.
Userland can check whether this feature is supported using a new param-
I915_PARAM_HAS_CONTEXT_FREQ_HINT. This flag is true for all guc submission
enabled platforms as they use SLPC for frequency management.
The Mesa usage model for this flag is here -
https://gitlab.freedesktop.org/sushmave/mesa/-/commits/compute_hint
- drm/i915/gt: Enable only one CCS for compute workload
Enable only one CCS engine by default with all the compute sices
allocated to it.
While generating the list of UABI engines to be exposed to the
user, exclude any additional CCS engines beyond the first
instance
***
NOTE: This W/A will make all DG2 SKUs appear like single CCS SKUs by
default to mitigate a hardware bug. All the EUs will still remain
usable, and all the userspace drivers have been confirmed to be able
to dynamically detect the change in number of CCS engines and adjust.
For the smaller percent of applications that get perf benefit from
letting the userspace driver dispatch across all 4 CCS engines we will
be introducing a sysfs control as a later patch to choose 4 CCS each
with 25% EUs (or 50% if 2 CCS).
NOTE: A regression has been reported at
https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10895
However Andi has been triaging the issue and we're closing in a fix
to the gap in the W/A implementation:
https://lists.freedesktop.org/archives/intel-gfx/2024-April/348747.html
Driver Changes:
- Add new and fix to existing workarounds: Wa_14018575942 (MTL),
Wa_16019325821 (Gen12.70), Wa_14019159160 (MTL), Wa_16015675438,
Wa_14020495402 (Gen12.70) (Tejas, John, Lucas)
- Fix UAF on destroy against retire race and remove two earlier
partial fixes (Janusz)
- Limit the reserved VM space to only the platforms that need it (Andi)
- Reset queue_priority_hint on parking for execlist platforms (Chris)
- Fix gt reset with GuC submission is disabled (Nirmoy)
- Correct capture of EIR register on hang (John)
- Remove usage of the deprecated ida_simple_xx() API
- Refactor confusing __intel_gt_reset() (Nirmoy)
- Fix the fix for GuC reset lock confusion (John)
- Simplify/extend platform check for Wa_14018913170 (John)
- Replace dev_priv with i915 (Andi)
- Add and use gt_to_guc() wrapper (Andi)
- Remove bogus null check (Rodrigo, Dan)
. Selftest improvements (Janusz, Nirmoy, Daniele)
The following changes since commit db7bbd13f08774cde0332c705f042e327fe21e73:
drm/i915: Check before removing mm notifier (2024-02-28 13:11:32 +0000)
are available in the Git repository at:
https://anongit.freedesktop.org/git/drm/drm-intel tags/drm-intel-gt-next-2024-04-26
for you to fetch changes up to 4d3421e04c5dc38baf15224c051256204f223c15:
drm/i915: Fix gt reset with GuC submission is disabled (2024-04-24 18:48:32 +0200)
----------------------------------------------------------------
UAPI Changes:
- drm/i915/guc: Use context hints for GT frequency
Allow user to provide a low latency context hint. When set, KMD
sends a hint to GuC which results in special handling for this
context. SLPC will ramp the GT frequency aggressively every time
it switches to this context. The down freq threshold will also be
lower so GuC will ramp down the GT freq for this context more slowly.
We also disable waitboost for this context as that will interfere with
the strategy.
We need to enable the use of SLPC Compute strategy during init, but
it will apply only to contexts that set this bit during context
creation.
Userland can check whether this feature is supported using a new param-
I915_PARAM_HAS_CONTEXT_FREQ_HINT. This flag is true for all guc submission
enabled platforms as they use SLPC for frequency management.
The Mesa usage model for this flag is here -
https://gitlab.freedesktop.org/sushmave/mesa/-/commits/compute_hint
- drm/i915/gt: Enable only one CCS for compute workload
Enable only one CCS engine by default with all the compute sices
allocated to it.
While generating the list of UABI engines to be exposed to the
user, exclude any additional CCS engines beyond the first
instance
***
NOTE: This W/A will make all DG2 SKUs appear like single CCS SKUs by
default to mitigate a hardware bug. All the EUs will still remain
usable, and all the userspace drivers have been confirmed to be able
to dynamically detect the change in number of CCS engines and adjust.
For the smaller percent of applications that get perf benefit from
letting the userspace driver dispatch across all 4 CCS engines we will
be introducing a sysfs control as a later patch to choose 4 CCS each
with 25% EUs (or 50% if 2 CCS).
NOTE: A regression has been reported at
https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10895
However Andi has been triaging the issue and we're closing in a fix
to the gap in the W/A implementation:
https://lists.freedesktop.org/archives/intel-gfx/2024-April/348747.html
Driver Changes:
- Add new and fix to existing workarounds: Wa_14018575942 (MTL),
Wa_16019325821 (Gen12.70), Wa_14019159160 (MTL), Wa_16015675438,
Wa_14020495402 (Gen12.70) (Tejas, John, Lucas)
- Fix UAF on destroy against retire race and remove two earlier
partial fixes (Janusz)
- Limit the reserved VM space to only the platforms that need it (Andi)
- Reset queue_priority_hint on parking for execlist platforms (Chris)
- Fix gt reset with GuC submission is disabled (Nirmoy)
- Correct capture of EIR register on hang (John)
- Remove usage of the deprecated ida_simple_xx() API
- Refactor confusing __intel_gt_reset() (Nirmoy)
- Fix the fix for GuC reset lock confusion (John)
- Simplify/extend platform check for Wa_14018913170 (John)
- Replace dev_priv with i915 (Andi)
- Add and use gt_to_guc() wrapper (Andi)
- Remove bogus null check (Rodrigo, Dan)
. Selftest improvements (Janusz, Nirmoy, Daniele)
----------------------------------------------------------------
Andi Shyti (7):
drm/i915/gt: Create the gt_to_guc() wrapper
drm/i915/guc: Use the new gt_to_guc() wrapper
drm/i915/gt: Limit the reserved VM space to only the platforms that need it
drm/i915/gem: Replace dev_priv with i915
drm/i915/gt: Disable HW load balancing for CCS
drm/i915/gt: Do not generate the command streamer for all the CCS
drm/i915/gt: Enable only one CCS for compute workload
Chris Wilson (1):
drm/i915/gt: Reset queue_priority_hint on parking
Christophe JAILLET (1):
drm/i915/guc: Remove usage of the deprecated ida_simple_xx() API
Daniele Ceraolo Spurio (1):
drm/i915/dg2: wait for HuC load completion before running selftests
Janusz Krzysztofik (5):
drm/i915/selftest_hangcheck: Check sanity with more patience
drm/i915/selftests: Fix dependency of some timeouts on HZ
drm/i915/vma: Fix UAF on destroy against retire race
drm/i915: Remove extra multi-gt pm-references
Revert "drm/i915: Wait for active retire before i915_active_fini()"
John Harrison (6):
drm/i915/guc: Correct capture of EIR register on hang
drm/i915/guc: Simplify/extend platform check for Wa_14018913170
drm/i915: Enable Wa_16019325821
drm/i915/guc: Add support for w/a KLVs
drm/i915/guc: Enable Wa_14019159160
drm/i915/guc: Fix the fix for reset lock confusion
Lucas De Marchi (1):
drm/i915: Drop WA 16015675438
Nirmoy Das (3):
drm/i915/selftests: Pick correct caching mode.
drm/i915: Refactor confusing __intel_gt_reset()
drm/i915: Fix gt reset with GuC submission is disabled
Radhakrishna Sripada (1):
drm/i915/xelpg: Add Wa_14020495402
Rodrigo Vivi (1):
drm/i915/guc: Remove bogus null check
Tejas Upadhyay (1):
drm/i915/mtl: Update workaround 14018575942
Vinay Belgaumkar (1):
drm/i915/guc: Use context hints for GT frequency
drivers/gpu/drm/i915/Makefile | 1 +
drivers/gpu/drm/i915/gem/i915_gem_context.c | 16 +++-
drivers/gpu/drm/i915/gem/i915_gem_context_types.h | 1 +
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 22 +-----
drivers/gpu/drm/i915/gem/i915_gem_shmem.c | 6 +-
drivers/gpu/drm/i915/gem/i915_gem_stolen.h | 8 +-
drivers/gpu/drm/i915/gem/i915_gem_tiling.c | 18 ++---
drivers/gpu/drm/i915/gem/i915_gem_userptr.c | 6 +-
drivers/gpu/drm/i915/gem/selftests/huge_pages.c | 14 ++--
.../gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c | 5 +-
drivers/gpu/drm/i915/gt/gen8_engine_cs.c | 22 ++++--
drivers/gpu/drm/i915/gt/gen8_ppgtt.c | 3 +
drivers/gpu/drm/i915/gt/intel_context_types.h | 1 +
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 23 +++++-
drivers/gpu/drm/i915/gt/intel_engine_pm.c | 3 -
drivers/gpu/drm/i915/gt/intel_engine_types.h | 8 +-
.../gpu/drm/i915/gt/intel_execlists_submission.c | 5 +-
drivers/gpu/drm/i915/gt/intel_ggtt.c | 9 +--
drivers/gpu/drm/i915/gt/intel_gt.c | 8 +-
drivers/gpu/drm/i915/gt/intel_gt.h | 14 +++-
drivers/gpu/drm/i915/gt/intel_gt_ccs_mode.c | 39 ++++++++++
drivers/gpu/drm/i915/gt/intel_gt_ccs_mode.h | 13 ++++
drivers/gpu/drm/i915/gt/intel_gt_irq.c | 6 +-
drivers/gpu/drm/i915/gt/intel_gt_pm.c | 2 +-
drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c | 2 +-
drivers/gpu/drm/i915/gt/intel_gt_regs.h | 7 ++
drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c | 8 +-
drivers/gpu/drm/i915/gt/intel_rc6.c | 4 +-
drivers/gpu/drm/i915/gt/intel_reset.c | 51 ++++++++++--
drivers/gpu/drm/i915/gt/intel_reset.h | 3 +-
drivers/gpu/drm/i915/gt/intel_rps.c | 6 +-
drivers/gpu/drm/i915/gt/intel_tlb.c | 2 +-
drivers/gpu/drm/i915/gt/intel_workarounds.c | 43 ++++++++--
drivers/gpu/drm/i915/gt/selftest_hangcheck.c | 2 +-
drivers/gpu/drm/i915/gt/selftest_reset.c | 2 +-
drivers/gpu/drm/i915/gt/selftest_slpc.c | 6 +-
.../gpu/drm/i915/gt/uc/abi/guc_actions_slpc_abi.h | 21 +++++
drivers/gpu/drm/i915/gt/uc/abi/guc_errors_abi.h | 1 +
drivers/gpu/drm/i915/gt/uc/abi/guc_klvs_abi.h | 7 ++
drivers/gpu/drm/i915/gt/uc/intel_gsc_fw.c | 4 +-
drivers/gpu/drm/i915/gt/uc/intel_gsc_proxy.c | 3 +-
drivers/gpu/drm/i915/gt/uc/intel_guc.c | 15 +++-
drivers/gpu/drm/i915/gt/uc/intel_guc.h | 2 +
drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 91 +++++++++++++++++++++-
drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c | 12 +--
drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c | 6 ++
drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h | 8 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 17 ++++
drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.h | 1 +
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 78 ++++++++++---------
drivers/gpu/drm/i915/gt/uc/intel_huc.c | 4 +-
drivers/gpu/drm/i915/gt/uc/intel_uc.c | 4 +
drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 4 +-
drivers/gpu/drm/i915/gt/uc/selftest_guc.c | 2 +-
drivers/gpu/drm/i915/i915_driver.c | 2 +-
drivers/gpu/drm/i915/i915_getparam.c | 6 ++
drivers/gpu/drm/i915/i915_vma.c | 52 ++++++++++---
drivers/gpu/drm/i915/selftests/i915_selftest.c | 36 ++++++++-
.../drm/i915/selftests/intel_scheduler_helpers.c | 6 +-
include/uapi/drm/i915_drm.h | 15 ++++
61 files changed, 598 insertions(+), 190 deletions(-)
create mode 100644 drivers/gpu/drm/i915/gt/intel_gt_ccs_mode.c
create mode 100644 drivers/gpu/drm/i915/gt/intel_gt_ccs_mode.h
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2024-06-12 12:57 Tvrtko Ursulin
0 siblings, 0 replies; 24+ messages in thread
From: Tvrtko Ursulin @ 2024-06-12 12:57 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave, Sima,
Here is the main pull request for drm-intel-gt-next targeting 6.11.
First is the new userspace API for allowing upload of custom context
state used for replaying GPU hang error state captures. This will be
used by Mesa (see
https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27594) for
debugging GPU hangs captured in the wild on real hardware. So far that
was only possible under simulation and that via some hacks. Also,
simulation in general has certain limitations to what hangs it can
reproduce. As the UAPI it is intended for Mesa developers only, it is
hidden behind a kconfig and runtime enablement switches.
Then there are fixes for hangs on Meteorlake due incorrect reduced CCS
configuration and a missing video engine workaround. Then fixes for a
couple race conditions in multi GT and breadcrumb handling, and a more
robust functional level reset by extending the timeout used.
A couple tiny cleanups here and there and finally one back-merge which
was required to land some display code base refactoring.
Regards,
Tvrtko
drm-intel-gt-next-2024-06-12:
UAPI Changes:
- Support replaying GPU hangs with captured context image (Tvrtko Ursulin)
Driver Changes:
Fixes/improvements/new stuff:
- Automate CCS Mode setting during engine resets [gt] (Andi Shyti)
- Revert "drm/i915: Remove extra multi-gt pm-references" (Janusz Krzysztofik)
- Fix HAS_REGION() usage in intel_gt_probe_lmem() (Ville Syrjälä)
- Disarm breadcrumbs if engines are already idle [gt] (Chris Wilson)
- Shadow default engine context image in the context (Tvrtko Ursulin)
- Support replaying GPU hangs with captured context image (Tvrtko Ursulin)
- avoid FIELD_PREP warning [guc] (Arnd Bergmann)
- Fix CCS id's calculation for CCS mode setting [gt] (Andi Shyti)
- Increase FLR timeout from 3s to 9s (Andi Shyti)
- Update workaround 14018575942 [mtl] (Angus Chen)
Future platform enablement:
- Enable w/a 16021333562 for DG2, MTL and ARL [guc] (John Harrison)
Miscellaneous:
- Pass the region ID rather than a bitmask to HAS_REGION() (Ville Syrjälä)
- Remove counter productive REGION_* wrappers (Ville Syrjälä)
- Fix typo [gem/i915_gem_ttm_move] (Deming Wang)
- Delete the live_hearbeat_fast selftest [gt] (Krzysztof Niemiec)
The following changes since commit 431c590c3ab0469dfedad3a832fe73556396ee52:
drm/tests: Add a unit test for range bias allocation (2024-05-16 12:50:14 +1000)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2024-06-12
for you to fetch changes up to 79655e867ad6dfde2734c67c7704c0dd5bf1e777:
drm/i915/mtl: Update workaround 14018575942 (2024-06-11 16:06:20 +0200)
----------------------------------------------------------------
UAPI Changes:
- Support replaying GPU hangs with captured context image (Tvrtko Ursulin)
Driver Changes:
Fixes/improvements/new stuff:
- Automate CCS Mode setting during engine resets [gt] (Andi Shyti)
- Revert "drm/i915: Remove extra multi-gt pm-references" (Janusz Krzysztofik)
- Fix HAS_REGION() usage in intel_gt_probe_lmem() (Ville Syrjälä)
- Disarm breadcrumbs if engines are already idle [gt] (Chris Wilson)
- Shadow default engine context image in the context (Tvrtko Ursulin)
- Support replaying GPU hangs with captured context image (Tvrtko Ursulin)
- avoid FIELD_PREP warning [guc] (Arnd Bergmann)
- Fix CCS id's calculation for CCS mode setting [gt] (Andi Shyti)
- Increase FLR timeout from 3s to 9s (Andi Shyti)
- Update workaround 14018575942 [mtl] (Angus Chen)
Future platform enablement:
- Enable w/a 16021333562 for DG2, MTL and ARL [guc] (John Harrison)
Miscellaneous:
- Pass the region ID rather than a bitmask to HAS_REGION() (Ville Syrjälä)
- Remove counter productive REGION_* wrappers (Ville Syrjälä)
- Fix typo [gem/i915_gem_ttm_move] (Deming Wang)
- Delete the live_hearbeat_fast selftest [gt] (Krzysztof Niemiec)
----------------------------------------------------------------
Andi Shyti (3):
drm/i915/gt: Automate CCS Mode setting during engine resets
drm/i915/gt: Fix CCS id's calculation for CCS mode setting
drm/i915: Increase FLR timeout from 3s to 9s
Angus Chen (1):
drm/i915/mtl: Update workaround 14018575942
Arnd Bergmann (1):
drm/i915/guc: avoid FIELD_PREP warning
Chris Wilson (1):
drm/i915/gt: Disarm breadcrumbs if engines are already idle
Deming Wang (1):
drm/i915/gem/i915_gem_ttm_move: Fix typo
Janusz Krzysztofik (1):
Revert "drm/i915: Remove extra multi-gt pm-references"
John Harrison (1):
drm/i915/guc: Enable w/a 16021333562 for DG2, MTL and ARL
Niemiec, Krzysztof (1):
drm/i915/gt: Delete the live_hearbeat_fast selftest
Tvrtko Ursulin (3):
Merge drm/drm-next into drm-intel-gt-next
drm/i915: Shadow default engine context image in the context
drm/i915: Support replaying GPU hangs with captured context image
Ville Syrjälä (3):
drm/i915: Fix HAS_REGION() usage in intel_gt_probe_lmem()
drm/i915: Pass the region ID rather than a bitmask to HAS_REGION()
drm/i915: Remove counter productive REGION_* wrappers
drivers/gpu/drm/i915/Kconfig.debug | 17 ++++
drivers/gpu/drm/i915/gem/i915_gem_context.c | 113 +++++++++++++++++++++
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 18 ++++
drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c | 2 +-
drivers/gpu/drm/i915/gt/intel_breadcrumbs.c | 15 ++-
drivers/gpu/drm/i915/gt/intel_context.c | 2 +
drivers/gpu/drm/i915/gt/intel_context.h | 22 ++++
drivers/gpu/drm/i915/gt/intel_context_types.h | 3 +
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 6 ++
drivers/gpu/drm/i915/gt/intel_gt_ccs_mode.c | 8 +-
drivers/gpu/drm/i915/gt/intel_gt_ccs_mode.h | 2 +-
drivers/gpu/drm/i915/gt/intel_gt_types.h | 8 ++
drivers/gpu/drm/i915/gt/intel_lrc.c | 8 +-
drivers/gpu/drm/i915/gt/intel_ring_submission.c | 8 +-
drivers/gpu/drm/i915/gt/intel_workarounds.c | 12 ++-
.../gpu/drm/i915/gt/selftest_engine_heartbeat.c | 110 --------------------
drivers/gpu/drm/i915/gt/uc/abi/guc_klvs_abi.h | 7 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 32 +++---
drivers/gpu/drm/i915/i915_drv.h | 4 +-
drivers/gpu/drm/i915/i915_params.c | 5 +
drivers/gpu/drm/i915/i915_params.h | 3 +-
drivers/gpu/drm/i915/i915_pci.c | 6 +-
drivers/gpu/drm/i915/intel_memory_region.c | 2 +-
drivers/gpu/drm/i915/intel_memory_region.h | 5 -
drivers/gpu/drm/i915/intel_uncore.c | 9 +-
drivers/gpu/drm/i915/selftests/mock_gem_device.c | 2 +-
include/uapi/drm/i915_drm.h | 27 +++++
27 files changed, 296 insertions(+), 160 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2024-07-04 7:31 Tvrtko Ursulin
2024-07-05 10:15 ` Daniel Vetter
0 siblings, 1 reply; 24+ messages in thread
From: Tvrtko Ursulin @ 2024-07-04 7:31 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave, Sima,
The final pull for 6.11 is quite small and only contains a handful of
fixes in areas such as stolen memory probing on ATS-M, GuC priority
handling, out of memory reporting noise downgrade and fence register
hanlding race condition reported by CI.
Regards,
Tvrtko
drm-intel-gt-next-2024-07-04:
Driver Changes:
Fixes/improvements/new stuff:
- Downgrade stolen lmem setup warning [gem] (Jonathan Cavitt)
- Evaluate GuC priority within locks [gt/uc] (Andi Shyti)
- Fix potential UAF by revoke of fence registers [gt] (Janusz Krzysztofik)
- Return NULL instead of '0' [gem] (Andi Shyti)
- Use the correct format specifier for resource_size_t [gem] (Andi Shyti)
- Suppress oom warning in favour of ENOMEM to userspace [gem] (Nirmoy Das)
Miscellaneous:
- Evaluate forcewake usage within locks [gt] (Andi Shyti)
- Fix typo in comment [gt/uc] (Andi Shyti)
The following changes since commit 79655e867ad6dfde2734c67c7704c0dd5bf1e777:
drm/i915/mtl: Update workaround 14018575942 (2024-06-11 16:06:20 +0200)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2024-07-04
for you to fetch changes up to 3b85152cb167bd24fe84ceb91b719b5904ca354f:
drm/i915/gem: Suppress oom warning in favour of ENOMEM to userspace (2024-06-28 00:11:01 +0200)
----------------------------------------------------------------
Driver Changes:
Fixes/improvements/new stuff:
- Downgrade stolen lmem setup warning [gem] (Jonathan Cavitt)
- Evaluate GuC priority within locks [gt/uc] (Andi Shyti)
- Fix potential UAF by revoke of fence registers [gt] (Janusz Krzysztofik)
- Return NULL instead of '0' [gem] (Andi Shyti)
- Use the correct format specifier for resource_size_t [gem] (Andi Shyti)
- Suppress oom warning in favour of ENOMEM to userspace [gem] (Nirmoy Das)
Miscellaneous:
- Evaluate forcewake usage within locks [gt] (Andi Shyti)
- Fix typo in comment [gt/uc] (Andi Shyti)
----------------------------------------------------------------
Andi Shyti (5):
drm/i915/gt: debugfs: Evaluate forcewake usage within locks
drm/i915/gt/uc: Fix typo in comment
drm/i915/gt/uc: Evaluate GuC priority within locks
drm/i915/gem: Return NULL instead of '0'
drm/i915/gem: Use the correct format specifier for resource_size_t
Janusz Krzysztofik (1):
drm/i915/gt: Fix potential UAF by revoke of fence registers
Jonathan Cavitt (1):
drm/i915/gem: Downgrade stolen lmem setup warning
Nirmoy Das (1):
drm/i915/gem: Suppress oom warning in favour of ENOMEM to userspace
drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 8 +++++--
drivers/gpu/drm/i915/gt/intel_ggtt_fencing.c | 1 +
drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c | 4 ++++
drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 27 ++++++++++++++---------
drivers/gpu/drm/i915/i915_scatterlist.c | 8 +++----
6 files changed, 32 insertions(+), 18 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PULL] drm-intel-gt-next
2024-07-04 7:31 Tvrtko Ursulin
@ 2024-07-05 10:15 ` Daniel Vetter
0 siblings, 0 replies; 24+ messages in thread
From: Daniel Vetter @ 2024-07-05 10:15 UTC (permalink / raw)
To: Tvrtko Ursulin
Cc: Dave Airlie, Daniel Vetter, Jani Nikula, Joonas Lahtinen,
Tvrtko Ursulin, Rodrigo Vivi, Thomas Zimmermann,
Maarten Lankhorst, Maxime Ripard, Thomas Hellström,
Oded Gabbay, Lucas De Marchi, dri-devel, intel-gfx, intel-xe,
dim-tools
On Thu, Jul 04, 2024 at 07:31:54AM +0000, Tvrtko Ursulin wrote:
>
> Hi Dave, Sima,
>
> The final pull for 6.11 is quite small and only contains a handful of
> fixes in areas such as stolen memory probing on ATS-M, GuC priority
> handling, out of memory reporting noise downgrade and fence register
> hanlding race condition reported by CI.
>
> Regards,
>
> Tvrtko
>
> drm-intel-gt-next-2024-07-04:
> Driver Changes:
>
> Fixes/improvements/new stuff:
>
> - Downgrade stolen lmem setup warning [gem] (Jonathan Cavitt)
> - Evaluate GuC priority within locks [gt/uc] (Andi Shyti)
> - Fix potential UAF by revoke of fence registers [gt] (Janusz Krzysztofik)
> - Return NULL instead of '0' [gem] (Andi Shyti)
> - Use the correct format specifier for resource_size_t [gem] (Andi Shyti)
> - Suppress oom warning in favour of ENOMEM to userspace [gem] (Nirmoy Das)
>
> Miscellaneous:
>
> - Evaluate forcewake usage within locks [gt] (Andi Shyti)
> - Fix typo in comment [gt/uc] (Andi Shyti)
> The following changes since commit 79655e867ad6dfde2734c67c7704c0dd5bf1e777:
>
> drm/i915/mtl: Update workaround 14018575942 (2024-06-11 16:06:20 +0200)
>
> are available in the Git repository at:
>
> https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2024-07-04
>
> for you to fetch changes up to 3b85152cb167bd24fe84ceb91b719b5904ca354f:
>
> drm/i915/gem: Suppress oom warning in favour of ENOMEM to userspace (2024-06-28 00:11:01 +0200)
Pulled, thanks.
-Sima
>
> ----------------------------------------------------------------
> Driver Changes:
>
> Fixes/improvements/new stuff:
>
> - Downgrade stolen lmem setup warning [gem] (Jonathan Cavitt)
> - Evaluate GuC priority within locks [gt/uc] (Andi Shyti)
> - Fix potential UAF by revoke of fence registers [gt] (Janusz Krzysztofik)
> - Return NULL instead of '0' [gem] (Andi Shyti)
> - Use the correct format specifier for resource_size_t [gem] (Andi Shyti)
> - Suppress oom warning in favour of ENOMEM to userspace [gem] (Nirmoy Das)
>
> Miscellaneous:
>
> - Evaluate forcewake usage within locks [gt] (Andi Shyti)
> - Fix typo in comment [gt/uc] (Andi Shyti)
>
> ----------------------------------------------------------------
> Andi Shyti (5):
> drm/i915/gt: debugfs: Evaluate forcewake usage within locks
> drm/i915/gt/uc: Fix typo in comment
> drm/i915/gt/uc: Evaluate GuC priority within locks
> drm/i915/gem: Return NULL instead of '0'
> drm/i915/gem: Use the correct format specifier for resource_size_t
>
> Janusz Krzysztofik (1):
> drm/i915/gt: Fix potential UAF by revoke of fence registers
>
> Jonathan Cavitt (1):
> drm/i915/gem: Downgrade stolen lmem setup warning
>
> Nirmoy Das (1):
> drm/i915/gem: Suppress oom warning in favour of ENOMEM to userspace
>
> drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 8 +++++--
> drivers/gpu/drm/i915/gt/intel_ggtt_fencing.c | 1 +
> drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c | 4 ++++
> drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h | 2 +-
> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 27 ++++++++++++++---------
> drivers/gpu/drm/i915/i915_scatterlist.c | 8 +++----
> 6 files changed, 32 insertions(+), 18 deletions(-)
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2024-08-23 9:55 Joonas Lahtinen
2024-08-27 10:59 ` Daniel Vetter
0 siblings, 1 reply; 24+ messages in thread
From: Joonas Lahtinen @ 2024-08-23 9:55 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave & Sima,
Mostly fixes in this drm-intel-gt-next PR at this time.
One thing to pay attention is the limitation of number of relocations to
INT_MAX which might impact synthetic tests, but no real workloads.
Regards, Joonas
***
drm-intel-gt-next-2024-08-23:
UAPI Changes:
- drm/i915: 2 GiB of relocations ought to be enough for anybody*
Limit the number of relocations to INT_MAX.
Only impact should be onsynthetic tests.
Driver Changes:
- Fix for #11396: GPU Hang and rcs0 reset on Cherrytrail platform
- Fix Virtual Memory mapping boundaries calculation (Andi)
- Fix for #11255: Long hangs in buddy allocator with DG2/A380 without
Resizable BAR since 6.9 (David)
- Mark the GT as dead when mmio is unreliable (Chris, Andi)
- Workaround additions / fixes for MTL, ARL and DG2 (John H, Nitin)
- Enable partial memory mapping of GPU virtual memory (Andi, Chris)
- Prevent NULL deref on intel_memory_regions_hw_probe (Jonathan, Dan)
- Avoid UAF on intel_engines_release (Krzysztof)
- Don't update PWR_CLK_STATE starting Gen12 (Umesh)
- Code and dmesg cleanups (Andi, Jesus, Luca)
The following changes since commit 3b85152cb167bd24fe84ceb91b719b5904ca354f:
drm/i915/gem: Suppress oom warning in favour of ENOMEM to userspace (2024-06-28 00:11:01 +0200)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2024-08-23
for you to fetch changes up to 255fc1703e42321b5afdedc8259ad03c7cc533ec:
drm/i915/gem: Calculate object page offset for partial memory mapping (2024-08-21 15:28:33 +0200)
----------------------------------------------------------------
UAPI Changes:
- Limit the number of relocations to INT_MAX (Tvrtko)
Only impact should be synthetic tests.
Driver Changes:
- Fix for #11396: GPU Hang and rcs0 reset on Cherrytrail platform
- Fix Virtual Memory mapping boundaries calculation (Andi)
- Fix for #11255: Long hangs in buddy allocator with DG2/A380 without
Resizable BAR since 6.9 (David)
- Mark the GT as dead when mmio is unreliable (Chris, Andi)
- Workaround additions / fixes for MTL, ARL and DG2 (John H, Nitin)
- Enable partial memory mapping of GPU virtual memory (Andi, Chris)
- Prevent NULL deref on intel_memory_regions_hw_probe (Jonathan, Dan)
- Avoid UAF on intel_engines_release (Krzysztof)
- Don't update PWR_CLK_STATE starting Gen12 (Umesh)
- Code and dmesg cleanups (Andi, Jesus, Luca)
----------------------------------------------------------------
Andi Shyti (6):
drm/i915/gem: Adjust vma offset for framebuffer mmap offset
drm/i915/gem: Fix Virtual Memory mapping boundaries calculation
drm/i915/gem: Improve pfn calculation readability in vm_fault_gtt()
drm/i915: Replace double blank with single blank after comma in gem/ and gt/
drm/i915/gem: Do not look for the exact address in node
drm/i915/gem: Calculate object page offset for partial memory mapping
Chris Wilson (1):
drm/i915/gt: Mark the GT as dead when mmio is unreliable
David Gow (2):
drm/i915: Allow evicting to use the requested placement
drm/i915: Attempt to get pages without eviction first
Jesus Narvaez (1):
drm/i915/guc: Change GEM_WARN_ON to guc_err to prevent taints in CI
John Harrison (2):
drm/i915/arl: Enable Wa_14019159160 for ARL
drm/i915/guc: Extend w/a 14019159160
Jonathan Cavitt (1):
drm/i915: Allow NULL memory region
Krzysztof Niemiec (1):
drm/i915/gt: Empty uabi engines list during intel_engines_release()
Luca Coelho (1):
drm/i915/gt: remove stray declaration of intel_gt_release_all()
Nitin Gote (2):
drm/i915/gt: Do not consider preemption during execlists_dequeue for gen8
drm/i915/gt: Add Wa_14019789679
Tvrtko Ursulin (1):
drm/i915: 2 GiB of relocations ought to be enough for anybody*
Umesh Nerlige Ramappa (1):
i915/perf: Remove code to update PWR_CLK_STATE for gen12
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 4 +-
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 73 +++++++++++++++++++---
drivers/gpu/drm/i915/gem/i915_gem_object_types.h | 2 +-
drivers/gpu/drm/i915/gem/i915_gem_ttm.c | 13 ++--
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +
.../gpu/drm/i915/gt/intel_execlists_submission.c | 6 +-
drivers/gpu/drm/i915/gt/intel_gpu_commands.h | 1 +
drivers/gpu/drm/i915/gt/intel_gt.h | 7 ++-
drivers/gpu/drm/i915/gt/intel_gt_types.h | 2 +
drivers/gpu/drm/i915/gt/intel_reset.c | 12 +++-
drivers/gpu/drm/i915/gt/intel_workarounds.c | 16 ++++-
drivers/gpu/drm/i915/gt/selftest_migrate.c | 2 +-
drivers/gpu/drm/i915/gt/uc/abi/guc_klvs_abi.h | 1 +
drivers/gpu/drm/i915/gt/uc/intel_guc.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 18 +++---
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 5 +-
drivers/gpu/drm/i915/gt/uc/intel_uc.c | 2 +-
drivers/gpu/drm/i915/i915_mm.c | 12 +++-
drivers/gpu/drm/i915/i915_mm.h | 3 +-
drivers/gpu/drm/i915/i915_perf.c | 33 ----------
drivers/gpu/drm/i915/intel_memory_region.c | 6 +-
drivers/gpu/drm/i915/intel_uncore.c | 7 ++-
.../gpu/drm/i915/selftests/intel_memory_region.c | 2 +-
23 files changed, 150 insertions(+), 81 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PULL] drm-intel-gt-next
2024-08-23 9:55 Joonas Lahtinen
@ 2024-08-27 10:59 ` Daniel Vetter
0 siblings, 0 replies; 24+ messages in thread
From: Daniel Vetter @ 2024-08-27 10:59 UTC (permalink / raw)
To: Joonas Lahtinen
Cc: Dave Airlie, Daniel Vetter, Jani Nikula, Tvrtko Ursulin,
Rodrigo Vivi, Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
On Fri, Aug 23, 2024 at 12:55:09PM +0300, Joonas Lahtinen wrote:
> Hi Dave & Sima,
>
> Mostly fixes in this drm-intel-gt-next PR at this time.
>
> One thing to pay attention is the limitation of number of relocations to
> INT_MAX which might impact synthetic tests, but no real workloads.
>
> Regards, Joonas
>
> ***
>
> drm-intel-gt-next-2024-08-23:
>
> UAPI Changes:
>
> - drm/i915: 2 GiB of relocations ought to be enough for anybody*
>
> Limit the number of relocations to INT_MAX.
>
> Only impact should be onsynthetic tests.
>
> Driver Changes:
>
> - Fix for #11396: GPU Hang and rcs0 reset on Cherrytrail platform
> - Fix Virtual Memory mapping boundaries calculation (Andi)
> - Fix for #11255: Long hangs in buddy allocator with DG2/A380 without
> Resizable BAR since 6.9 (David)
> - Mark the GT as dead when mmio is unreliable (Chris, Andi)
> - Workaround additions / fixes for MTL, ARL and DG2 (John H, Nitin)
> - Enable partial memory mapping of GPU virtual memory (Andi, Chris)
>
> - Prevent NULL deref on intel_memory_regions_hw_probe (Jonathan, Dan)
> - Avoid UAF on intel_engines_release (Krzysztof)
>
> - Don't update PWR_CLK_STATE starting Gen12 (Umesh)
> - Code and dmesg cleanups (Andi, Jesus, Luca)
>
> The following changes since commit 3b85152cb167bd24fe84ceb91b719b5904ca354f:
>
> drm/i915/gem: Suppress oom warning in favour of ENOMEM to userspace (2024-06-28 00:11:01 +0200)
>
> are available in the Git repository at:
>
> https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2024-08-23
Pulled, thanks.
-Sima
>
> for you to fetch changes up to 255fc1703e42321b5afdedc8259ad03c7cc533ec:
>
> drm/i915/gem: Calculate object page offset for partial memory mapping (2024-08-21 15:28:33 +0200)
>
> ----------------------------------------------------------------
> UAPI Changes:
>
> - Limit the number of relocations to INT_MAX (Tvrtko)
>
> Only impact should be synthetic tests.
>
> Driver Changes:
>
> - Fix for #11396: GPU Hang and rcs0 reset on Cherrytrail platform
> - Fix Virtual Memory mapping boundaries calculation (Andi)
> - Fix for #11255: Long hangs in buddy allocator with DG2/A380 without
> Resizable BAR since 6.9 (David)
> - Mark the GT as dead when mmio is unreliable (Chris, Andi)
> - Workaround additions / fixes for MTL, ARL and DG2 (John H, Nitin)
> - Enable partial memory mapping of GPU virtual memory (Andi, Chris)
>
> - Prevent NULL deref on intel_memory_regions_hw_probe (Jonathan, Dan)
> - Avoid UAF on intel_engines_release (Krzysztof)
>
> - Don't update PWR_CLK_STATE starting Gen12 (Umesh)
> - Code and dmesg cleanups (Andi, Jesus, Luca)
>
> ----------------------------------------------------------------
> Andi Shyti (6):
> drm/i915/gem: Adjust vma offset for framebuffer mmap offset
> drm/i915/gem: Fix Virtual Memory mapping boundaries calculation
> drm/i915/gem: Improve pfn calculation readability in vm_fault_gtt()
> drm/i915: Replace double blank with single blank after comma in gem/ and gt/
> drm/i915/gem: Do not look for the exact address in node
> drm/i915/gem: Calculate object page offset for partial memory mapping
>
> Chris Wilson (1):
> drm/i915/gt: Mark the GT as dead when mmio is unreliable
>
> David Gow (2):
> drm/i915: Allow evicting to use the requested placement
> drm/i915: Attempt to get pages without eviction first
>
> Jesus Narvaez (1):
> drm/i915/guc: Change GEM_WARN_ON to guc_err to prevent taints in CI
>
> John Harrison (2):
> drm/i915/arl: Enable Wa_14019159160 for ARL
> drm/i915/guc: Extend w/a 14019159160
>
> Jonathan Cavitt (1):
> drm/i915: Allow NULL memory region
>
> Krzysztof Niemiec (1):
> drm/i915/gt: Empty uabi engines list during intel_engines_release()
>
> Luca Coelho (1):
> drm/i915/gt: remove stray declaration of intel_gt_release_all()
>
> Nitin Gote (2):
> drm/i915/gt: Do not consider preemption during execlists_dequeue for gen8
> drm/i915/gt: Add Wa_14019789679
>
> Tvrtko Ursulin (1):
> drm/i915: 2 GiB of relocations ought to be enough for anybody*
>
> Umesh Nerlige Ramappa (1):
> i915/perf: Remove code to update PWR_CLK_STATE for gen12
>
> drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 4 +-
> drivers/gpu/drm/i915/gem/i915_gem_mman.c | 73 +++++++++++++++++++---
> drivers/gpu/drm/i915/gem/i915_gem_object_types.h | 2 +-
> drivers/gpu/drm/i915/gem/i915_gem_ttm.c | 13 ++--
> drivers/gpu/drm/i915/gt/intel_engine_cs.c | 2 +
> .../gpu/drm/i915/gt/intel_execlists_submission.c | 6 +-
> drivers/gpu/drm/i915/gt/intel_gpu_commands.h | 1 +
> drivers/gpu/drm/i915/gt/intel_gt.h | 7 ++-
> drivers/gpu/drm/i915/gt/intel_gt_types.h | 2 +
> drivers/gpu/drm/i915/gt/intel_reset.c | 12 +++-
> drivers/gpu/drm/i915/gt/intel_workarounds.c | 16 ++++-
> drivers/gpu/drm/i915/gt/selftest_migrate.c | 2 +-
> drivers/gpu/drm/i915/gt/uc/abi/guc_klvs_abi.h | 1 +
> drivers/gpu/drm/i915/gt/uc/intel_guc.c | 2 +-
> drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 18 +++---
> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 5 +-
> drivers/gpu/drm/i915/gt/uc/intel_uc.c | 2 +-
> drivers/gpu/drm/i915/i915_mm.c | 12 +++-
> drivers/gpu/drm/i915/i915_mm.h | 3 +-
> drivers/gpu/drm/i915/i915_perf.c | 33 ----------
> drivers/gpu/drm/i915/intel_memory_region.c | 6 +-
> drivers/gpu/drm/i915/intel_uncore.c | 7 ++-
> .../gpu/drm/i915/selftests/intel_memory_region.c | 2 +-
> 23 files changed, 150 insertions(+), 81 deletions(-)
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2024-09-06 10:55 Joonas Lahtinen
0 siblings, 0 replies; 24+ messages in thread
From: Joonas Lahtinen @ 2024-09-06 10:55 UTC (permalink / raw)
To: Dave Airlie, Simona Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave & Sima,
Here goes the final drm-intel-gt-next towards 6.12.
Primarily addition of fan speed hwmon, W/A fixups for ARL, add missing register
for userspace into DG2/MTL/ARL. Then a few smaller fixes.
Regards, Joonas
***
drm-intel-gt-next-2024-09-06:
Driver Changes:
- Expose fan speed via hwmon (Raag)
- Correction to Wa_14019159160 on ARL (John H)
- Whitelist COMMON_SLICE_CHICKEN1 for UMD access on DG2/MTL/ARL (Dnyaneshwar)
- Do not attempt to load the GSC multiple times to avoid hanging GSC HW (Daniele)
- Populate /sys/class/drm/cardX/engines/ even if one engine fails (Andi)
- Use kmemdup_array instead of kmemdup for multiple allocation (Yu)
- Remove extra unlikely() (Hongbo)
The following changes since commit 255fc1703e42321b5afdedc8259ad03c7cc533ec:
drm/i915/gem: Calculate object page offset for partial memory mapping (2024-08-21 15:28:33 +0200)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2024-09-06
for you to fetch changes up to 596a7f1084e49cc65072c458c348861e9b9ceab9:
drm/i915: Remove extra unlikely helper (2024-09-05 15:44:37 -0400)
----------------------------------------------------------------
Driver Changes:
- Expose fan speed via hwmon (Raag)
- Correction to Wa_14019159160 on ARL (John H)
- Whitelist COMMON_SLICE_CHICKEN1 for UMD access on DG2/MTL/ARL (Dnyaneshwar)
- Do not attempt to load the GSC multiple times to avoid hanging GSC HW (Daniele)
- Populate /sys/class/drm/cardX/engines/ even if one engine fails (Andi)
- Use kmemdup_array instead of kmemdup for multiple allocation (Yu)
- Remove extra unlikely() (Hongbo)
----------------------------------------------------------------
Andi Shyti (1):
drm/i915/gt: Continue creating engine sysfs files even after a failure
Daniele Ceraolo Spurio (1):
drm/i915: Do not attempt to load the GSC multiple times
Dnyaneshwar Bhadane (1):
drm/i915/gt: Whitelist COMMON_SLICE_CHICKEN1 for UMD access.
Hongbo Li (1):
drm/i915: Remove extra unlikely helper
John Harrison (1):
drm/i915/guc: Fix missing enable of Wa_14019159160 on ARL
Raag Jadav (1):
drm/i915/hwmon: expose fan speed
Yu Jiaoliang (1):
drm/i915/gt: Use kmemdup_array instead of kmemdup for multiple allocation
.../ABI/testing/sysfs-driver-intel-i915-hwmon | 8 ++
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 2 +-
drivers/gpu/drm/i915/gt/intel_gt_regs.h | 2 +
drivers/gpu/drm/i915/gt/intel_workarounds.c | 9 +--
drivers/gpu/drm/i915/gt/sysfs_engines.c | 5 +-
drivers/gpu/drm/i915/gt/uc/intel_gsc_uc.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_uc_fw.h | 5 ++
drivers/gpu/drm/i915/i915_hwmon.c | 88 ++++++++++++++++++++++
9 files changed, 112 insertions(+), 11 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2024-10-23 9:16 Tvrtko Ursulin
0 siblings, 0 replies; 24+ messages in thread
From: Tvrtko Ursulin @ 2024-10-23 9:16 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave, Sima,
This is the main pull request for 6.13 merge window.
PXP GuC auto-teardown feature got enabled, GPU reset robustness improvement
for Haswell and basic PMU functionality was enabled for Gen2 platforms.
The rest is a handful of small cleanups.
Regards,
Tvrtko
drm-intel-gt-next-2024-10-23:
Driver Changes:
Fixes/improvements/new stuff:
- Enable PXP GuC autoteardown flow [guc] (Juston Li)
- Retry RING_HEAD reset until it sticks [gt] (Nitin Gote)
- Add basic PMU support for gen2 [pmu] (Ville Syrjälä)
Miscellaneous:
- Prevent a possible int overflow in wq offsets [guc] (Nikita Zhandarovich)
- PMU code cleanups (Lucas De Marchi)
- Fixed "CPU" -> "GPU" typo [gt] (Zhang He)
- Gen2/3 interrupt handling cleanup (Ville Syrjälä)
The following changes since commit 596a7f1084e49cc65072c458c348861e9b9ceab9:
drm/i915: Remove extra unlikely helper (2024-09-05 15:44:37 -0400)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2024-10-23
for you to fetch changes up to 6ef0e3ef2662db71d363af77ce31fa940bb7d525:
drm/i915/gt: Retry RING_HEAD reset until it get sticks (2024-10-22 11:35:07 +0200)
----------------------------------------------------------------
Driver Changes:
Fixes/improvements/new stuff:
- Enable PXP GuC autoteardown flow [guc] (Juston Li)
- Retry RING_HEAD reset until it get sticks [gt] (Nitin Gote)
- Add basic PMU support for gen2 [pmu] (Ville Syrjälä)
Miscellaneous:
- Prevent a possible int overflow in wq offsets [guc] (Nikita Zhandarovich)
- PMU code cleanups (Lucas De Marchi)
- Fixed "CPU" -> "GPU" typo [gt] (Zhang He)
- Gen2/3 interrupt handling cleanup (Ville Syrjälä)
----------------------------------------------------------------
Juston Li (1):
drm/i915/guc: Enable PXP GuC autoteardown flow
Lucas De Marchi (2):
drm/i915/pmu: Drop is_igp()
drm/i915/pmu: Use event_to_pmu()
Nikita Zhandarovich (1):
drm/i915/guc: prevent a possible int overflow in wq offsets
Nitin Gote (1):
drm/i915/gt: Retry RING_HEAD reset until it get sticks
Ville Syrjälä (3):
drm/i915/gt: Nuke gen2_irq_{enable,disable}()
drm/i915/gt: s/gen3/gen2/
drm/i915/pmu: Add support for gen2
Zhang He (1):
drm/i915/gt: Fixed "CPU" -> "GPU" typo
drivers/gpu/drm/i915/gt/gen2_engine_cs.c | 23 ++--------
drivers/gpu/drm/i915/gt/gen2_engine_cs.h | 6 +--
drivers/gpu/drm/i915/gt/intel_engine_regs.h | 2 +-
drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c | 2 +-
drivers/gpu/drm/i915/gt/intel_ring_submission.c | 38 ++++++++++++----
drivers/gpu/drm/i915/gt/uc/intel_guc.c | 8 ++++
drivers/gpu/drm/i915/gt/uc/intel_guc_fwif.h | 1 +
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 4 +-
drivers/gpu/drm/i915/i915_drv.h | 3 ++
drivers/gpu/drm/i915/i915_pmu.c | 54 +++++++++++++----------
drivers/gpu/drm/i915/pxp/intel_pxp.c | 2 +-
11 files changed, 82 insertions(+), 61 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2024-12-18 9:48 Joonas Lahtinen
0 siblings, 0 replies; 24+ messages in thread
From: Joonas Lahtinen @ 2024-12-18 9:48 UTC (permalink / raw)
To: Dave Airlie, Simona Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave & Sima,
Here goes the pre-holidays drm-intel-gt-next PR towards 6.14 as promised.
As the main improvement there is engine busyness accuracy improvements
for GuC submission platforms and fixup for BO mapping corner cases.
The rest is smaller refactoring and improvements.
Happy Holidays!
Regards, Joonas
***
drm-intel-gt-next-2024-12-18:
Driver Changes:
- More accurate engine busyness metrics with GuC submission (Umesh)
- Ensure BO segment offset never exceeds allowed max (Krzysztof)
- Flush GuC CT receive tasklet during reset preparation (Zhanjun)
- Code cleanups and refactoring (David, Lucas)
- Debugging improvements (Jesus)
- Selftest improvements (Sk)
The following changes since commit 6ef0e3ef2662db71d363af77ce31fa940bb7d525:
drm/i915/gt: Retry RING_HEAD reset until it get sticks (2024-10-22 11:35:07 +0200)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2024-12-18
for you to fetch changes up to f373ebec18a75d671908e81ed9925aebf279ec2f:
drm/i915/guc: Update guc_err message to show outstanding g2h responses (2024-12-17 11:38:50 -0800)
----------------------------------------------------------------
Driver Changes:
- More accurate engine busyness metrics with GuC submission (Umesh)
- Ensure partial BO segment offset never exceeds allowed max (Krzysztof)
- Flush GuC CT receive tasklet during reset preparation (Zhanjun)
- Code cleanups and refactoring (David, Lucas)
- Debugging improvements (Jesus)
- Selftest improvements (Sk)
----------------------------------------------------------------
Dr. David Alan Gilbert (1):
drm/i915/gt: Remove unused execlists_unwind_incomplete_requests
Jesus Narvaez (1):
drm/i915/guc: Update guc_err message to show outstanding g2h responses
Krzysztof Karas (1):
drm/i915: ensure segment offset never exceeds allowed max
Lucas De Marchi (4):
drm/i915/pmu: Rename cpuhp_slot to cpuhp_state
drm/i915/pmu: Stop setting event_init to NULL
drm/i915/pmu: Replace closed with registered
drm/i915/pmu: Remove pointless synchronize_rcu() call
Sk Anirban (1):
drm/i915/selftests: Add delay to stabilize frequency in live_rps_power
Umesh Nerlige Ramappa (3):
i915/guc: Reset engine utilization buffer before registration
i915/guc: Ensure busyness counter increases motonically
i915/guc: Accumulate active runtime on gt reset
Zhanjun Dong (1):
drm/i915/guc: Flush ct receive tasklet during reset preparation
drivers/gpu/drm/i915/gt/intel_engine.h | 3 --
drivers/gpu/drm/i915/gt/intel_engine_types.h | 5 ++
.../gpu/drm/i915/gt/intel_execlists_submission.c | 9 ----
drivers/gpu/drm/i915/gt/selftest_rps.c | 1 +
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 53 ++++++++++++++++++++--
drivers/gpu/drm/i915/i915_mm.c | 4 +-
drivers/gpu/drm/i915/i915_pmu.c | 50 ++++++++------------
drivers/gpu/drm/i915/i915_pmu.h | 4 +-
8 files changed, 79 insertions(+), 50 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2025-01-10 9:03 Joonas Lahtinen
0 siblings, 0 replies; 24+ messages in thread
From: Joonas Lahtinen @ 2025-01-10 9:03 UTC (permalink / raw)
To: Dave Airlie, Simona Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave & Sima,
Here goes the post-Holidays drm-intel-gt-next PR towards kernel 6.14 as
promised.
There is an engine reset improvement for Haswell (Gen7.5) and older platforms
and the rest is smaller fixes and dead code removal.
Regards, Joonas
***
drm-intel-gt-next-2025-01-10:
Driver Changes:
- More robust engine resets on Haswell and older (Nitin)
- Dead code removal (David)
- Selftest, logging and tracing improvements (Sk, Nitin, Sebastian,
Apoorva)
The following changes since commit f373ebec18a75d671908e81ed9925aebf279ec2f:
drm/i915/guc: Update guc_err message to show outstanding g2h responses (2024-12-17 11:38:50 -0800)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2025-01-10
for you to fetch changes up to 6f0572fa8fd6758f7a0ed2e96b429cb5b4323e12:
drm/i915/gt: Prefer IS_ENABLED() instead of defined() on config option (2025-01-09 13:43:16 +0100)
----------------------------------------------------------------
Driver Changes:
- More robust engine resets on Haswell and older (Nitin)
- Dead code removal (David)
- Selftest, logging and tracing improvements (Sk, Nitin, Sebastian,
Apoorva)
----------------------------------------------------------------
Apoorva Singh (1):
drm/i915/gt: Prevent uninitialized pointer reads
Dr. David Alan Gilbert (3):
drm/i915: Remove deadcode
drm/i915: Remove unused intel_huc_suspend
drm/i915: Remove unused intel_ring_cacheline_align
Nitin Gote (3):
drm/i915/gt: Increase a time to retry RING_HEAD reset
drm/i915/gt: Use ENGINE_TRACE for tracing.
drm/i915/gt: Prefer IS_ENABLED() instead of defined() on config option
Sebastian Brzezinka (1):
drm/i915/gt: Log reason for setting TAINT_WARN at reset
Sk Anirban (1):
drm/i915/selftests: Implement frequency logging for energy reading validation
drivers/gpu/drm/i915/gem/i915_gem_lmem.c | 23 -------------------
drivers/gpu/drm/i915/gem/i915_gem_lmem.h | 2 --
drivers/gpu/drm/i915/gt/intel_reset.c | 6 ++++-
drivers/gpu/drm/i915/gt/intel_ring.c | 24 --------------------
drivers/gpu/drm/i915/gt/intel_ring.h | 1 -
drivers/gpu/drm/i915/gt/intel_ring_submission.c | 30 +++++++++++++++----------
drivers/gpu/drm/i915/gt/selftest_migrate.c | 2 +-
drivers/gpu/drm/i915/gt/selftest_rc6.c | 17 +++++++++++---
drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_huc.c | 13 -----------
drivers/gpu/drm/i915/gt/uc/intel_huc.h | 1 -
drivers/gpu/drm/i915/i915_active.c | 18 ---------------
drivers/gpu/drm/i915/i915_active.h | 1 -
13 files changed, 39 insertions(+), 101 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2025-02-26 8:13 Tvrtko Ursulin
0 siblings, 0 replies; 24+ messages in thread
From: Tvrtko Ursulin @ 2025-02-26 8:13 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave, Sima,
Here comes the main pull request for 6.15.
New sysfs UAPI for configuring the GuC power profiles is probably the most
interesting addition. Other than that, there is one performance workaround
for Gen12 platforms and a bunch of fixes, mostly around the GuC code. PMU
code was consolidated to use kernel facility for CPU hotplug handling which
required a merge during the development window.
Other than those just a collection of cleanups and selftest fixes.
Regards,
Tvrtko
drm-intel-gt-next-2025-02-26:
UAPI Changes:
- Add sysfs for SLPC power profiles [slpc] (Vinay Belgaumkar)
Driver Changes:
Fixes/improvements/new stuff:
- Fix zero delta busyness issue [pmu] (Umesh Nerlige Ramappa)
- Fix page cleanup on DMA remap failure (Brian Geffon)
- Debug print LRC state entries only if the context is pinned [guc] (Daniele Ceraolo Spurio)
- Drop custom hotplug code [pmu] (Lucas De Marchi)
- Use spin_lock_irqsave() in interruptible context [guc] (Krzysztof Karas)
- Add wait on depth stall done bit handling [gen12] (Juha-Pekka Heikkila)
Miscellaneous:
- Change throttle criteria for rps [selftest] (Raag Jadav)
- Add debug print about hw config table size (John Harrison)
- Include requested frequency in slow firmware load messages [uc] (John Harrison)
- Remove i915_pmu_event_event_idx() [pmu] (Lucas De Marchi)
- Remove unused live_context_for_engine (Dr. David Alan Gilbert)
- Add Wa_22010465259 in its respective WA list (Ranu Maurya)
- Correct frequency handling in RPS power measurement [selftests] (Sk Anirban)
- Add helper function slpc_measure_power [guc/slpc] (Sk Anirban)
- Revert "drm/i915/gt: Log reason for setting TAINT_WARN at reset" [gt] (Sebastian Brzezinka)
- Avoid using uninitialized context [selftests] (Krzysztof Karas)
- Use struct_size() helper in kmalloc() (luoqing)
- Use prandom in selftest [selftests] (Markus Theil)
- Replace kmap with its safer kmap_local_page counterpart [gt] (Andi Shyti)
Merges:
- Merge drm/drm-next into drm-intel-gt-next (Tvrtko Ursulin)
The following changes since commit 2014c95afecee3e76ca4a56956a936e23283f05b:
Linux 6.14-rc1 (2025-02-02 15:39:26 -0800)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2025-02-26
for you to fetch changes up to 7ded94bd11d47a8ddef051aef1d1a42d8191e09f:
drm/i915/gt: add wait on depth stall done bit handling (2025-02-18 12:37:04 +0100)
----------------------------------------------------------------
UAPI Changes:
- Add sysfs for SLPC power profiles [slpc] (Vinay Belgaumkar)
Driver Changes:
Fixes/improvements/new stuff:
- Fix zero delta busyness issue [pmu] (Umesh Nerlige Ramappa)
- Fix page cleanup on DMA remap failure (Brian Geffon)
- Debug print LRC state entries only if the context is pinned [guc] (Daniele Ceraolo Spurio)
- Drop custom hotplug code [pmu] (Lucas De Marchi)
- Use spin_lock_irqsave() in interruptible context [guc] (Krzysztof Karas)
- Add wait on depth stall done bit handling [gen12] (Juha-Pekka Heikkila)
Miscellaneous:
- Change throttle criteria for rps [selftest] (Raag Jadav)
- Add debug print about hw config table size (John Harrison)
- Include requested frequency in slow firmware load messages [uc] (John Harrison)
- Remove i915_pmu_event_event_idx() [pmu] (Lucas De Marchi)
- Remove unused live_context_for_engine (Dr. David Alan Gilbert)
- Add Wa_22010465259 in its respective WA list (Ranu Maurya)
- Correct frequency handling in RPS power measurement [selftests] (Sk Anirban)
- Add helper function slpc_measure_power [guc/slpc] (Sk Anirban)
- Revert "drm/i915/gt: Log reason for setting TAINT_WARN at reset" [gt] (Sebastian Brzezinka)
- Avoid using uninitialized context [selftests] (Krzysztof Karas)
- Use struct_size() helper in kmalloc() (luoqing)
- Use prandom in selftest [selftests] (Markus Theil)
- Replace kmap with its safer kmap_local_page counterpart [gt] (Andi Shyti)
Merges:
- Merge drm/drm-next into drm-intel-gt-next (Tvrtko Ursulin)
----------------------------------------------------------------
Andi Shyti (1):
drm/i915/gt: Replace kmap with its safer kmap_local_page counterpart
Brian Geffon (1):
drm/i915: Fix page cleanup on DMA remap failure
Daniele Ceraolo Spurio (1):
drm/i915/guc: Debug print LRC state entries only if the context is pinned
Dr. David Alan Gilbert (1):
drm/i915: Remove unused live_context_for_engine
John Harrison (2):
drm/i915: Add debug print about hw config table size
drm/i915/uc: Include requested frequency in slow firmware load messages
Juha-Pekka Heikkila (1):
drm/i915/gt: add wait on depth stall done bit handling
Krzysztof Karas (2):
drm/i915/selftests: avoid using uninitialized context
drm/i915/gt: Use spin_lock_irqsave() in interruptible context
Lucas De Marchi (2):
drm/i915/pmu: Remove i915_pmu_event_event_idx()
drm/i915/pmu: Drop custom hotplug code
Markus Theil (1):
drm/i915/selftests: use prandom in selftest
Raag Jadav (1):
drm/i915/selftest: Change throttle criteria for rps
Ranu Maurya (1):
drm/i915: Add Wa_22010465259 in its respective WA list
Sebastian Brzezinka (1):
Revert "drm/i915/gt: Log reason for setting TAINT_WARN at reset"
Sk Anirban (2):
drm/i915/selftests: Correct frequency handling in RPS power measurement
drm/i915/guc/slpc: Add helper function slpc_measure_power
Tvrtko Ursulin (1):
Merge drm/drm-next into drm-intel-gt-next
Umesh Nerlige Ramappa (1):
drm/i915/pmu: Fix zero delta busyness issue
Vinay Belgaumkar (1):
drm/i915/slpc: Add sysfs for SLPC power profiles
luoqing (1):
selftests: i915: Use struct_size() helper in kmalloc()
drivers/gpu/drm/i915/gem/i915_gem_shmem.c | 6 +-
drivers/gpu/drm/i915/gem/selftests/mock_context.c | 38 -------
drivers/gpu/drm/i915/gem/selftests/mock_context.h | 3 -
drivers/gpu/drm/i915/gem/selftests/mock_dmabuf.c | 3 +-
drivers/gpu/drm/i915/gt/intel_ggtt_fencing.c | 4 +-
drivers/gpu/drm/i915/gt/intel_gt_regs.h | 3 +
drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c | 47 ++++++++
drivers/gpu/drm/i915/gt/intel_reset.c | 6 +-
drivers/gpu/drm/i915/gt/intel_rps.c | 4 +
drivers/gpu/drm/i915/gt/intel_workarounds.c | 19 ++--
drivers/gpu/drm/i915/gt/selftest_rps.c | 13 +--
drivers/gpu/drm/i915/gt/selftest_slpc.c | 17 ++-
drivers/gpu/drm/i915/gt/shmem_utils.c | 8 +-
.../gpu/drm/i915/gt/uc/abi/guc_actions_slpc_abi.h | 5 +
drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c | 11 +-
drivers/gpu/drm/i915/gt/uc/intel_guc_hwconfig.c | 3 +
drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 65 +++++++++++
drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.h | 1 +
drivers/gpu/drm/i915/gt/uc/intel_guc_slpc_types.h | 3 +
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 40 +++++--
drivers/gpu/drm/i915/gt/uc/intel_huc.c | 12 ++-
drivers/gpu/drm/i915/i915_module.c | 2 -
drivers/gpu/drm/i915/i915_pmu.c | 120 +--------------------
drivers/gpu/drm/i915/i915_pmu.h | 11 --
drivers/gpu/drm/i915/selftests/i915_gem.c | 7 +-
drivers/gpu/drm/i915/selftests/i915_gem_gtt.c | 4 +-
26 files changed, 228 insertions(+), 227 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2025-03-12 23:24 Tvrtko Ursulin
0 siblings, 0 replies; 24+ messages in thread
From: Tvrtko Ursulin @ 2025-03-12 23:24 UTC (permalink / raw)
To: Dave Airlie, Daniel Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave, Sima,
Here comes the final pull request for 6.15.
Main thing is the bump of the reported mmap ioctl feature level, which
enables Mesa to reliably detect full partial mmap support. Then the DRM
client implementation gains vmap support and the remaining changes are just
some refactors and cleanups.
Regards,
Tvrtko
drm-intel-gt-next-2025-03-12:
UAPI Changes:
- Increase I915_PARAM_MMAP_GTT_VERSION version to indicate support for partial mmaps (José Roberto de Souza)
Driver Changes:
Fixes/improvements/new stuff:
- Implement vmap/vunmap GEM object functions (Asbjørn Sloth Tønnesen)
Miscellaneous:
- Various register definition cleanups (Ville Syrjälä)
- Fix typo in a comment [gt/uc] (Yuichiro Tsuji)
The following changes since commit 7ded94bd11d47a8ddef051aef1d1a42d8191e09f:
drm/i915/gt: add wait on depth stall done bit handling (2025-02-18 12:37:04 +0100)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2025-03-12
for you to fetch changes up to bfef148f3680e6b9d28e7fca46d9520f80c5e50e:
drm/i915: Increase I915_PARAM_MMAP_GTT_VERSION version to indicate support for partial mmaps (2025-03-11 07:04:51 -0700)
----------------------------------------------------------------
UAPI Changes:
- Increase I915_PARAM_MMAP_GTT_VERSION version to indicate support for partial mmaps (José Roberto de Souza)
Driver Changes:
Fixes/improvements/new stuff:
- Implement vmap/vunmap GEM object functions (Asbjørn Sloth Tønnesen)
Miscellaneous:
- Various register definition cleanups (Ville Syrjälä)
- Fix typo in a comment [gt/uc] (Yuichiro Tsuji)
----------------------------------------------------------------
Asbjørn Sloth Tønnesen (1):
drm/i915: implement vmap/vunmap GEM object functions
José Roberto de Souza (1):
drm/i915: Increase I915_PARAM_MMAP_GTT_VERSION version to indicate support for partial mmaps
Ville Syrjälä (12):
drm/i915: Bump RING_FAULT engine ID bits
drm/i915: Relocate RING_FAULT bits
drm/i915: Use REG_BIT() & co. for ring fault registers
drm/i915: Document which RING_FAULT bits apply to which platforms
drm/i915: Introduce RING_FAULT_VADDR_MASK
drm/i915: Extract gen8_report_fault()
drm/i915: Use REG_BIT() & co. for CHV EU/slice fuse bits
drm/i915: Reoder CHV EU/slice fuse bits
drm/i915: Use REG_BIT() & co. for BDW+ EU/slice fuse bits
drm/i915: Reoder BDW+ EU/slice fuse bits
drm/i915: Use REG_BIT() & co. for gen9+ timestamp freq registers
drm/i915: Reoder gen9+ timestamp freq register bits
Yuichiro Tsuji (1):
drm/i915/gt/uc: Fix typo in a comment
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 5 +-
drivers/gpu/drm/i915/gem/i915_gem_object.c | 26 +++++
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 5 +-
drivers/gpu/drm/i915/gt/intel_gt.c | 89 ++++++--------
drivers/gpu/drm/i915/gt/intel_gt_clock_utils.c | 10 +-
drivers/gpu/drm/i915/gt/intel_gt_mcr.c | 5 +-
drivers/gpu/drm/i915/gt/intel_gt_regs.h | 136 ++++++++++------------
drivers/gpu/drm/i915/gt/intel_sseu.c | 56 +++++----
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 9 +-
9 files changed, 166 insertions(+), 175 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2025-05-08 6:21 Joonas Lahtinen
0 siblings, 0 replies; 24+ messages in thread
From: Joonas Lahtinen @ 2025-05-08 6:21 UTC (permalink / raw)
To: Dave Airlie, Simona Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave & Sima,
Here goes the drm-intel-gt-next PR towards 6.16. No new features, just fixes.
SLPC wait boost fix for DG1/DG2, engine reset fix for HSW.
Move to usleep_range for <20ms timeouts and fix splats on early probe
errors.
The rest is usual cleanups and improvements to selftests.
Regards, Joonas
***
drm-intel-gt-next-2025-05-08-1:
Driver Changes:
- Fix SLPC wait boosting reference counting to avoid getting stuck on non-boost
frequency on power saving profile on DG1/DG2 (Vinay)
- Add 20ms delay to engine reset for robustness on HSW (Nitin)
- Use proper sleeping functions for timeouts shorter than 20ms (Andi)
- Fix fence not released on early probe errors for HuC (Janusz)
- Remove const from struct i915_wa list allocation (Kees)
- Apply SPDX license format where missing and use single-line format (Andi)
- Whitespace fixes (Dan, Andi)
- Selftest improvements (Mikolaj, Badal, Sk,
The following changes since commit bfef148f3680e6b9d28e7fca46d9520f80c5e50e:
drm/i915: Increase I915_PARAM_MMAP_GTT_VERSION version to indicate support for partial mmaps (2025-03-11 07:04:51 -0700)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2025-05-08-1
for you to fetch changes up to b2602a84ff85926be8d1fb7186128976c9f76c3f:
drm/i915/gt: Remove const from struct i915_wa list allocation (2025-05-07 12:34:59 +0200)
----------------------------------------------------------------
Driver Changes:
- Fix SLPC wait boosting reference counting to avoid getting stuck on non-boost
frequency on power saving profile on DG1/DG2 (Vinay)
- Add 20ms delay to engine reset for robustness on HSW (Nitin)
- Use proper sleeping functions for timeouts shorter than 20ms (Andi)
- Fix fence not released on early probe errors for HuC (Janusz)
- Remove const from struct i915_wa list allocation (Kees)
- Apply SPDX license format where missing and use single-line format (Andi)
- Whitespace fixes (Dan, Andi)
- Selftest improvements (Mikolaj, Badal, Sk,
----------------------------------------------------------------
Andi Shyti (4):
drm/i915/gt: Fix SPDX license format
drm/i915/gt: Remove trailing blank lines
drm/i915/gt: Use proper sleeping functions for timeouts shorter than 20ms
drm/i915/gem: Convert SPDX headers to single-line format
Badal Nilawar (1):
drm/i915: Disable RPG during live selftest
Dan Carpenter (1):
drm/i915/gsc: delete a stray tab in intel_gsc_fw_get_binary_info()
Janusz Krzysztofik (1):
drm/i915/huc: Fix fence not released on early probe errors
Kees Cook (1):
drm/i915/gt: Remove const from struct i915_wa list allocation
Mikolaj Wasiak (2):
i915/selftest/igt_mmap: let mmap tests run in kthread
i915/gt/selftests: Disable lrc_timestamp test
Nitin Gote (1):
drm/i915/gt: Add delay to let engine resumes properly
Sk Anirban (1):
drm/i915/selftests: Refactor RC6 power measurement and error handling
Vinay Belgaumkar (1):
drm/i915/slpc: Balance the inc/dec for num_waiters
drivers/gpu/drm/i915/gem/i915_gem_busy.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_clflush.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_clflush.h | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_context.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_context.h | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_context_types.h | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_domain.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_internal.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_ioctls.h | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_mman.h | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_object.c | 1 +
drivers/gpu/drm/i915/gem/i915_gem_object.h | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_object_types.h | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_pages.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_phys.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_pm.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_pm.h | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_shmem.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_shrinker.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_throttle.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_tiling.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gem_userptr.c | 5 +-
drivers/gpu/drm/i915/gem/i915_gem_wait.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gemfs.c | 3 +-
drivers/gpu/drm/i915/gem/i915_gemfs.h | 3 +-
drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c | 14 +++++-
drivers/gpu/drm/i915/gt/intel_gtt.c | 1 -
drivers/gpu/drm/i915/gt/intel_lrc.c | 1 -
drivers/gpu/drm/i915/gt/intel_mocs.c | 1 -
drivers/gpu/drm/i915/gt/intel_rc6.c | 19 ++------
drivers/gpu/drm/i915/gt/intel_ring_submission.c | 8 +++-
drivers/gpu/drm/i915/gt/intel_rps.c | 14 ++++--
drivers/gpu/drm/i915/gt/intel_wopcm.h | 3 +-
drivers/gpu/drm/i915/gt/intel_workarounds.c | 2 +-
drivers/gpu/drm/i915/gt/selftest_lrc.c | 9 ++++
drivers/gpu/drm/i915/gt/selftest_rc6.c | 54 +++++++++++++++-------
drivers/gpu/drm/i915/gt/selftest_tlb.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_gsc_fw.c | 2 +-
drivers/gpu/drm/i915/gt/uc/intel_huc.c | 11 ++---
drivers/gpu/drm/i915/gt/uc/intel_huc.h | 1 +
drivers/gpu/drm/i915/gt/uc/intel_uc.c | 1 +
drivers/gpu/drm/i915/selftests/i915_selftest.c | 18 ++++++++
46 files changed, 140 insertions(+), 108 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2025-07-02 7:44 Tvrtko Ursulin
0 siblings, 0 replies; 24+ messages in thread
From: Tvrtko Ursulin @ 2025-07-02 7:44 UTC (permalink / raw)
To: Dave Airlie, Simona Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave, Sima,
Here comes the first pull request for 6.17.
Headline feature is that Ville was able to move DG1 out of force probe.
Other than that mostly fixes in the GuC backend.
One cross-merge to fix the build.
Regards,
Tvrtko
drm-intel-gt-next-2025-07-02:
Driver Changes:
Fixes/improvements/new stuff:
- Avoid GuC scheduling stalls [guc] (Julia Filipchuk)
- Remove force_probe requirement for DG1 (Ville Syrjälä)
- Handle errors correctly to avoid losing GuC-2-Host messages [guc] (Jesus Narvaez)
- Avoid double wakeref put if GuC context deregister failed [guc] (Jesus Narvaez)
- Avoid timeline memory leak with signals and legacy platforms [ringbuf] (Janusz Krzysztofik)
- Fix MEI (discrete) interrupt handler on RT kernels [gsc] (Junxiao Chang)
Miscellaneous:
- Allow larger memory allocation [selftest] (Mikolaj Wasiak)
- Use provided dma_fence_is_chain (Tvrtko Ursulin)
- Fix build error with GCOV and AutoFDO enabled [pmu] (Tzung-Bi Shih)
- Fix build error some more (Arnd Bergmann)
- Reduce stack usage in igt_vma_pin1() (Arnd Bergmann)
- Move out engine related macros from i915_drv.h (Krzysztof Karas)
- Move GEM_QUIRK_PIN_SWIZZLED_PAGES to i915_gem.h (Krzysztof Karas)
The following changes since commit f8bb3ed3197966fb60bedcbdc126d2bd5bc0a77f:
drm/nouveau/tegra: Fix error pointer vs NULL return in nvkm_device_tegra_resource_addr() (2025-05-24 14:36:07 +1000)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2025-07-02
for you to fetch changes up to dccf655f69002d496a527ba441b4f008aa5bebbf:
drm/i915/gsc: mei interrupt top half should be in irq disabled context (2025-06-30 14:21:06 -0400)
----------------------------------------------------------------
Driver Changes:
Fixes/improvements/new stuff:
- Avoid GuC scheduling stalls [guc] (Julia Filipchuk)
- Remove force_probe requirement for DG1 (Ville Syrjälä)
- Handle errors correctly to avoid losing GuC-2-Host messages [guc] (Jesus Narvaez)
- Avoid double wakeref put if GuC context deregister failed [guc] (Jesus Narvaez)
- Avoid timeline memory leak with signals and legacy platforms [ringbuf] (Janusz Krzysztofik)
- Fix MEI (discrete) interrupt handler on RT kernels [gsc] (Junxiao Chang)
Miscellaneous:
- Allow larger memory allocation [selftest] (Mikolaj Wasiak)
- Use provided dma_fence_is_chain (Tvrtko Ursulin)
- Fix build error with GCOV and AutoFDO enabled [pmu] (Tzung-Bi Shih)
- Fix build error some more (Arnd Bergmann)
- Reduce stack usage in igt_vma_pin1() (Arnd Bergmann)
- Move out engine related macros from i915_drv.h (Krzysztof Karas)
- Move GEM_QUIRK_PIN_SWIZZLED_PAGES to i915_gem.h (Krzysztof Karas)
----------------------------------------------------------------
Arnd Bergmann (2):
drm/i915: fix build error some more
drm/i915: reduce stack usage in igt_vma_pin1()
Janusz Krzysztofik (1):
drm/i915/gt: Fix timeline left held on VMA alloc error
Jesus Narvaez (2):
drm/i915/guc: Check if expecting reply before decrementing outstanding_submission_g2h
drm/i915/guc: Handle race condition where wakeref count drops below 0
Joonas Lahtinen (2):
Revert "drm/i915/gem: Allow EXEC_CAPTURE on recoverable contexts on DG1"
Merge drm/drm-next into drm-intel-gt-next
Julia Filipchuk (1):
drm/i915/guc: Enable DUAL_QUEUE_WA for newer platforms
Junxiao Chang (1):
drm/i915/gsc: mei interrupt top half should be in irq disabled context
Krzysztof Karas (2):
drm/i915: Move out engine related macros from i915_drv.h
drm/i915: move GEM_QUIRK_PIN_SWIZZLED_PAGES to i915_gem.h
Mikolaj Wasiak (1):
drm/i915/selftest: allow larger memory allocation
Tvrtko Ursulin (1):
drm/i915: Use provided dma_fence_is_chain
Tzung-Bi Shih (1):
drm/i915/pmu: Fix build error with GCOV and AutoFDO enabled
Ville Syrjälä (2):
drm/i915/gem: Allow EXEC_CAPTURE on recoverable contexts on DG1
drm/i915/pci: Remove force_probe requirement for DG1
drivers/gpu/drm/i915/gem/i915_gem_wait.c | 7 +----
drivers/gpu/drm/i915/gt/intel_engine.h | 31 ++++++++++++++++++++
drivers/gpu/drm/i915/gt/intel_gsc.c | 2 +-
drivers/gpu/drm/i915/gt/intel_ring_submission.c | 3 +-
drivers/gpu/drm/i915/gt/uc/intel_guc.c | 7 ++++-
drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 19 ++++++++++---
drivers/gpu/drm/i915/i915_drv.h | 33 ----------------------
drivers/gpu/drm/i915/i915_gem.h | 2 ++
drivers/gpu/drm/i915/i915_pci.c | 1 -
drivers/gpu/drm/i915/i915_pmu.c | 6 ++--
drivers/gpu/drm/i915/i915_vma.c | 20 +++++++++++++
drivers/gpu/drm/i915/i915_vma.h | 22 ++-------------
.../gpu/drm/i915/selftests/intel_memory_region.c | 14 ++-------
13 files changed, 86 insertions(+), 81 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PULL] drm-intel-gt-next
@ 2025-09-01 13:03 Joonas Lahtinen
0 siblings, 0 replies; 24+ messages in thread
From: Joonas Lahtinen @ 2025-09-01 13:03 UTC (permalink / raw)
To: Dave Airlie, Simona Vetter
Cc: Jani Nikula, Joonas Lahtinen, Tvrtko Ursulin, Rodrigo Vivi,
Thomas Zimmermann, Maarten Lankhorst, Maxime Ripard,
Thomas Hellström, Oded Gabbay, Lucas De Marchi, dri-devel,
intel-gfx, intel-xe, dim-tools
Hi Dave & Sima,
Here goes the first drm-intel-gt-next PR towards v6.18.
No new features, more of a -fixes PR this time.
Workaround fixes for older platforms, one overflow fix and an
cmpxchg optimization for x86 platforms.
Then the usual set of selftest improvements, smaller fixes and cleanups.
Regards, Joonas
***
drm-intel-gt-next-2025-09-01:
Driver Changes:
- Apply multiple JSL/EHL/Gen7/Gen6 workaround properly at context level (Sebastian)
- Protect against overflow in active_engine() (Krzysztof)
- Use try_cmpxchg64() in __active_lookup() (Uros)
- Enable GuC CT_DEAD output in regular debug builds (John)
- Static checker and style fixes (Sebastian)
- Selftest improvements (Krzysztof)
The following changes since commit dccf655f69002d496a527ba441b4f008aa5bebbf:
drm/i915/gsc: mei interrupt top half should be in irq disabled context (2025-06-30 14:21:06 -0400)
are available in the Git repository at:
https://gitlab.freedesktop.org/drm/i915/kernel.git tags/drm-intel-gt-next-2025-09-01
for you to fetch changes up to b7a855f488c6825e3dc06e78c49326491bef6f98:
drm/i915/gt: Relocate Gen6 context-specific workaround (2025-08-18 20:32:25 -0100)
----------------------------------------------------------------
Driver Changes:
- Apply multiple JSL/EHL/Gen7/Gen6 workaround properly at context level (Sebastian)
- Protect against overflow in active_engine() (Krzysztof)
- Use try_cmpxchg64() in __active_lookup() (Uros)
- Enable GuC CT_DEAD output in regular debug builds (John)
- Static checker and style fixes (Sebastian)
- Selftest improvements (Krzysztof)
----------------------------------------------------------------
John Harrison (1):
drm/i915/guc: Enable CT_DEAD output in regular debug builds
Krzysztof Karas (3):
drm/i915/selftests: Do not overwrite error code after intel_context_migrate_clear() call
drm/i915/selftests: Do not leak vm_area_struct on early return
drm/i915/gt: Protect against overflow in active_engine()
Sebastian Brzezinka (5):
drm/i915: Add braces around the else block in clflush_write32()
drm/i915: Replace empty conditional with continue in eb_relocate_vma()
drm/i915/gt: Relocate compression repacking WA for JSL/EHL
drm/i915/gt: Relocate Gen7 context-specific workarounds
drm/i915/gt: Relocate Gen6 context-specific workaround
Uros Bizjak (1):
drm/i915/active: Use try_cmpxchg64() in __active_lookup()
drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 57 +++++++++---------
drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c | 70 ++++++++++------------
drivers/gpu/drm/i915/gt/intel_workarounds.c | 49 +++++++--------
drivers/gpu/drm/i915/gt/selftest_hangcheck.c | 2 +
drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 11 ++--
drivers/gpu/drm/i915/gt/uc/intel_guc_ct.h | 2 +-
drivers/gpu/drm/i915/i915_active.c | 5 +-
7 files changed, 98 insertions(+), 98 deletions(-)
^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2025-09-01 13:03 UTC | newest]
Thread overview: 24+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-02 7:44 [PULL] drm-intel-gt-next Tvrtko Ursulin
-- strict thread matches above, loose matches on Subject: below --
2025-09-01 13:03 Joonas Lahtinen
2025-05-08 6:21 Joonas Lahtinen
2025-03-12 23:24 Tvrtko Ursulin
2025-02-26 8:13 Tvrtko Ursulin
2025-01-10 9:03 Joonas Lahtinen
2024-12-18 9:48 Joonas Lahtinen
2024-10-23 9:16 Tvrtko Ursulin
2024-09-06 10:55 Joonas Lahtinen
2024-08-23 9:55 Joonas Lahtinen
2024-08-27 10:59 ` Daniel Vetter
2024-07-04 7:31 Tvrtko Ursulin
2024-07-05 10:15 ` Daniel Vetter
2024-06-12 12:57 Tvrtko Ursulin
2024-04-26 7:17 Joonas Lahtinen
2024-02-28 14:02 Tvrtko Ursulin
2024-02-15 10:06 Tvrtko Ursulin
2024-02-16 2:58 ` Dave Airlie
2024-02-16 9:31 ` Thomas Hellström
2024-02-16 9:33 ` Thomas Hellström
2024-02-16 9:41 ` Joonas Lahtinen
2024-02-20 15:14 ` Joonas Lahtinen
2023-12-15 12:11 Joonas Lahtinen
2023-12-08 16:18 Joonas Lahtinen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).