Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Intel-gfx] [PATCH v3 0/5] Expose RPS thresholds in sysfs
@ 2023-05-25  8:11 Tvrtko Ursulin
  2023-05-25  8:11 ` [Intel-gfx] [PATCH 1/5] drm/i915: Move setting of rps thresholds to init Tvrtko Ursulin
                   ` (8 more replies)
  0 siblings, 9 replies; 10+ messages in thread
From: Tvrtko Ursulin @ 2023-05-25  8:11 UTC (permalink / raw)
  To: Intel-gfx, dri-devel

From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

From patch 4:

    User feedback indicates significant performance gains are possible in
    specific games with non default RPS up/down thresholds.

    Expose these tunables via sysfs which will allow users to achieve best
    performance when running games and best power efficiency elsewhere.

    Note this patch supports non GuC based platforms only.

    References: https://gitlab.freedesktop.org/drm/intel/-/issues/8389

Issue 8389 suggests 10-15% performance gains are possible with tweaked
thresholds.

One question is are we able to find a "one size fits all" values.

However regardless of that, given we already expose frequency controls in sysfs
with the same reasoning of allowing system owners explicit control if so wanted,
I think exposing the thresholds can be equally justified.

v2:
 * Hopefully fixed the debug build issue.
 * Re-program the hw registers on change too!

v3:
 * Added a patch which captures the thresholds in error state, to account for
   any potential instability which may be caused by moving away from defaults.
 * Fixed one checkpatch warning.

Test-with: 20230523105139.626772-1-tvrtko.ursulin@linux.intel.com

Tvrtko Ursulin (5):
  drm/i915: Move setting of rps thresholds to init
  drm/i915: Record default rps threshold values
  drm/i915: Add helpers for managing rps thresholds
  drm/i915: Expose RPS thresholds in sysfs
  drm/i915: Include RPS threshold in error state

 drivers/gpu/drm/i915/gt/intel_gt_sysfs_pm.c | 108 ++++++++++++++++++++
 drivers/gpu/drm/i915/gt/intel_gt_types.h    |   3 +
 drivers/gpu/drm/i915/gt/intel_rps.c         |  83 +++++++++++++--
 drivers/gpu/drm/i915/gt/intel_rps.h         |   4 +
 drivers/gpu/drm/i915/i915_gpu_error.c       |   5 +
 drivers/gpu/drm/i915/i915_gpu_error.h       |   5 +
 6 files changed, 197 insertions(+), 11 deletions(-)

-- 
2.39.2


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

end of thread, other threads:[~2023-05-26  1:39 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-25  8:11 [Intel-gfx] [PATCH v3 0/5] Expose RPS thresholds in sysfs Tvrtko Ursulin
2023-05-25  8:11 ` [Intel-gfx] [PATCH 1/5] drm/i915: Move setting of rps thresholds to init Tvrtko Ursulin
2023-05-25  8:11 ` [Intel-gfx] [PATCH 2/5] drm/i915: Record default rps threshold values Tvrtko Ursulin
2023-05-25  8:11 ` [Intel-gfx] [PATCH 3/5] drm/i915: Add helpers for managing rps thresholds Tvrtko Ursulin
2023-05-25  8:11 ` [Intel-gfx] [PATCH 4/5] drm/i915: Expose RPS thresholds in sysfs Tvrtko Ursulin
2023-05-25  8:11 ` [Intel-gfx] [PATCH 5/5] drm/i915: Include RPS threshold in error state Tvrtko Ursulin
2023-05-25  8:56 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Expose RPS thresholds in sysfs (rev3) Patchwork
2023-05-25  8:57 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2023-05-25 18:58 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2023-05-26  1:39 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox