[igt-dev] [PATCH i-g-t v2] tests/kms_async: Add test to validate asynchronous flips

[Petri Latvala]

On Tue, Apr 14, 2020 at 03:35:37PM -0700, Paulo Zanoni wrote:
> Em ter, 2020-04-14 às 15:41 +0530, Karthik B S escreveu:
> > Asynchronous flips are issued using the page flip IOCTL.
> > The test consists of two subtests. The first subtest waits for
> > the page flip event to be received before giving the next flip,
> > and the second subtest doesn't wait for page flip events.
> > 
> > The test passes if the IOCTL is successful.
> > 
> > v2: Add authors in the test file. (Paulo)
> >     Reduce the run time and timeouts to suit IGT needs. (Paulo)
> >     Replace igt_debug's with igt_assert's to catch slow flips. (Paulo)
> >     Follow IGT coding style regarding spaces. (Paulo)
> >     Make set up code part of igt_fixture. (Paulo)
> >     Skip the test if async flips are not supported. (Paulo)
> >     Replace suggested-by. (Paulo)
> >     Added description for test and subtests.
> > 
> > Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
> > Signed-off-by: Karthik B S <karthik.b.s at intel.com>
> > ---
> >  tests/Makefile.sources |   1 +
> >  tests/kms_async.c      | 243 +++++++++++++++++++++++++++++++++++++++++
> >  tests/meson.build      |   1 +
> >  3 files changed, 245 insertions(+)
> >  create mode 100644 tests/kms_async.c
> > 
> > diff --git a/tests/Makefile.sources b/tests/Makefile.sources
> > index 4e44c98c..eedf4fcb 100644
> > --- a/tests/Makefile.sources
> > +++ b/tests/Makefile.sources
> > @@ -29,6 +29,7 @@ TESTS_progs = \
> >  	fbdev \
> >  	kms_3d \
> >  	kms_addfb_basic \
> > +	kms_async \
> >  	kms_atomic \
> >  	kms_atomic_interruptible \
> >  	kms_atomic_transition \
> > diff --git a/tests/kms_async.c b/tests/kms_async.c
> > new file mode 100644
> > index 00000000..6031ff4e
> > --- /dev/null
> > +++ b/tests/kms_async.c
> 
> Async what? :)
> 
> kms_async_flips.c is probably better IMHO since it's clearer on what
> the test tests.
> 
> Also, do we have to add this to some IGT test list file if we want this
> in CI? Can the IGT/CI people comment here please?

No. The full round executes everything except the tests matching
tests/intel-ci/blacklist.txt.


-- 
Petri Latvala