From 0d4baedb1ccbe54c898957ae5b08bd504b44408c Mon Sep 17 00:00:00 2001
From: tahaelbayad <taha.elbayad@studio.unibo.it>
Date: Fri, 13 Dec 2024 19:45:43 +0100
Subject: [PATCH] add victor's comment

---
 .github/workflows/CI.yml                        |  3 ++-
 .github/workflows/TestRunnerSnitch.yml          |  9 ++++++++-
 ....yml => TestRunnerTiledSnitchSequential.yml} | 17 +++++++++++++++--
 3 files changed, 25 insertions(+), 4 deletions(-)
 rename .github/workflows/{TestRunnerTiledSnitch.yml => TestRunnerTiledSnitchSequential.yml} (71%)

diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
index 2cad9742..e359aaca 100644
--- a/.github/workflows/CI.yml
+++ b/.github/workflows/CI.yml
@@ -103,9 +103,10 @@ jobs:
         testRQGEMM
         TestRQAdd
         testRQGEMMTransB
+      num-cores: 9
 
   snitch-kernels-tiled-singlebuffer-L2:
-    uses: ./.github/workflows/TestRunnerTiledSnitch.yml
+    uses: ./.github/workflows/TestRunnerTiledSnitchSequential.yml
     with:
       tests-config: |
         [
diff --git a/.github/workflows/TestRunnerSnitch.yml b/.github/workflows/TestRunnerSnitch.yml
index 3a0f9a0f..c7ae8c09 100644
--- a/.github/workflows/TestRunnerSnitch.yml
+++ b/.github/workflows/TestRunnerSnitch.yml
@@ -6,6 +6,13 @@ on:
       test-names:
         required: true
         type: string
+      num-cores:
+        required: true
+        type: number
+      simulator:
+        required: false
+        default: "banshee"
+        type: string      
 
 jobs:
   test-runner-snitch:
@@ -26,7 +33,7 @@ jobs:
           echo "$testNames" | while IFS= read -r testName; do
             if [[ -n "$testName" ]]; then
               echo "Running test: $testName"
-              python testRunner_snitch.py -t Tests/$testName --simulator=banshee --cores=9 --toolchain_install_dir /app/install/riscv-llvm/
+              python testRunner_snitch.py -t Tests/$testName --simulator=${{ inputs.simulator }} --cores=${{ inputs.num-cores }} --toolchain_install_dir /app/install/riscv-llvm/
             fi
           done
         shell: bash
\ No newline at end of file
diff --git a/.github/workflows/TestRunnerTiledSnitch.yml b/.github/workflows/TestRunnerTiledSnitchSequential.yml
similarity index 71%
rename from .github/workflows/TestRunnerTiledSnitch.yml
rename to .github/workflows/TestRunnerTiledSnitchSequential.yml
index c1099580..10a101a9 100644
--- a/.github/workflows/TestRunnerTiledSnitch.yml
+++ b/.github/workflows/TestRunnerTiledSnitchSequential.yml
@@ -1,4 +1,4 @@
-name: TestRunnerTiledSnitch
+name: TestRunnerTiledSnitchSequential
 
 on:
   workflow_call:
@@ -6,6 +6,19 @@ on:
       tests-config:
         required: true
         type: string
+      num-cores:
+        required: false
+        default: 9
+        type: number
+      default-memory-level:
+        required: false
+        default: "L2"
+        type: string
+      simulator:
+        required: false
+        default: "banshee"
+        type: string
+
 
 jobs:
 
@@ -40,7 +53,7 @@ jobs:
             L1_values=$(echo "$test" | jq -r '.L1[]')
             for L1_value in $L1_values; do
               echo "Running test: $testName with L1: $L1_value"
-              python testRunner_tiled_snitch.py -t Tests/$testName --cores=9 --simulator=banshee --l1 $L1_value --defaultMemLevel=L2 --toolchain_install_dir /app/install/riscv-llvm/
+              python testRunner_tiled_snitch.py -t Tests/$testName --cores=${{ inputs.num-cores }} --simulator=${{ inputs.simulator }} --l1 $L1_value --defaultMemLevel=${{ inputs.default-memory-level }} --toolchain_install_dir /app/install/riscv-llvm/
             done
           done
         shell: bash