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

2016-11-25 Thread Petri Latvala
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

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

2016-10-18 Thread Ville Syrjälä
On Tue, Oct 18, 2016 at 12:03:12PM +0300, Petri Latvala wrote: > On Tue, Oct 18, 2016 at 09:34:49AM +0200, Daniel Vetter wrote: > > Re S4: If it indeed improves coverage (i.e. calls our shutdown hooks and > > all these S4-only callbacks) then adding it to BAT sounds reasonable. > > Still there's

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

2016-10-18 Thread Petri Latvala
On Tue, Oct 18, 2016 at 09:34:49AM +0200, Daniel Vetter wrote: > Re S4: If it indeed improves coverage (i.e. calls our shutdown hooks and > all these S4-only callbacks) then adding it to BAT sounds reasonable. > Still there's the issue of where to get the machine time from. I really do > think you

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

2016-10-18 Thread Daniel Vetter
On Mon, Oct 17, 2016 at 05:46:50PM +0300, Imre Deak 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

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

2016-10-17 Thread Imre Deak
ter <dan...@ffwll.ch> > > Cc: intel-gfx@lists.freedesktop.org > > Subject: Re: [Intel-gfx] [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

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

2016-10-17 Thread Saarinen, Jani
-gfx] [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 > >

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

2016-10-17 Thread Jani Nikula
On Mon, 17 Oct 2016, Imre Deak 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

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

2016-10-17 Thread Imre Deak
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

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

2016-10-17 Thread Daniel Vetter
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

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

2016-10-17 Thread Imre Deak
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.