From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by gabe.freedesktop.org (Postfix) with ESMTPS id 2244A6EB28 for ; Fri, 20 Aug 2021 21:49:33 +0000 (UTC) Date: Fri, 20 Aug 2021 14:44:44 -0700 Message-ID: <8735r3aj03.wl-ashutosh.dixit@intel.com> From: "Dixit, Ashutosh" In-Reply-To: <20210820020327.612886-1-mastanx.katragadda@intel.com> References: <20210820020327.612886-1-mastanx.katragadda@intel.com> MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII Subject: Re: [igt-dev] [i-g-t] tests/i915/exec_balancer: Added Skip invalid-bonds List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: Mastan Katragadda Cc: igt-dev@lists.freedesktop.org, tejaskumarx.surendrakumar.upadhyay@intel.com List-ID: 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 > Signed-off-by: Mastan Katragadda > --- > 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 >