[PATCH 00/43] Stop using xe_gt as a register MMIO target
Matt Roper
matthew.d.roper at intel.com
Wed Sep 4 00:21:01 UTC 2024
A "GT" is just one subunit of an Intel GPU (i.e., the part that contains
the hardware engines, GuC, and programmable execution units). It is not
directly related to register MMIO, although a subset of the registers
our driver interacts with are within the GT. The driver also interacts
with, and programs, several other parts of the GPU that live outside the
GT (e.g., sgunit, pcode, display, etc.), so using an 'xe_gt' structure
in the driver as the target for all MMIO register operations causes a
lot of confusion and doesn't match how the hardware is actually
organized.
For register MMIO operations, the things that truly matter are:
* Where the MMIO region is mapped for CPU access.
* Any extra offset that should be "automatically" added to some
xe_reg offsets. (e.g., the 0x380000 offset for GSI registers in the
media GT)
* Extra metadata for size, valid/invalid subregions of the map, etc.
that can be utilized by debug builds to perform extra checks and
assertions to catch coding mistakes.
Let's add a dedicated 'xe_mmio' structure that encapsulates this
specific information and can be used as a target for MMIO operations.
For now an xe_mmio structure is present inside every xe_gt and xe_tile,
and can be used as the target for GT and non-GT operations respectively.
In the future additional xe_mmio substructures can be added for other
specific cases.
Note that there's a (currently unused) "mmio_ext" infrastructure in the
driver that appears to be an attempt to work around the GT-centric way
the driver has been doing register MMIO. That infrastructure is simply
replaced with an additional instance of "struct xe_mmio" that lives at
the tile level. This will allow standard register access logic for
accessing non-GT registers that exist in a very different BAR region
and/or reside in a different iomap. Once code actually shows up to use
"mmio_ext" it will probably get renamed and accessed via "xe_foo->mmio"
or "xe_mmio_for_foo(xe)."
Once this general refactor lands, a follow-up will be add some extra
checking in debug builds to catch cases where the driver might be
performing MMIO accesses incorrectly (for example, accessing GT
registers through a non-GT MMIO which wouldn't apply proper GSI
offsets).
Since converting the entire driver from xe_gt to xe_mmio for register
access operations is a lot of churn, the original conversion includes
some _Generic compatibility defines to temporarily allow either xe_gt or
xe_mmio to be used. This allows individual parts of the driver to be
converted in separate patches for ease of review. The compatibility
macros are removed again at the end of the series.
Matt Roper (43):
drm/xe: Move forcewake to 'gt.pm' substructure
drm/xe: Create dedicated xe_mmio structure
drm/xe: Clarify size of MMIO region
drm/xe: Move GSI offset adjustment fields into 'struct xe_mmio'
drm/xe: Populate GT's mmio iomap from tile during init
drm/xe: Switch mmio_ext to use 'struct xe_mmio'
drm/xe: Add xe_device backpointer to xe_mmio
drm/xe: Adjust mmio code to pass VF substructure to SRIOV code
drm/xe: Switch MMIO interface to take xe_mmio instead of xe_gt
drm/xe/irq: Convert register access to use xe_mmio
drm/xe/pcode: Convert register access to use xe_mmio
drm/xe/hwmon: Convert register access to use xe_mmio
drm/xe/vram: Convert register access to use xe_mmio
drm/xe/compat-i915: Convert register access to use xe_mmio
drm/xe/lmtt: Convert register access to use xe_mmio
drm/xe/stolen: Convert register access to use xe_mmio
drm/xe/device: Convert register access to use xe_mmio
drm/xe/pci: Convert register access to use xe_mmio
drm/xe/wa: Convert register access to use xe_mmio
drm/xe/uc: Convert register access to use xe_mmio
drm/xe/guc: Convert register access to use xe_mmio
drm/xe/huc: Convert register access to use xe_mmio
drm/xe/gsc: Convert register access to use xe_mmio
drm/xe/query: Convert register access to use xe_mmio
drm/xe/mcr: Convert register access to use xe_mmio
drm/xe/mocs: Convert register access to use xe_mmio
drm/xe/hw_engine: Convert register access to use xe_mmio
drm/xe/gt_throttle: Convert register access to use xe_mmio
drm/xe/pat: Convert register access to use xe_mmio
drm/xe/wopcm: Convert register access to use xe_mmio
drm/xe/oa: Convert register access to use xe_mmio
drm/xe/topology: Convert register access to use xe_mmio
drm/xe/execlist: Convert register access to use xe_mmio
drm/xe/gt_clock: Convert register access to use xe_mmio
drm/xe/reg_sr: Convert register access to use xe_mmio
drm/xe/gt: Convert register access to use xe_mmio
drm/xe/sriov: Convert register access to use xe_mmio
drm/xe/tlb: Convert register access to use xe_mmio
drm/xe/gt_idle: Convert register access to use xe_mmio
drm/xe/forcewake: Convert register access to use xe_mmio
drm/xe/ggtt: Convert register access to use xe_mmio
drm/xe/ccs_mode: Convert register access to use xe_mmio
drm/xe/mmio: Drop compatibility macros
.../drm/xe/compat-i915-headers/intel_uncore.h | 36 ++---
drivers/gpu/drm/xe/tests/xe_mocs.c | 4 +-
drivers/gpu/drm/xe/xe_assert.h | 2 +-
drivers/gpu/drm/xe/xe_device.c | 37 +++--
drivers/gpu/drm/xe/xe_device.h | 3 +-
drivers/gpu/drm/xe/xe_device_types.h | 51 +++++--
drivers/gpu/drm/xe/xe_execlist.c | 19 +--
drivers/gpu/drm/xe/xe_force_wake.c | 4 +-
drivers/gpu/drm/xe/xe_ggtt.c | 8 +-
drivers/gpu/drm/xe/xe_gsc.c | 23 +--
drivers/gpu/drm/xe/xe_gsc_proxy.c | 4 +-
drivers/gpu/drm/xe/xe_gt.c | 10 +-
drivers/gpu/drm/xe/xe_gt_ccs_mode.c | 2 +-
drivers/gpu/drm/xe/xe_gt_clock.c | 6 +-
drivers/gpu/drm/xe/xe_gt_freq.c | 2 +-
drivers/gpu/drm/xe/xe_gt_idle.c | 17 +--
drivers/gpu/drm/xe/xe_gt_mcr.c | 39 ++---
drivers/gpu/drm/xe/xe_gt_printk.h | 2 +-
drivers/gpu/drm/xe/xe_gt_sriov_pf.c | 2 +-
drivers/gpu/drm/xe/xe_gt_sriov_pf_service.c | 6 +-
drivers/gpu/drm/xe/xe_gt_sriov_vf.c | 10 +-
drivers/gpu/drm/xe/xe_gt_sriov_vf.h | 5 +-
drivers/gpu/drm/xe/xe_gt_throttle.c | 4 +-
drivers/gpu/drm/xe/xe_gt_tlb_invalidation.c | 8 +-
drivers/gpu/drm/xe/xe_gt_topology.c | 8 +-
drivers/gpu/drm/xe/xe_gt_types.h | 20 +--
drivers/gpu/drm/xe/xe_guc.c | 60 ++++----
drivers/gpu/drm/xe/xe_guc_ads.c | 2 +-
drivers/gpu/drm/xe/xe_guc_pc.c | 34 ++---
drivers/gpu/drm/xe/xe_huc.c | 6 +-
drivers/gpu/drm/xe/xe_hw_engine.c | 29 ++--
drivers/gpu/drm/xe/xe_hwmon.c | 16 +--
drivers/gpu/drm/xe/xe_irq.c | 63 ++++----
drivers/gpu/drm/xe/xe_lmtt.c | 2 +-
drivers/gpu/drm/xe/xe_mmio.c | 134 +++++++++---------
drivers/gpu/drm/xe/xe_mmio.h | 35 +++--
drivers/gpu/drm/xe/xe_mocs.c | 16 +--
drivers/gpu/drm/xe/xe_oa.c | 48 ++++---
drivers/gpu/drm/xe/xe_pat.c | 14 +-
drivers/gpu/drm/xe/xe_pci.c | 24 +++-
drivers/gpu/drm/xe/xe_pcode.c | 4 +-
drivers/gpu/drm/xe/xe_query.c | 7 +-
drivers/gpu/drm/xe/xe_reg_sr.c | 17 +--
drivers/gpu/drm/xe/xe_sriov.c | 2 +-
drivers/gpu/drm/xe/xe_trace.h | 6 +-
drivers/gpu/drm/xe/xe_ttm_stolen_mgr.c | 8 +-
drivers/gpu/drm/xe/xe_uc_fw.c | 17 +--
drivers/gpu/drm/xe/xe_vram.c | 7 +-
drivers/gpu/drm/xe/xe_wa.c | 4 +-
drivers/gpu/drm/xe/xe_wopcm.c | 12 +-
50 files changed, 486 insertions(+), 413 deletions(-)
--
2.45.2
More information about the Intel-xe
mailing list