Re: [PATCH i-g-t] tests/intel: Add current IGT PID to driver options

[Sebastian Brzezinka <sebastian.brzezinka@intel.com>]

Hi Krszysztof,

On Thu May 28, 2026 at 7:59 AM CEST, Krzysztof Karas wrote:
> i915 mmap selftests lack trusted address space in userspace,
> so use a driver parameter to pass IGT process's PID to borrow
> memory from for mapping. This will help to restore test
> coverage, which was temporarily limited when following series
> on kernel side was merged:
> e8c3344765aa ("drm/i915/selftests: Prevent userspace mapping invalidation")
> 82eb94b1fa13 ("drm/i915/selftests: Run vma tests only if current->mm is present")
>
> Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com>
> ---
>  tests/intel/i915_selftest.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/intel/i915_selftest.c b/tests/intel/i915_selftest.c
> index 6972de64c..1a8385883 100644
> --- a/tests/intel/i915_selftest.c
> +++ b/tests/intel/i915_selftest.c
> @@ -649,8 +649,8 @@ int igt_main()
>  	igt_kselftests("i915", opts, NULL, "mock", NULL);
>  
>  	igt_assert(snprintf(opts, sizeof(opts),
> -			    "live_selftests=-1 disable_display=1 st_filter=%s",
> -			    env) < sizeof(opts));
> +			    "live_selftests=-1 disable_display=1 st_filter=%s st_userspace_pid=%d",
> +			    env, getpid()) < sizeof(opts));
>  	igt_kselftests("i915", opts, "live_selftests", "live", NULL);
>  
>  	igt_assert(snprintf(opts, sizeof(opts),
LGTM
Reviewed-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com>


-- 
Best regards,
Sebastian