[Intel-gfx] [PATCH] intel-ci: Add gem_exec_suspend/basic-S3/S4-devices to BAT

[Jani Nikula]

On Mon, 17 Oct 2016, Imre Deak <imre.deak at intel.com> wrote:
> 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 at intel.com>
>> 
>> What's the impact on BAT runtime with this? 
>
> 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.

If our CI was sufficiently clever, it could detect these patches to
change the test list, and let us all know what the impact is...

BR,
Jani.



>
> --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 at gem_exec_store@basic-default
>> >  igt at gem_exec_store@basic-render
>> >  igt at gem_exec_store@basic-vebox
>> >  igt at gem_exec_suspend@basic
>> > +igt at gem_exec_suspend@basic-s3-devices
>> >  igt at gem_exec_suspend@basic-s3
>> > -igt at gem_exec_suspend@basic-s4
>> > +igt at gem_exec_suspend@basic-s4-devices
>> >  igt at gem_flink_basic@bad-flink
>> >  igt at gem_flink_basic@bad-open
>> >  igt at gem_flink_basic@basic
>> > -- 
>> > 2.5.0
>> > 
>> > _______________________________________________
>> > Intel-gfx mailing list
>> > Intel-gfx at lists.freedesktop.org
>> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Jani Nikula, Intel Open Source Technology Center