diff mbox

tests: Move pm_rps to the right Makefile target

Message ID 1390139359-4141-1-git-send-email-daniel.vetter@ffwll.ch (mailing list archive)
State New, archived
Headers show

Commit Message

Daniel Vetter Jan. 19, 2014, 1:49 p.m. UTC
If it's not in the multi-test target group testrunners won't pick up
on the fact that they need to enumerate subtests first.

Cc: jeff.mcgee@intel.com
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
---
 tests/Makefile.sources | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

jeff.mcgee@intel.com Jan. 20, 2014, 3:06 p.m. UTC | #1
On Sun, Jan 19, 2014 at 02:49:19PM +0100, Daniel Vetter wrote:
> If it's not in the multi-test target group testrunners won't pick up
> on the fact that they need to enumerate subtests first.
> 

OK. I haven't yet tried piglit, so missed this detail. Thanks

Jeff
Daniel Vetter Jan. 20, 2014, 4:14 p.m. UTC | #2
On Mon, Jan 20, 2014 at 09:06:40AM -0600, Jeff McGee wrote:
> On Sun, Jan 19, 2014 at 02:49:19PM +0100, Daniel Vetter wrote:
> > If it's not in the multi-test target group testrunners won't pick up
> > on the fact that they need to enumerate subtests first.
> > 
> 
> OK. I haven't yet tried piglit, so missed this detail. Thanks

Hm, how are you running testcases then on your side? Or are you just
running all the subtests in one go (which also works)?
-Daniel
jeff.mcgee@intel.com Jan. 21, 2014, 10:54 p.m. UTC | #3
On Mon, Jan 20, 2014 at 05:14:23PM +0100, Daniel Vetter wrote:
> On Mon, Jan 20, 2014 at 09:06:40AM -0600, Jeff McGee wrote:
> > On Sun, Jan 19, 2014 at 02:49:19PM +0100, Daniel Vetter wrote:
> > > If it's not in the multi-test target group testrunners won't pick up
> > > on the fact that they need to enumerate subtests first.
> > > 
> > 
> > OK. I haven't yet tried piglit, so missed this detail. Thanks
> 
> Hm, how are you running testcases then on your side? Or are you just
> running all the subtests in one go (which also works)?
> -Daniel

I'm just manually building and running the test binary that I'm interested
in, which is just pm_rps for now. I'm working in Android for which overall
support is still a little spotty I think. Or at least that was my initial
perception and so I'm keeping things simple.

Jeff
diff mbox

Patch

diff --git a/tests/Makefile.sources b/tests/Makefile.sources
index c586bd3ed5a2..74ea1b528eae 100644
--- a/tests/Makefile.sources
+++ b/tests/Makefile.sources
@@ -62,6 +62,7 @@  TESTS_progs_M = \
 	kms_setmode \
 	pm_lpsp \
 	pm_pc8 \
+	pm_rps \
 	prime_self_import \
 	template \
 	$(NULL)
@@ -122,7 +123,6 @@  TESTS_progs = \
 	gen3_render_tiledy_blits \
 	pm_psr \
 	pm_rc6_residency \
-	pm_rps \
 	prime_udl \
 	$(NULL)