[igt-dev] [PATCH i-g-t 1/2] i915/kms_chamelium: Require gen < 9 for VGA testing

[Summers, Stuart]

On Mon, 2019-02-18 at 12:57 +0200, Petri Latvala wrote:
> On Thu, Feb 14, 2019 at 04:40:42PM -0800, Stuart Summers via igt-dev
> wrote:
> > From: Stuart Summers <jssummer at jssummer-desk1.amr.corp.intel.com>
> > 
> > VGA not supported on gen 9 and above. Add an explicit requirement
> > to skip VGA tests to a Chamelium on later gens.
> > 
> > Signed-off-by: Stuart Summers <stuart.summers at intel.com>
> > ---
> >  tests/kms_chamelium.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
> > index 64f87d3ae..ab89c5d47 100644
> > --- a/tests/kms_chamelium.c
> > +++ b/tests/kms_chamelium.c
> > @@ -1015,6 +1015,8 @@ igt_main
> >  
> >  	igt_subtest_group {
> >  		igt_fixture {
> > +			int gen =
> > intel_gen(intel_get_drm_devid(data.drm_fd));
> > +			igt_require_f(gen < 9, "VGA not supported\n");
> >  			require_connector_present(
> >  			    &data, DRM_MODE_CONNECTOR_VGA);
> 
> The obvious first comment is that you're not necessarily running on
> an
> intel device. But let's put that aside, there's a more important
> question.

Good point. From your below feedback, I'm thinking of just dropping
this. If it seems interesting to keep this around (if I hear more
feedback here), I'll clean this up.

> 
> What problem does this solve? The line directly after checks for a
> VGA

I personally found this useful to differenciate an invalid system
configuration causing CI to complain (i.e. someone forgot to plug in
the cable) vs an intentional limitation (i.e. the platform doesn't
support this). I agree it isn't strictly necessary since the subsequent
check will prevent any actual test content from being exercised.

Also, maybe a precident of checking all platform capabilities for each
of these isn't really the right direction. I'll drop this. Thanks for
the feedback!

> connector being present. VGA via DP not good enough? If there's
> problems with that, surely there are better ways to get around it
> than
> ignoring it.
> 
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3270 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/igt-dev/attachments/20190219/61bb4ddd/attachment.bin>