On ma, 2016-10-17 at 18:01 +0300, Saarinen, Jani wrote: > HI, > > -----Original Message----- > > From: Intel-gfx [mailto:intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx] On > > Behalf > > Of Imre Deak > > Sent: Monday, October 17, 2016 5:47 PM > > To: Daniel Vetter <daniel@xxxxxxxx> > > Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > > Subject: Re: [PATCH] intel-ci: Add > > gem_exec_suspend/basic- > > S3/S4-devices to BAT > > > > On ma, 2016-10-17 at 16:32 +0200, Daniel Vetter wrote: > > > On Mon, Oct 17, 2016 at 03:39:04PM +0300, Imre Deak wrote: > > > > Add gem_exec_suspend/basic-s3-devices and basic-s4-devices > > > > subtests > > > > to BAT. At the same time remove basic-s4 from the list, which > > > > is atm > > > > implicitly disabled via HIBERNATION=n in kconfig. We would need > > > > at > > > > least basic S4 coverage provided by basic-s4-devices, which > > > > requires > > > > HIBERNATION=y. > > > > > > > > Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx> > > > > > > What's the impact on BAT runtime with this? > Are these also stable enough to be added so that we are not causing > flip-flops? Based on my tests, they are. > > I measured 8 sec for S3-devices and 9 sec for S4-devices on my APL. > > > > > Afaik we're already over budget ... Where do you safe the time to > > > afford this? > > > > I didn't, but we don't have any S4 coverage in CI atm and it's the > > minimum > > that can be added. The S3-devices subtest is not critical, although > > it would be > > useful for cases where we wouldn't get any logs for the full S3 > > test. > > > > --Imre > > > > > -Daniel > > > > > > > --- > > > > tests/intel-ci/fast-feedback.testlist | 3 ++- > > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/tests/intel-ci/fast-feedback.testlist > > > > b/tests/intel- > > > > ci/fast-feedback.testlist index f59ec98..efa7e86 100644 > > > > --- a/tests/intel-ci/fast-feedback.testlist > > > > +++ b/tests/intel-ci/fast-feedback.testlist > > > > @@ -73,8 +73,9 @@ igt@gem_exec_store@basic-default > > > > igt@gem_exec_store@basic-render > > > > igt@gem_exec_store@basic-vebox > > > > igt@gem_exec_suspend@basic > > > > +igt@gem_exec_suspend@basic-s3-devices > > > > igt@gem_exec_suspend@basic-s3 > > > > -igt@gem_exec_suspend@basic-s4 > > > > +igt@gem_exec_suspend@basic-s4-devices > > > > igt@gem_flink_basic@bad-flink > > > > igt@gem_flink_basic@bad-open > > > > igt@gem_flink_basic@basic > > > > -- > > > > 2.5.0 > > > > > > Jani Saarinen > Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo > > > _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
