Re: [igt-dev] [PATCH i-g-t v1 1/1] tests/gem_pxp: Pre-MTL prep: PXP is now global.

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Mon, 05 Dec 2022 16:48:56 -0800, Alan Previn wrote:
>
> From the PXP IGT, we assume PXP is a GT subsystem when we
> access then PXP debugfs to force a teardown for testiability.
> As part of kernel side PXP preparation work, the recent
> Intel GFX patchwork's series titled "drm/i915/pxp: Promote
> pxp subsystem to top-level of i915" has moved PXP to be a
> top level i915 subsystem. See kernel side rev9 URL at
> https://patchwork.freedesktop.org/patch/514043/?series=111650&rev=1).
>
> To align with that, we need to change the debugfs path for pxp to
> ensure several of the substests (that rely on debugfs to simulate
> events that cannot be simulated by user-space) continue to work as
> expected.

Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>

>
> NOTE: this patch should only be merged after the kernel side
> has merged.

Well either way there will be failures: new kernel with old igt or the
other way round...

> Signed-off-by: Alan Previn <alan.previn.teres.alexis@intel.com>
> ---
>  tests/i915/gem_pxp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/i915/gem_pxp.c b/tests/i915/gem_pxp.c
> index ffad9b5a..6cf254ca 100644
> --- a/tests/i915/gem_pxp.c
> +++ b/tests/i915/gem_pxp.c
> @@ -847,7 +847,7 @@ static void trigger_pxp_debugfs_forced_teardown(int i915)
>	int fd, ret;
>	char str[32];
>
> -	fd = igt_debugfs_open(i915, "gt/pxp/terminate_state", O_RDWR);
> +	fd = igt_debugfs_open(i915, "pxp/terminate_state", O_RDWR);
>	igt_assert_f(fd >= 0, "Can't open pxp termination debugfs\n");
>	ret = snprintf(str, sizeof(str), "0x1");
>	igt_assert(ret > 2 && ret < sizeof(str));
>
> base-commit: c001793d5f22deb01918b6ba52af829794582df1
> --
> 2.34.1
>