[PATCH i-g-t] Revert "tests/intel/gem_exec_capture: Deal with DG1 recoverable vs. capture uapi changes"

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed Jun 25 11:08:55 UTC 2025


Hi Krzysztof,
On 2025-06-10 at 16:18:52 +0200, Krzysztof Niemiec wrote:
> This reverts commit be0b7853bb2ded0b183e87bdaeab4318a7e2cd52.
> 
> The corresponding change in the kernel [1] that made this test work on
> DG1 was ultimately reverted [2], which broke the test again. Bring back
> the code to the original state.
> 
> [1] https://lore.kernel.org/all/20250411144313.11660-2-ville.syrjala@linux.intel.com
> [2] https://lore.kernel.org/all/20250522064127.24293-1-joonas.lahtinen@linux.intel.com/
> 
> Signed-off-by: Krzysztof Niemiec <krzysztof.niemiec at intel.com>

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/intel/gem_exec_capture.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/intel/gem_exec_capture.c b/tests/intel/gem_exec_capture.c
> index c0549c0e3..37c1f7255 100644
> --- a/tests/intel/gem_exec_capture.c
> +++ b/tests/intel/gem_exec_capture.c
> @@ -658,7 +658,7 @@ static bool needs_recoverable_ctx(int fd)
>  		return false;
>  
>  	devid = intel_get_drm_devid(fd);
> -	return intel_graphics_ver(devid) > IP_VER(12, 10);
> +	return gem_has_lmem(fd) ||  intel_graphics_ver(devid) > IP_VER(12, 0);
>  }
>  
>  #define find_first_available_engine(fd, ctx, e, saved) \
> -- 
> 2.45.2
> 


More information about the igt-dev mailing list