From: Andi Shyti <andi.shyti@linux.intel.com>
To: Intel GFX <intel-gfx@lists.freedesktop.org>,
DRI Devel <dri-devel@lists.freedesktop.org>
Cc: "Sujaritha Sundaresan" <sujaritha.sundaresan@intel.com>,
"Andi Shyti" <andi@etezian.org>,
"Lucas De Marchi" <lucas.demarchi@intel.com>,
"Chris Wilson" <chris@chris-wilson.co.uk>,
"Andi Shyti" <andi.shyti@linux.intel.com>,
"Michał Winiarski" <michal.winiarski@intel.com>
Subject: [PATCH v9 0/6] More preparation for multi gt patches
Date: Sun, 19 Dec 2021 23:24:54 +0200 [thread overview]
Message-ID: <20211219212500.61432-1-andi.shyti@linux.intel.com> (raw)
Hi,
thanks Matt for merging the first 10 patches from v8. This series
contains only the last 6 patches from the previous that make use
of the newly inserted to_gt() to reach the ggtt that at the end
moves from the i915 structure to the gt.
I have reshuffled a bit the order of the patche (refer to the
changelog) so that from this series only patch 2 remains
unreviewed.
Thanks Matt and Sujaritha for the reviews.
Andi
Changelog:
==========
Patchwork: https://patchwork.freedesktop.org/series/97020/
v8 -> v9:
- The patch is down to the last 6 patches.
- Reshuffled a bit the order, it's more intuitive to apply the
changes starting from i915/gt/, i915/, i915/gem, i915/display,
i915/selftests
- Addressed Matt's comments in v8.
- Added Matt and Sujaritha's r-b tags
v7 -> v8:
- Removed patch 11 from v7 that was allocating statically the
ggtt in the gt structure instead of a dynamic allocation. As
Matt pointed out, we can have GT's sharing the same GGTT.
- The whole i915->ggtt to gt->ggtt patch is split in 5 patches
instead of one single to make it easier to review.
- The last patch removes i915->ggtt and allocates the gt->ggtt
with drmm_kzalloc in the early probe and mock device.
v6 -> v7:
- Patch 1: add a note about the double presence of
__intel_gt_init_early() and intel_gt_init_early().
- Added all Matt's r-b's for patches 2-10.
- Added a patch 12 that moves the i915->ggtt into gt->ggtt.
v5 -> v6:
- fixed the assignement of i915->gt->ggtt = ggtt in the mock gem
device that was making use of it before.
v4 -> v5:
- use to_gt() instead of to_root_gt() and use Michał work done
previously.
- split the /i915->gt/to_gt()/ patch in smaller chunks in order
to make review easier. (Thanks Lucas)
v3 -> v4:
- the intel_gt_init_early() has been split as it was causing
some headaches for the order of the early initialization. The
split has been done keeping in mind the coming next patch in
the series that wil make this a static function.
v2 -> v3:
- sed -i ... took too much freedom and changed more than it was
supposed to.
- fix a compile error which did not appear in my local build
v1 -> v2:
- patch 2: do not use anymore the reference i915->gt but use
to_root_gt(), coming from Matt Roper's patch.
- fix some comments from Chris.
Andi Shyti (4):
drm/i915/selftests: Use to_gt() helper
drm/i915/pxp: Use to_gt() helper
drm/i915: Rename i915->gt to i915->gt0
drm/i915: Move the GGTT from i915 private data to the GT
Andi Shyti (1):
drm/i915: Remove unused i915->ggtt
Michał Winiarski (5):
drm/i915/gt: Use to_gt() helper for GGTT accesses
drm/i915: Use to_gt() helper for GGTT accesses
drm/i915/gem: Use to_gt() helper for GGTT accesses
drm/i915/display: Use to_gt() helper for GGTT accesses
drm/i915/selftests: Use to_gt() helper for GGTT accesses
drivers/gpu/drm/i915/display/intel_fbc.c | 2 +-
drivers/gpu/drm/i915/display/intel_fbdev.c | 2 +-
.../drm/i915/display/intel_plane_initial.c | 2 +-
drivers/gpu/drm/i915/gem/i915_gem_context.h | 2 +-
.../gpu/drm/i915/gem/i915_gem_execbuffer.c | 2 +-
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 19 +++++++-------
drivers/gpu/drm/i915/gem/i915_gem_pm.c | 2 +-
drivers/gpu/drm/i915/gem/i915_gem_shrinker.c | 6 ++---
drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 8 +++---
drivers/gpu/drm/i915/gem/i915_gem_tiling.c | 15 ++++++-----
.../i915/gem/selftests/i915_gem_client_blt.c | 2 +-
.../drm/i915/gem/selftests/i915_gem_context.c | 2 +-
.../drm/i915/gem/selftests/i915_gem_mman.c | 19 +++++++-------
.../drm/i915/gem/selftests/i915_gem_object.c | 2 +-
drivers/gpu/drm/i915/gt/intel_ggtt.c | 14 +++++-----
drivers/gpu/drm/i915/gt/intel_ggtt_fencing.c | 6 ++---
drivers/gpu/drm/i915/gt/intel_gt.c | 7 +++--
drivers/gpu/drm/i915/gt/intel_gt.h | 2 +-
drivers/gpu/drm/i915/gt/intel_region_lmem.c | 4 +--
drivers/gpu/drm/i915/gt/selftest_reset.c | 2 +-
drivers/gpu/drm/i915/gvt/dmabuf.c | 2 +-
drivers/gpu/drm/i915/i915_debugfs.c | 4 +--
drivers/gpu/drm/i915/i915_driver.c | 10 ++++---
drivers/gpu/drm/i915/i915_drv.h | 4 +--
drivers/gpu/drm/i915/i915_gem.c | 23 ++++++++--------
drivers/gpu/drm/i915/i915_gem_gtt.c | 6 ++---
drivers/gpu/drm/i915/i915_getparam.c | 2 +-
drivers/gpu/drm/i915/i915_perf.c | 4 +--
drivers/gpu/drm/i915/selftests/i915_gem.c | 8 +++---
drivers/gpu/drm/i915/selftests/i915_gem_gtt.c | 26 ++++++++++---------
drivers/gpu/drm/i915/selftests/i915_request.c | 2 +-
drivers/gpu/drm/i915/selftests/i915_vma.c | 22 +++++++++-------
.../gpu/drm/i915/selftests/mock_gem_device.c | 11 +++++---
drivers/gpu/drm/i915/selftests/mock_gtt.c | 9 +++----
drivers/gpu/drm/i915/selftests/mock_gtt.h | 3 ++-
35 files changed, 137 insertions(+), 119 deletions(-)
--
2.34.1
next reply other threads:[~2021-12-19 21:25 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-19 21:24 Andi Shyti [this message]
2021-12-19 21:24 ` [PATCH v9 1/6] drm/i915/gt: Use to_gt() helper for GGTT accesses Andi Shyti
2021-12-21 16:47 ` Matt Roper
2021-12-21 19:59 ` [PATCH v10 " Andi Shyti
2021-12-19 21:24 ` [PATCH v9 2/6] drm/i915: " Andi Shyti
2021-12-21 17:01 ` Matt Roper
2021-12-21 19:46 ` Andi Shyti
2022-01-03 21:17 ` Matt Roper
2022-01-04 17:52 ` Umesh Nerlige Ramappa
2022-01-04 22:27 ` Andi Shyti
2022-01-04 22:35 ` [PATCH v10 " Andi Shyti
2022-01-05 5:31 ` Matt Roper
2021-12-19 21:24 ` [PATCH v9 3/6] drm/i915/gem: " Andi Shyti
2021-12-19 21:24 ` [PATCH v9 4/6] drm/i915/display: " Andi Shyti
2021-12-19 21:24 ` [PATCH v9 5/6] drm/i915/selftests: " Andi Shyti
2021-12-19 21:25 ` [PATCH v9 6/6] drm/i915: Remove unused i915->ggtt Andi Shyti
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211219212500.61432-1-andi.shyti@linux.intel.com \
--to=andi.shyti@linux.intel.com \
--cc=andi@etezian.org \
--cc=chris@chris-wilson.co.uk \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=lucas.demarchi@intel.com \
--cc=michal.winiarski@intel.com \
--cc=sujaritha.sundaresan@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).