diff mbox

[i-g-t] intel-ci: Reorder tests to have vgem-basic@unload tested first

Message ID 1476697990-22373-1-git-send-email-petri.latvala@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Petri Latvala Oct. 17, 2016, 9:53 a.m. UTC
Signed-off-by: Petri Latvala <petri.latvala@intel.com>
CC: Chris Wilson <chris@chris-wilson.co.uk>
---

(Possible duplicate, I didn't see this arrive on the mailing list myself)

To match Chris's intention.

The test itself will receive fixes later, and another module loading
test will be added at the end. This change is just for mitigation.



tests/intel-ci/fast-feedback.testlist | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Chris Wilson Oct. 17, 2016, 10:29 a.m. UTC | #1
On Mon, Oct 17, 2016 at 12:53:10PM +0300, Petri Latvala wrote:
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> CC: Chris Wilson <chris@chris-wilson.co.uk>
> ---
> 
> (Possible duplicate, I didn't see this arrive on the mailing list myself)
> 
> To match Chris's intention.
> 
> The test itself will receive fixes later, and another module loading
> test will be added at the end. This change is just for mitigation.

The test itself is fine, and working correctly to explain that the
machine setup is unable to even run the controlled unload test. What we
are missing is the information why the CI machines are unhappy.
-Chris
diff mbox

Patch

diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
index f59ec98..711245b 100644
--- a/tests/intel-ci/fast-feedback.testlist
+++ b/tests/intel-ci/fast-feedback.testlist
@@ -1,3 +1,4 @@ 
+igt@vgem_basic@unload
 igt@core_auth@basic-auth
 igt@core_prop_blob@basic
 igt@drv_getparams_basic@basic-eu-total
@@ -243,4 +244,3 @@  igt@vgem_basic@dmabuf-mmap
 igt@vgem_basic@mmap
 igt@vgem_basic@second-client
 igt@vgem_basic@sysfs
-igt@vgem_basic@unload