From: Karolina Drobnik <karolina.drobnik@intel.com>
To: igt-dev@lists.freedesktop.org
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Subject: [igt-dev] [PATCH i-g-t 0/2] i915/i915_pm_rps: Add fence-order and engine-order subtests
Date: Thu, 28 Jul 2022 17:06:56 +0200 [thread overview]
Message-ID: <cover.1659014747.git.karolina.drobnik@intel.com> (raw)
This series adds two tests that check if the iteration order of fences has
an impact on applying waitboost. These regression tests make sure that
the heuristic works as expected, no matter the order in which we wait on
fences. For more background, see [1] and [2].
In addition to this, the patchset also includes a simple fix for
pm_rps_exit_handler, added in preparation for multi-gt support and per-test
sysfs files initialization.
Following the review comments, the initial patch was split up into two. So,
this series is actually v5 of the "i915/i915_pm_rps: Check impact of fence
ordering on waitboosting" patch. The previous versions can be seen at [3].
v5:
- Added missing close() calls on sysfs handle
- Extracted pm_rps_exit_handler change into a dedicated patch
v4:
- Added test descriptions with igt_describe()
v3:
- Removed unnecessary calls to gem_write
v2:
- Introduced batch_create() with arbitration points to prevent hangs
- Added engine-order subtest
- Added reporting of GPU frequency
- Changed runtimes and switched prints from us to ms
[1] - https://lore.kernel.org/intel-gfx/cover.1657289332.git.karolina.drobnik@intel.com/
[2] - https://gitlab.freedesktop.org/drm/intel/-/issues/6284
[3] - https://patchwork.freedesktop.org/patch/493954
Chris Wilson (2):
i915/i915_pm_rps: Check impact of fence ordering on waitboosting
i915/i915_pm_rps: Write values in pm_rps_exit_handler only for min-max
configs
tests/i915/i915_pm_rps.c | 265 ++++++++++++++++++++++++++++++++++++++-
1 file changed, 259 insertions(+), 6 deletions(-)
--
2.25.1
next reply other threads:[~2022-07-28 15:07 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-28 15:06 Karolina Drobnik [this message]
2022-07-28 15:06 ` [igt-dev] [PATCH i-g-t 1/2] i915/i915_pm_rps: Check impact of fence ordering on waitboosting Karolina Drobnik
2022-07-28 15:33 ` Kamil Konieczny
2022-07-28 15:06 ` [igt-dev] [PATCH i-g-t 2/2] i915/i915_pm_rps: Write values in pm_rps_exit_handler only for min-max configs Karolina Drobnik
2022-07-28 15:34 ` Kamil Konieczny
2022-07-28 15:42 ` [igt-dev] ✓ Fi.CI.BAT: success for i915/i915_pm_rps: Add fence-order and engine-order subtests Patchwork
2022-07-28 22:18 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
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=cover.1659014747.git.karolina.drobnik@intel.com \
--to=karolina.drobnik@intel.com \
--cc=chris@chris-wilson.co.uk \
--cc=igt-dev@lists.freedesktop.org \
/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