mbox series

[0/4] Invalidate TLB cache for all GT's

Message ID 20230801141955.383305-1-andi.shyti@linux.intel.com (mailing list archive)
Headers show
Series Invalidate TLB cache for all GT's | expand

Message

Andi Shyti Aug. 1, 2023, 2:19 p.m. UTC
Hi,

This series revisits a once-trendy topic: TLB invalidation
support for multi-gt.  It's a theme that's been passed around and
reshaped by several of us.

I've filtered out most of the original changes from this series,
initially sent by Mauro [1]. His ideas were inspired by some
changes from Chris, who in turn was inspired by a change from
me[2], all of which stemmed from offline talks by Chris (it's
like a game of inspiration tag!).

What I've done is simply rebase and refactor the patches,
focusing solely on the changes needed for multi-gt TLB
invalidation in MMIO memory. The result is a leaner, more
targeted proposal.

Other patches from the original series might follow.

Thanks,
Andi

[1] https://patchwork.freedesktop.org/series/106293/
[2] https://patchwork.freedesktop.org/series/103831/

Andi Shyti (2):
  i915/drm/gt: Move the gt defines in the gt directory
  drm/i915: Remove unnecessary include

Chris Wilson (2):
  drm/i915/gt: Move TLB invalidation to its own file
  drm/i915: Invalidate the TLBs on each GT

 drivers/gpu/drm/i915/Makefile                 |   1 +
 .../gpu/drm/i915/gem/i915_gem_object_types.h  |   4 +-
 drivers/gpu/drm/i915/gem/i915_gem_pages.c     |  15 +-
 drivers/gpu/drm/i915/gt/intel_gt.c            | 140 +--------------
 drivers/gpu/drm/i915/gt/intel_gt.h            |  12 --
 drivers/gpu/drm/i915/gt/intel_gt_defines.h    |  11 ++
 drivers/gpu/drm/i915/gt/intel_ppgtt.c         |   4 +-
 drivers/gpu/drm/i915/gt/intel_tlb.c           | 159 ++++++++++++++++++
 drivers/gpu/drm/i915/gt/intel_tlb.h           |  29 ++++
 drivers/gpu/drm/i915/gt/selftest_tlb.c        |   3 +-
 drivers/gpu/drm/i915/i915_drv.h               |   1 -
 drivers/gpu/drm/i915/i915_vma.c               |  15 +-
 12 files changed, 231 insertions(+), 163 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/gt/intel_gt_defines.h
 create mode 100644 drivers/gpu/drm/i915/gt/intel_tlb.c
 create mode 100644 drivers/gpu/drm/i915/gt/intel_tlb.h

Comments

Mauro Carvalho Chehab Aug. 1, 2023, 2:40 p.m. UTC | #1
On 8/1/23 16:19, Andi Shyti wrote:
> Hi,
>
> This series revisits a once-trendy topic: TLB invalidation
> support for multi-gt.  It's a theme that's been passed around and
> reshaped by several of us.
>
> I've filtered out most of the original changes from this series,
> initially sent by Mauro [1]. His ideas were inspired by some
> changes from Chris, who in turn was inspired by a change from
> me[2], all of which stemmed from offline talks by Chris (it's
> like a game of inspiration tag!).
>
> What I've done is simply rebase and refactor the patches,
> focusing solely on the changes needed for multi-gt TLB
> invalidation in MMIO memory. The result is a leaner, more
> targeted proposal.

Series look good to me. Feel free to add my R-B to the patches I didn't 
sign already.

Reviewed-by: Mauro Carvalho Chehab <mchehab@kernel.org>


>
> Other patches from the original series might follow.
>
> Thanks,
> Andi
>
> [1] https://patchwork.freedesktop.org/series/106293/
> [2] https://patchwork.freedesktop.org/series/103831/
>
> Andi Shyti (2):
>    i915/drm/gt: Move the gt defines in the gt directory
>    drm/i915: Remove unnecessary include
>
> Chris Wilson (2):
>    drm/i915/gt: Move TLB invalidation to its own file
>    drm/i915: Invalidate the TLBs on each GT
>
>   drivers/gpu/drm/i915/Makefile                 |   1 +
>   .../gpu/drm/i915/gem/i915_gem_object_types.h  |   4 +-
>   drivers/gpu/drm/i915/gem/i915_gem_pages.c     |  15 +-
>   drivers/gpu/drm/i915/gt/intel_gt.c            | 140 +--------------
>   drivers/gpu/drm/i915/gt/intel_gt.h            |  12 --
>   drivers/gpu/drm/i915/gt/intel_gt_defines.h    |  11 ++
>   drivers/gpu/drm/i915/gt/intel_ppgtt.c         |   4 +-
>   drivers/gpu/drm/i915/gt/intel_tlb.c           | 159 ++++++++++++++++++
>   drivers/gpu/drm/i915/gt/intel_tlb.h           |  29 ++++
>   drivers/gpu/drm/i915/gt/selftest_tlb.c        |   3 +-
>   drivers/gpu/drm/i915/i915_drv.h               |   1 -
>   drivers/gpu/drm/i915/i915_vma.c               |  15 +-
>   12 files changed, 231 insertions(+), 163 deletions(-)
>   create mode 100644 drivers/gpu/drm/i915/gt/intel_gt_defines.h
>   create mode 100644 drivers/gpu/drm/i915/gt/intel_tlb.c
>   create mode 100644 drivers/gpu/drm/i915/gt/intel_tlb.h
>