All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/12] Explicit MCR handling and MTL steering
@ 2022-09-19 22:32 ` Matt Roper
  0 siblings, 0 replies; 40+ messages in thread
From: Matt Roper @ 2022-09-19 22:32 UTC (permalink / raw)
  To: intel-gfx; +Cc: dri-devel

Steering of multicast/replicated registers becomes a bit more
complicated on Meteor Lake.  Whereas previously the control register we
used to manage the steering was only used by our driver[*], software's
control of steering has now been consolidated with the controls for
various other hardware/firmware agents into a single register.  We can
no longer utilize pre-programmed implicit steering since other firmware
agents may change the steering target and not restore it afterward;
we'll need to explicitly steer all types of MCR registers (including the
GSLICE/COMPUTE/DSS ranges that have been handled implicitly in the
past).  Furthermore, since multiple agents will now be sharing a single
steering control register, races are possible.  To address this, the
hardware adds a new MCR semaphore register which is supposed to be used
to temporarily lock the steering while performing MCR operations.

It's going to become important for us to handle accesses of multicast
registers very explicitly going forward.  This series provides some prep
work for that by updating our register definitions to clearly define
registers as either MCR or non-MCR and ensure that we're using the
intel_gt_mcr_*() functions rather than intel_uncore_*() when operating
on MCR registers.  By declaring MCR registers as a new C type (i.e., not
an i915_reg_t) we can have the compiler help us find any mistakes where
non-MCR functions are used on MCR registers and vice-versa.

This series also includes the general MCR steering tables and logic, but
does not yet introduce the support for the semaphore register that will
be used to coordinate steering updates with other agents; we'll provide
that support in a separate series once this preparation work has landed.


[*] This is a bit of an oversimplification; there are some hardware and
software debug tools that use the same MCR_SELECTOR register that i915
does and which could potentially re-steer MCR accesses behind our back.
E.g., simply using IGT's "intel_reg" tool to write the MCR_SELECTOR
register at the wrong time could interfere with driver operation.  But
given that these debug facilities require root privileges to run and are
only used by people intentionally debugging the driver or hardware, we
can ignore such races for real-world usage.

Matt Roper (12):
  drm/i915/gen8: Create separate reg definitions for new MCR registers
  drm/i915/xehp: Create separate reg definitions for new MCR registers
  drm/i915/gt: Drop a few unused register definitions
  drm/i915/gt: Correct prefix on a few registers
  drm/i915/xehp: Check for faults on primary GAM
  drm/i915: Define MCR registers explicitly
  drm/i915/gt: Always use MCR functions on multicast registers
  drm/i915/guc: Handle save/restore of MCR registers explicitly
  drm/i915/gt: Add MCR-specific workaround initializers
  drm/i915: Define multicast registers as a new type
  drm/i915/mtl: Add multicast steering for render GT
  drm/i915/mtl: Add multicast steering for media GT

 drivers/gpu/drm/i915/gt/intel_engine_cs.c     |   4 +-
 drivers/gpu/drm/i915/gt/intel_ggtt.c          |   4 +-
 drivers/gpu/drm/i915/gt/intel_gt.c            |  40 +-
 drivers/gpu/drm/i915/gt/intel_gt_mcr.c        | 149 ++++--
 drivers/gpu/drm/i915/gt/intel_gt_mcr.h        |  14 +-
 drivers/gpu/drm/i915/gt/intel_gt_regs.h       | 157 +++---
 drivers/gpu/drm/i915/gt/intel_gt_types.h      |   9 +-
 drivers/gpu/drm/i915/gt/intel_gtt.c           |  44 +-
 drivers/gpu/drm/i915/gt/intel_gtt.h           |   2 +-
 drivers/gpu/drm/i915/gt/intel_mocs.c          |  12 +-
 drivers/gpu/drm/i915/gt/intel_workarounds.c   | 470 +++++++++++-------
 .../gpu/drm/i915/gt/intel_workarounds_types.h |   9 +-
 .../gpu/drm/i915/gt/selftest_workarounds.c    |   2 +-
 drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c    |  60 ++-
 .../gpu/drm/i915/gt/uc/intel_guc_capture.c    |   8 +-
 drivers/gpu/drm/i915/gt/uc/intel_guc_fw.c     |  12 +-
 drivers/gpu/drm/i915/gvt/handlers.c           |  19 +-
 drivers/gpu/drm/i915/gvt/mmio_context.c       |  16 +-
 drivers/gpu/drm/i915/i915_pci.c               |   1 +
 drivers/gpu/drm/i915/i915_reg_defs.h          |  22 +-
 drivers/gpu/drm/i915/intel_gvt_mmio_table.c   |  12 +-
 drivers/gpu/drm/i915/intel_pm.c               |  20 +-
 22 files changed, 700 insertions(+), 386 deletions(-)

-- 
2.37.3


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

end of thread, other threads:[~2022-09-30 21:55 UTC | newest]

Thread overview: 40+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-19 22:32 [PATCH 00/12] Explicit MCR handling and MTL steering Matt Roper
2022-09-19 22:32 ` [Intel-gfx] " Matt Roper
2022-09-19 22:32 ` [PATCH 01/12] drm/i915/gen8: Create separate reg definitions for new MCR registers Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-26  6:15   ` Balasubramani Vivekanandan
2022-09-26  6:15     ` [Intel-gfx] " Balasubramani Vivekanandan
2022-09-28  8:25   ` Balasubramani Vivekanandan
2022-09-28  8:25     ` [Intel-gfx] " Balasubramani Vivekanandan
2022-09-19 22:32 ` [PATCH 02/12] drm/i915/xehp: " Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-28  9:50   ` Balasubramani Vivekanandan
2022-09-28  9:50     ` [Intel-gfx] " Balasubramani Vivekanandan
2022-09-30 21:54     ` Matt Roper
2022-09-30 21:54       ` Matt Roper
2022-09-19 22:32 ` [PATCH 03/12] drm/i915/gt: Drop a few unused register definitions Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-19 22:32 ` [PATCH 04/12] drm/i915/gt: Correct prefix on a few registers Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-19 22:32 ` [PATCH 05/12] drm/i915/xehp: Check for faults on primary GAM Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-26 12:28   ` Balasubramani Vivekanandan
2022-09-19 22:32 ` [PATCH 06/12] drm/i915: Define MCR registers explicitly Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-19 22:32 ` [PATCH 07/12] drm/i915/gt: Always use MCR functions on multicast registers Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-19 22:32 ` [PATCH 08/12] drm/i915/guc: Handle save/restore of MCR registers explicitly Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-19 22:32 ` [PATCH 09/12] drm/i915/gt: Add MCR-specific workaround initializers Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-19 22:32 ` [PATCH 10/12] drm/i915: Define multicast registers as a new type Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-19 22:32 ` [PATCH 11/12] drm/i915/mtl: Add multicast steering for render GT Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-19 22:32 ` [PATCH 12/12] drm/i915/mtl: Add multicast steering for media GT Matt Roper
2022-09-19 22:32   ` [Intel-gfx] " Matt Roper
2022-09-20  0:46 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Explicit MCR handling and MTL steering Patchwork
2022-09-20  0:46 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2022-09-20  1:10 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2022-09-20 10:18 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2022-09-20 22:37   ` Matt Roper

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.