Re: [igt-dev] [PATCH i-g-t] Check for available outputs

["Gupta, Nidhi1" <nidhi1.gupta@intel.com>]

Thanks for the review Chris, what I am trying to do is to execute 
subtest only on the available output not on all, as all the tests which 
I grouped together are dual display tests so I merged them together , 
can you suggest some way to achieve the same if not using dynamic subtest?


-----Original Message-----
From: Chris Wilson <chris@chris-wilson.co.uk> 
Sent: Wednesday, December 2, 2020 3:00 PM
To: igt-dev@lists.freedesktop.org; Gupta, Nidhi1 <nidhi1.gupta@intel.com>
Cc: Gupta, Nidhi1 <nidhi1.gupta@intel.com>; Latvala, Petri <petri.latvala@intel.com>
Subject: Re: [igt-dev] [PATCH i-g-t] Check for available outputs

Quoting nidhi1.gupta@intel.com (2020-12-02 09:17:47)
> From: Nidhi Gupta <nidhi1.gupta@intel.com>
> 
> In kms_cursor_legacy will test all the pipes defined in IGT which is 6 
> weather the output is available or not.Later it has to be analysed and 
> discard it as valid skip.To save this time updated in test to first 
> check the available outputs and then execute the test.

There needs to be much strongly guidelines on when to use dynamic.
Subtests by another name is not ideal.

These are definitely not all the same test, labelling them as minor variations of the same theme is ridiculous.

You are trying to work around a "problem" caused by another tool.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev