Module: Mesa
Branch: master
Commit: 1d75595da4a72c2ca0eb2b9b88db09a05a873e67
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=1d75595da4a72c2ca0eb2b9b88db09a05a873e67

Author: Andres Gomez <[email protected]>
Date:   Thu Feb 20 17:32:08 2020 +0200

gitlab-ci: Change devices format to <api-vendor-deviceId>

In preparation to having "vk" (Vulkan) along "gl" (OpenGL/ES).

This is so it is clearer which traces belong to which API and also for
the build jobs.

Signed-off-by: Andres Gomez <[email protected]>
Reviewed-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Alexandros Frantzis <[email protected]>

---

 .gitlab-ci.yml                     |  2 +-
 .gitlab-ci/traces.yml              |  6 +++---
 .gitlab-ci/tracie/README.md        | 12 ++++++------
 .gitlab-ci/tracie/tests/test.sh    | 24 +++++++++++++-----------
 .gitlab-ci/tracie/tests/traces.yml |  4 ++--
 5 files changed, 25 insertions(+), 23 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 6a69e6e853b..e6891997e51 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -736,5 +736,5 @@ llvmpipe-traces:
   variables:
     LIBGL_ALWAYS_SOFTWARE: "true"
     GALLIUM_DRIVER: "llvmpipe"
-    DEVICE_NAME: "vmware-llvmpipe"
+    DEVICE_NAME: "gl-vmware-llvmpipe"
   extends: .traces-test
diff --git a/.gitlab-ci/traces.yml b/.gitlab-ci/traces.yml
index d390f185c20..cbb668071a0 100644
--- a/.gitlab-ci/traces.yml
+++ b/.gitlab-ci/traces.yml
@@ -5,13 +5,13 @@ traces-db:
 traces:
   - path: 
glmark2/desktop-blur-radius=5:effect=blur:passes=1:separable=true:windows=4.rdc
     expectations:
-      - device: vmware-llvmpipe
+      - device: gl-vmware-llvmpipe
         checksum: 8867f3a41f180626d0d4b7661ff5c0f4
   - path: glmark2/jellyfish.rdc
     expectations:
-      - device: vmware-llvmpipe
+      - device: gl-vmware-llvmpipe
         checksum: e0fe979fee129c0ed42a3059d1a4e1c9
   - path: glxgears/glxgears.trace
     expectations:
-      - device: vmware-llvmpipe
+      - device: gl-vmware-llvmpipe
         checksum: 02aca9b4b4ad6fd60331df6e4f87f2cd
diff --git a/.gitlab-ci/tracie/README.md b/.gitlab-ci/tracie/README.md
index 5c1f8a881ac..8ea51c6d598 100644
--- a/.gitlab-ci/tracie/README.md
+++ b/.gitlab-ci/tracie/README.md
@@ -17,13 +17,13 @@ traces-db:
 traces:
   - path: glmark2/jellyfish.rdc
     expectations:
-      - device: intel-0x3185
+      - device: gl-intel-0x3185
         checksum: 58359ea4caf6ad44c6b65526881bbd17
-      - device: vmware-llvmpipe
+      - device: gl-vmware-llvmpipe
         checksum: d82267c25a0decdad7b563c56bb81106
   - path: supertuxkart/supertuxkart-antediluvian-abyss.rdc
     expectations:
-      - device: intel-0x3185
+      - device: gl-intel-0x3185
         checksum: ff827f7eb069afd87cc305a422cba939
 ```
 
@@ -56,9 +56,9 @@ To enable trace testing on a new device:
 
    ```yaml
    my-hardware-traces:
+     extends: .traces-test-gl
      variables:
        DEVICE_NAME: "myhardware"
-     extends: .traces-test
    ```
 
 2. Update the .gitlab-ci/traces.yml file with expectations for the new device.
@@ -110,8 +110,8 @@ appended.
 
 Examples:
 
-    python3 dump_traces_images.py --device-name=vmware-llvmpipe mytrace.trace
-    python3 dump_traces_images.py --device-name=vmware-llvmpipe 
--calls=2075,3300 mytrace.trace
+    python3 dump_traces_images.py --device-name=gl-vmware-llvmpipe 
mytrace.trace
+    python3 dump_traces_images.py --device-name=gl-vmware-llvmpipe 
--calls=2075,3300 mytrace.trace
 
 ### Running the replay scripts locally
 
diff --git a/.gitlab-ci/tracie/tests/test.sh b/.gitlab-ci/tracie/tests/test.sh
index 3ef2144fd84..4d52578c47e 100755
--- a/.gitlab-ci/tracie/tests/test.sh
+++ b/.gitlab-ci/tracie/tests/test.sh
@@ -31,8 +31,10 @@ assert() {
 }
 
 run_tracie() {
-    # Run tests for the .testtrace types, using the "test-device" device name.
-    DEVICE_NAME=test-device CI_PROJECT_DIR="$TEST_DIR" \
+    # Run tests for the .testtrace types, using the "gl-test-device" and 
"vk-test-device" device names.
+    DEVICE_NAME=gl-test-device CI_PROJECT_DIR="$TEST_DIR" \
+        "$TEST_DIR/tracie.sh" "$TEST_DIR/tests/traces.yml" testtrace && \
+    DEVICE_NAME=vk-test-device CI_PROJECT_DIR="$TEST_DIR" \
         "$TEST_DIR/tracie.sh" "$TEST_DIR/tests/traces.yml" testtrace
 }
 
@@ -102,7 +104,7 @@ tracie_ignores_unspecified_trace_types() {
 
     echo "  - path: trace1/empty.trace" >> "$TEST_DIR/tests/traces.yml"
     echo "    expectations:" >> "$TEST_DIR/tests/traces.yml"
-    echo "    - device: test-device" >> "$TEST_DIR/tests/traces.yml"
+    echo "    - device: gl-test-device" >> "$TEST_DIR/tests/traces.yml"
     echo "      checksum: 000000000000000" >> "$TEST_DIR/tests/traces.yml"
     # For the tests we only scan for the .testtrace type,
     # so the .trace file added below should be ignored.
@@ -160,11 +162,11 @@ tracie_stores_only_logs_on_checksum_match() {
     run_tracie
     assert "[ $? = 0 ]"
 
-    assert "[ -f 
"$TEST_DIR/results/trace1/test/test-device/magenta.testtrace.log" ]"
-    assert "[ -f 
"$TEST_DIR/results/trace2/test/test-device/olive.testtrace.log" ]"
+    assert "[ -f 
"$TEST_DIR/results/trace1/test/gl-test-device/magenta.testtrace.log" ]"
+    assert "[ -f 
"$TEST_DIR/results/trace2/test/vk-test-device/olive.testtrace.log" ]"
 
-    assert "[ ! -f 
"$TEST_DIR/results/trace1/test/test-device/magenta.testtrace-0.png" ]"
-    assert "[ ! -f 
"$TEST_DIR/results/trace2/test/test-device/olive.testtrace-0.png" ]"
+    assert "[ ! -f 
"$TEST_DIR/results/trace1/test/gl-test-device/magenta.testtrace-0.png" ]"
+    assert "[ ! -f 
"$TEST_DIR/results/trace2/test/vk-test-device/olive.testtrace-0.png" ]"
 
     ls -lR "$TEST_DIR"
 
@@ -181,8 +183,8 @@ tracie_stores_images_on_checksum_mismatch() {
     run_tracie
     assert "[ $? != 0 ]"
 
-    assert "[ ! -f 
"$TEST_DIR/results/trace1/test/test-device/magenta.testtrace-0.png" ]"
-    assert "[ -f 
"$TEST_DIR/results/trace2/test/test-device/olive.testtrace-0.png" ]"
+    assert "[ ! -f 
"$TEST_DIR/results/trace1/test/gl-test-device/magenta.testtrace-0.png" ]"
+    assert "[ -f 
"$TEST_DIR/results/trace2/test/vk-test-device/olive.testtrace-0.png" ]"
 
     destroy_repo "$repo"
 }
@@ -194,8 +196,8 @@ tracie_stores_images_on_request() {
     (export TRACIE_STORE_IMAGES=1; run_tracie)
     assert "[ $? = 0 ]"
 
-    assert "[ -f 
"$TEST_DIR/results/trace1/test/test-device/magenta.testtrace-0.png" ]"
-    assert "[ -f 
"$TEST_DIR/results/trace2/test/test-device/olive.testtrace-0.png" ]"
+    assert "[ -f 
"$TEST_DIR/results/trace1/test/gl-test-device/magenta.testtrace-0.png" ]"
+    assert "[ -f 
"$TEST_DIR/results/trace2/test/vk-test-device/olive.testtrace-0.png" ]"
 
     ls -lR "$TEST_DIR"
 
diff --git a/.gitlab-ci/tracie/tests/traces.yml 
b/.gitlab-ci/tracie/tests/traces.yml
index dc6dd04b526..64e3e4fce2b 100644
--- a/.gitlab-ci/tracie/tests/traces.yml
+++ b/.gitlab-ci/tracie/tests/traces.yml
@@ -1,9 +1,9 @@
 traces:
   - path: trace1/magenta.testtrace
     expectations:
-      - device: test-device
+      - device: gl-test-device
         checksum: 8e0a801367e1714463475a824dab363b
   - path: trace2/olive.testtrace
     expectations:
-      - device: test-device
+      - device: vk-test-device
         checksum: 5efda83854befe0155ff8517a58d5b51

_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to