Merge branch 'qualcomm-linux:main' into main

Author: nklazy

.github/workflows/check-executable-permissions.yml

@@ -0,0 +1,46 @@
+name: Enforce Script Executable Permissions
+
+on:
+  pull_request_target:
+    branches: [ "main" ]
+    paths:
+      - '**/run.sh'
+      - '**/*.sh'
+  push:
+    branches: [ "main" ]
+  workflow_dispatch:
+
+jobs:
+  permissions:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Detect missing executable permissions on shell scripts
+        run: |
+          # Find all .sh and run.sh scripts without +x
+          BAD=$(find . -type f \( -name "*.sh" -o -name "run.sh" \) ! -perm -u=x)
+          if [ -n "$BAD" ]; then
+            echo "::error file=run.sh,line=1::❌ Some shell scripts are missing executable permissions. This can break CI and LAVA. Please fix before merging."
+            echo "::error file=run.sh,line=2::To fix, run: find . -name '*.sh' -o -name 'run.sh' | xargs chmod +x && git add . && git commit -m 'Fix: restore executable bits on scripts' && git push"
+            echo ""
+            echo "The following scripts need 'chmod +x':"
+            echo "$BAD"
+            # Output a PR annotation for each file
+            echo "$BAD" | while read -r file; do
+              echo "::error file=$file,line=1::$file is not executable. Please run: chmod +x $file && git add $file"
+            done
+            exit 1
+          else
+            echo "✅ All shell scripts have correct executable permissions."
+          fi
+
+      - name: Detect accidental executables on non-shell files (optional, warning only)
+        run: |
+          # (Advanced/optional) Warn if any non-.sh file has +x (customize as needed)
+          OTHER_EXEC=$(find . -type f ! -name '*.sh' ! -name 'run.sh' -perm -u=x)
+          if [ -n "$OTHER_EXEC" ]; then
+            echo "::warning file=run.sh,line=1::Warning: Non-shell files with executable permissions detected. Review if needed."
+            echo "$OTHER_EXEC"
+          fi

.github/workflows/preflight-checker-workflow.yml

@@ -6,12 +6,16 @@ on:
     branches: [ "main" ]
   workflow_dispatch:
 
+permissions:
+ contents: read
+ security-events: write
+
 jobs:
   checker:
     uses: qualcomm-linux/qli-actions/.github/workflows/multi-checker.yml@main
     with:
         repolinter: true # default: true
-        semgrep: false # default: true
+        semgrep: true # default: true
         copyright-license-detector: true # default: true
         pr-check-emails: true # default: true
 

.github/workflows/shellcheck.yml

@@ -1,6 +1,11 @@
 name: Shell Lint
 
-on: [pull_request, push]
+on:
+  pull_request_target:
+    branches: [ "main" ]
+  push:
+    branches: [ "main" ]
+  workflow_dispatch:
 
 jobs:
   shellcheck:

Runner/plans/qcom-next-ci-premerge.yaml

@@ -52,4 +52,8 @@ run:
         - $PWD/utils/send-to-lava.sh $PWD/suites/Kernel/FunctionalArea/baseport/storage/storage.res || true
         - $PWD/suites/Kernel/FunctionalArea/baseport/wpss_remoteproc/run.sh || true
         - $PWD/utils/send-to-lava.sh $PWD/suites/Kernel/FunctionalArea/baseport/wpss_remoteproc/wpss_remoteproc.res || true
+        - $PWD/suites/Kernel/FunctionalArea/DCVS/Freq_Scaling/run.sh || true
+        - $PWD/utils/send-to-lava.sh $PWD/suites/Kernel/FunctionalArea/DCVS/Freq_Scaling/Freq_Scaling.res || true
+        - $PWD/suites/Kernel/FunctionalArea/Scheduler/CPU_affinity/run.sh || true
+        - $PWD/utils/send-to-lava.sh $PWD/suites/Kernel/FunctionalArea/Scheduler/CPU_affinity/CPU_affinity.res || true
         - $PWD/utils/result_parse.sh

Runner/run-test.sh

@@ -34,6 +34,7 @@ fi
 # Store results
 RESULTS_PASS=""
 RESULTS_FAIL=""
+RESULTS_SKIP=""
 
 execute_test_case() {
     test_path=$1
@@ -43,20 +44,37 @@ execute_test_case() {
         run_script="$test_path/run.sh"
         if [ -f "$run_script" ]; then
             log "Executing test case: $test_name"
-            if (cd "$test_path" && sh "./run.sh"); then
-                log_pass "$test_name passed"
-                if [ -z "$RESULTS_PASS" ]; then
-                    RESULTS_PASS="$test_name"
+            (cd "$test_path" && sh "./run.sh")
+            res_file="$test_path/$test_name.res"
+            if [ -f "$res_file" ]; then
+                if grep -q "SKIP" "$res_file"; then
+                    log_skip "$test_name skipped"
+                    if [ -z "$RESULTS_SKIP" ]; then
+                        RESULTS_SKIP="$test_name"
+                    else
+                        RESULTS_SKIP=$(printf "%s\n%s" "$RESULTS_SKIP" "$test_name")
+                    fi
+                elif grep -q "PASS" "$res_file"; then
+                    log_pass "$test_name passed"
+                    if [ -z "$RESULTS_PASS" ]; then
+                        RESULTS_PASS="$test_name"
+                    else
+                        RESULTS_PASS=$(printf "%s\n%s" "$RESULTS_PASS" "$test_name")
+                    fi
+                elif grep -q "FAIL" "$res_file"; then
+                    log_fail "$test_name failed"
+                    if [ -z "$RESULTS_FAIL" ]; then
+                        RESULTS_FAIL="$test_name"
+                    else
+                        RESULTS_FAIL=$(printf "%s\n%s" "$RESULTS_FAIL" "$test_name")
+                    fi
                 else
-                    RESULTS_PASS=$(printf "%s\n%s" "$RESULTS_PASS" "$test_name")
+                    log_fail "$test_name: unknown result in .res file"
+                    RESULTS_FAIL=$(printf "%s\n%s" "$RESULTS_FAIL" "$test_name (unknown result)")
                 fi
             else
-                log_fail "$test_name failed"
-                if [ -z "$RESULTS_FAIL" ]; then
-                    RESULTS_FAIL="$test_name"
-                else
-                    RESULTS_FAIL=$(printf "%s\n%s" "$RESULTS_FAIL" "$test_name")
-                fi
+                log_fail "$test_name: .res file not found"
+                RESULTS_FAIL=$(printf "%s\n%s" "$RESULTS_FAIL" "$test_name (.res not found)")
             fi
         else
             log_error "No run.sh found in $test_path"
@@ -95,6 +113,9 @@ print_summary() {
     echo
     echo "FAILED:"
     [ -n "$RESULTS_FAIL" ] && printf "%s\n" "$RESULTS_FAIL" || echo " None"
+    echo
+    echo "SKIPPED:"
+    [ -n "$RESULTS_SKIP" ] && printf "%s\n" "$RESULTS_SKIP" || echo " None"
     log_info "=================================="
 }
 

Runner/suites/Kernel/FunctionalArea/DCVS/Freq_Scaling/run.sh

@@ -86,9 +86,11 @@ kill $LOAD_PID
 if [ "$CURRENT_FREQ" -gt "$MIN_FREQ" ]; then
     log_pass "DCVS scaling appears functional. Test Passed"
     echo "$TESTNAME PASS" > "$res_file"
+    exit 0
 else
     log_fail "DCVS did not scale as expected. Test Failed"
     echo "$TESTNAME FAIL" > "$res_file"
+    exit 1
 fi
 
 log_info "----------- Completed $TESTNAME Test ------------"

Runner/suites/Kernel/FunctionalArea/Scheduler/CPU_affinity/run.sh

@@ -39,7 +39,7 @@ log_info "-------- Starting $TESTNAME Functional Test --------"
 
 check_dependencies taskset top chrt zcat grep
 
-REQUIRED_CONFIGS="CONFIG_SCHED_DEBUG CONFIG_CGROUP_SCHED CONFIG_SMP"
+REQUIRED_CONFIGS="CONFIG_CGROUP_SCHED CONFIG_SMP"
 for config in $REQUIRED_CONFIGS; do
     if check_kernel_config "$config"; then
         log_pass "$config is enabled"
@@ -83,9 +83,11 @@ log_info "Scheduling Policy: $SCHED_POLICY"
 if echo "$SCHED_POLICY" | grep -q "SCHED_OTHER"; then
     log_pass "Default scheduling policy detected. Test passed"
     echo "$TESTNAME PASS" > "$res_file"
+    exit 0
 else
     log_fail "Unexpected scheduling policy. Test Failed"
     echo "$TESTNAME FAIL" > "$res_file"
+    exit 1
 fi
 
 kill $TASK_PID

Runner/suites/Kernel/FunctionalArea/baseport/BWMON/run.sh

@@ -83,8 +83,10 @@ done
 if $incremented; then
     log_pass "$TESTNAME : Test Passed"
     echo "$TESTNAME PASS" > "$res_file" 
+    exit 0
 else
     log_fail "$TESTNAME : Test Failed"
     echo "$TESTNAME FAIL" > "$res_file"
+    exit 1
 fi
 log_info "-------------------Completed $TESTNAME Testcase----------------------------"

Runner/suites/Kernel/FunctionalArea/baseport/Buses/run.sh

@@ -47,8 +47,10 @@ output=$(i2c-msm-test -v -D /dev/i2c-0 -l | grep "ret:1")
 if echo "$output" | grep -q "Reading"; then
     log_pass "$TESTNAME : Test Passed"
     echo "$TESTNAME PASS" > "$res_file"
+    exit 0
 else
     log_fail "$TESTNAME : Test Failed"
-    echo "$TESTNAME FAIL" > "$res_file" 
+    echo "$TESTNAME FAIL" > "$res_file"
+    exit 1 
 fi
 log_info "-------------------Completed $TESTNAME Testcase----------------------------"

Runner/suites/Kernel/FunctionalArea/baseport/CPUFreq_Validation/run.sh

@@ -109,9 +109,11 @@ log_info "=== Final Result ==="
 if [ "$overall_pass" -eq 0 ]; then
     log_pass "$TESTNAME: All policies passed"
     echo "$TESTNAME PASS" > "$res_file"
+    exit 0
 else
     log_fail "$TESTNAME: One or more policies failed"
     echo "$TESTNAME FAIL" > "$res_file"
+    exit 1
 fi
 
 rm -rf "$status_dir"