Re: [igt-dev] [i-g-t] tests/i915/exec_balancer: Added Skip invalid-bonds

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

On Thu, 19 Aug 2021 19:03:27 -0700, Mastan Katragadda wrote:
>
> Added skip which ever gen12+ platforms not supporting bonding.
>
> Disable bonding on gen12+ platforms aside from ones already supported by
> the i915 - TGL, RKL, and ADL-S.
>
> Link: https://cgit.freedesktop.org/drm-tip/commit/?id=ce7e75c7ef1bf8ea3d947da8c674d2f40fd7d73
>
> Changes since V1:
>	- Updated skip assertion logic (Ashutosh)

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

> Signed-off-by: Mastan Katragadda <mastanx.katragadda@intel.com>
> ---
>  tests/i915/gem_exec_balancer.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/tests/i915/gem_exec_balancer.c b/tests/i915/gem_exec_balancer.c
> index 2f98950b..419efda7 100644
> --- a/tests/i915/gem_exec_balancer.c
> +++ b/tests/i915/gem_exec_balancer.c
> @@ -337,6 +337,7 @@ static void invalid_bonds(int i915)
>	};
>	uint32_t handle;
>	void *ptr;
> +	int ret;
>
>	memset(&engines, 0, sizeof(engines));
>	igt_assert_eq(__set_param_fresh_context(i915, p), 0);
> @@ -349,7 +350,9 @@ static void invalid_bonds(int i915)
>		bonds[n].num_bonds = 1;
>	}
>	engines.extensions = to_user_pointer(&bonds);
> -	igt_assert_eq(__set_param_fresh_context(i915, p), 0);
> +	ret = __set_param_fresh_context(i915, p);
> +	igt_skip_on_f(ret  == -ENODEV, "Bonding not supported\n");
> +	igt_assert_eq(ret, 0);
>
>	bonds[0].base.next_extension = -1ull;
>	igt_assert_eq(__set_param_fresh_context(i915, p), -EFAULT);
> --
> 2.25.1
>