Re: [igt-dev] [PATCH i-g-t 20/20] tests/psr: Add more PSR subtests to fast feedback list

From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Cc: "igt-dev@lists.freedesktop.org" <igt-dev@lists.freedesktop.org>
Subject: Re: [igt-dev] [PATCH i-g-t 20/20] tests/psr: Add more PSR subtests to fast feedback list
Date: Tue, 24 Apr 2018 14:28:17 -0700	[thread overview]
Message-ID: <20180424212817.GM1994@intel.com> (raw)
In-Reply-To: <1524592123.24461.4.camel@dk-H97M-D3H>

On Tue, Apr 24, 2018 at 10:48:43AM -0700, Dhinakaran Pandiyan wrote:
> 
> 
> 
> On Tue, 2018-04-24 at 15:32 +0000, Saarinen, Jani wrote:
> > HI, 
> > 
> > > -----Original Message-----
> > > From: igt-dev [mailto:igt-dev-bounces@lists.freedesktop.org] On Behalf Of Petri
> > > Latvala
> > > Sent: tiistai 24. huhtikuuta 2018 12.34
> > > To: Pandiyan, Dhinakaran <dhinakaran.pandiyan@intel.com>; Vivi, Rodrigo
> > > <rodrigo.vivi@intel.com>
> > > Cc: igt-dev@lists.freedesktop.org
> > > Subject: Re: [igt-dev] [PATCH i-g-t 20/20] tests/psr: Add more PSR subtests to
> > > fast feedback list
> > > 
> > > 
> > > 
> > > On 04/24/2018 02:58 AM, Dhinakaran Pandiyan wrote:
> > > >
> > > >
> > > > On Mon, 2018-04-23 at 16:06 -0700, Rodrigo Vivi wrote:
> > > >> On Sat, Apr 14, 2018 at 06:13:40PM -0700, Dhinakaran Pandiyan wrote:
> > > >>> On Fri, 2018-04-13 at 12:16 +0300, Petri Latvala wrote:
> > > >>>> On Wed, Apr 11, 2018 at 07:14:36PM +0300, Pandiyan, Dhinakaran wrote:
> > > >>>>> Yeah, that is expected. The kernel patches are not in yet.
> > > >>>> The requirement for using --sink-crc is going away when kernel side
> > > >>>> patches land? Or how is this planned?
> > > >>>>
> > > >>> Irrespective of whether or when the kernel patches land, --sink-crc
> > > >>> will not be used for CI. --sink-crc is being retained for developers
> > > >>> to manually use it for debug and possibly fix the code in future.
> > > >>>
> > > >>> Having said that, the kernel patches adding time stamp support
> > > >>> should land soon, after which the tests will not skip.
> > > >> Ok, I finally looked to all patches on this series and I support all of them.
> > > >>
> > > >> My only concern is actually with patch 14 that is around this discussion here...
> > > >>
> > > >> For me it seems that or you run with interactive debug on or with
> > > >> sink_crc and CI is not running with any so it will all skip.
> > > >>
> > > >> Unless I missunderstood something on that patch...
> > > >>
> > > >> If CI still runs without requiring any extra option I'm in favor of
> > > >> the whole series and I'm tending to merge it soon...
> > > >>
> > > > CI will not need any options. Timestamps are supported by default on
> > > > all platforms except CHV/VLV. We don't have those platforms in CI and
> > > > I am planning to nuke PSR support for them.
> > > >
> > > 
> > > These machines are VLV:
> > > fi-byt-j1900
> > > fi-byt-n2820
> > Also CHV=BSW => fi-bsw-n3050
> > 
> 
> No psr panels on them.

Ok, so it seems no need for waiting...

Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
on the series.

but let's see if Petri has more concerns.

> 
> 
> > > 
> > > 
> > > --
> > > Petri Latvala
> > > 
> > > _______________________________________________
> > > igt-dev mailing list
> > > igt-dev@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/igt-dev
> > _______________________________________________
> > igt-dev mailing list
> > igt-dev@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/igt-dev
> 
> _______________________________________________
> igt-dev mailing list
> igt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev