[igt-dev] [PATCH i-g-t v2] tests/kms_cursor_legacy: Disable Intel's PSR2 selective fetch in this test
Hogander, Jouni
jouni.hogander at intel.com
Thu Nov 25 06:27:39 UTC 2021
On Wed, 2021-11-24 at 13:10 -0800, José Roberto de Souza wrote:
> When doing a primary or sprite plane flip, PSR2 selective fetch code
> also adds all the planes including cursor that overlaps with the
> area being updated, so this causes legacy cursor API calls to
> wait for a pending atomic commit to finish causing tests that do
> checks with vblank counters.
>
> So here when running in an Intel platform that has PSR2 selective
> fetch enabled, it will switch to PSR1 before executing the subtests.
> Because what this whole test mostly wants to do, is check if
> userspace
> can do asynchronous cursors updates.
>
> v2:
> - rename functions and add documentation
Reviewed-by: Jouni Högander <jouni.hogander at intel.com>
>
> Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/2346
> Cc: Mika Kahola <mika.kahola at intel.com>
> Cc: Jouni Hogander <jouni.hogander at intel.com>
> Cc: Petri Latvala <petri.latvala at intel.com>
> Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
> ---
> lib/igt_psr.c | 44
> +++++++++++++++++++++++++++++++++++++++
> lib/igt_psr.h | 3 +++
> tests/kms_cursor_legacy.c | 10 +++++++++
> 3 files changed, 57 insertions(+)
>
> diff --git a/lib/igt_psr.c b/lib/igt_psr.c
> index 8fa8b192f..98eb28b40 100644
> --- a/lib/igt_psr.c
> +++ b/lib/igt_psr.c
> @@ -280,3 +280,47 @@ bool i915_psr2_selective_fetch_check(int drm_fd)
>
> return ret;
> }
> +
> +/**
> + * i915_psr2_sel_fetch_to_psr1
> + *
> + * Check if PSR2 selective fetch is enabled, if yes switch to PSR1
> and returns
> + * true otherwise returns false.
> + * This function should be called from tests that are not compatible
> with PSR2
> + * selective fetch.
> + *
> + * Returns:
> + * True if PSR mode changed to PSR1, false otherwise.
> + */
> +bool i915_psr2_sel_fetch_to_psr1(int drm_fd)
> +{
> + int debugfs_fd;
> + bool ret = false;
> +
> + if (!is_i915_device(drm_fd))
> + return ret;
> +
> + debugfs_fd = igt_debugfs_dir(drm_fd);
> + if (psr2_selective_fetch_check(debugfs_fd)) {
> + psr_set(drm_fd, debugfs_fd, PSR_MODE_1);
> + ret = true;
> + }
> +
> + close(debugfs_fd);
> + return ret;
> +}
> +
> +/**
> + * i915_psr2_sel_fetch_restore
> + *
> + * Restore PSR2 selective fetch after tests were executed, this
> function should
> + * only be called if i915_psr2_sel_fetch_to_psr1() returned true.
> + */
> +void i915_psr2_sel_fetch_restore(int drm_fd)
> +{
> + int debugfs_fd;
> +
> + debugfs_fd = igt_debugfs_dir(drm_fd);
> + psr_set(drm_fd, debugfs_fd, PSR_MODE_2_SEL_FETCH);
> + close(debugfs_fd);
> +}
> diff --git a/lib/igt_psr.h b/lib/igt_psr.h
> index a075f336d..76cd26c65 100644
> --- a/lib/igt_psr.h
> +++ b/lib/igt_psr.h
> @@ -49,4 +49,7 @@ void psr_print_debugfs(int debugfs_fd);
>
> bool i915_psr2_selective_fetch_check(int drm_fd);
>
> +bool i915_psr2_sel_fetch_to_psr1(int drm_fd);
> +void i915_psr2_sel_fetch_restore(int drm_fd);
> +
> #endif
> diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
> index cd2f84984..d50155e89 100644
> --- a/tests/kms_cursor_legacy.c
> +++ b/tests/kms_cursor_legacy.c
> @@ -27,6 +27,7 @@
>
> #include "i915/gem.h"
> #include "igt.h"
> +#include "igt_psr.h"
> #include "igt_rand.h"
> #include "igt_stats.h"
>
> @@ -1421,6 +1422,7 @@ igt_main
> {
> const int ncpus = sysconf(_SC_NPROCESSORS_ONLN);
> igt_display_t display = { .drm_fd = -1 };
> + bool intel_psr2_restore = false;
> int i;
>
> igt_fixture {
> @@ -1428,6 +1430,12 @@ igt_main
> kmstest_set_vt_graphics_mode();
>
> igt_display_require(&display, display.drm_fd);
> + /*
> + * Not possible to evade vblank after a primary or
> sprite plane
> + * page flip with cursor legacy APIS when Intel's PSR2
> selective
> + * fetch is enabled, so switching PSR1 for this whole
> test.
> + */
> + intel_psr2_restore =
> i915_psr2_sel_fetch_to_psr1(display.drm_fd);
> }
>
> /*Test description for pipe-* and all-pipe-* tests*/
> @@ -1609,6 +1617,8 @@ igt_main
> }
>
> igt_fixture {
> + if (intel_psr2_restore)
> + i915_psr2_sel_fetch_restore(display.drm_fd);
> igt_display_fini(&display);
> }
> }
More information about the igt-dev
mailing list