Skip to content

Commit 2e44fdd

Browse files
committed
Add more logging when saving LAVA results
When saving LAVA results logfile is sometimes in a wrong path. This patch adds more logging to make sure results are written to the proper file. Signed-off-by: Milosz Wasilewski <milosz.wasilewski@oss.qualcomm.com>
1 parent d7f9247 commit 2e44fdd

File tree

7 files changed

+24
-3
lines changed

7 files changed

+24
-3
lines changed

Runner/suites/Kernel/Baseport/IPCC/run.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,12 @@ count=$(echo "$output" | grep -c "running")
4545

4646
if [ $count -eq 4 ]; then
4747
log_pass "$TESTNAME : Test Passed"
48+
log_info "Writing to file $res_file"
4849
echo "$TESTNAME PASS" > "$res_file"
4950
exit 0
5051
else
5152
log_fail "$TESTNAME : Test Failed"
53+
log_info "Writing to file $res_file"
5254
echo "$TESTNAME FAIL" > "$res_file"
5355
exit 1
5456
fi

Runner/suites/Kernel/Baseport/adsp_remoteproc/run.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,7 @@ discover_audio_stack_and_snapshot
159159
# --- Check DT presence for ADSP -----------------------------------------------
160160
if ! dt_has_remoteproc_fw "$FW"; then
161161
log_skip "$TESTNAME SKIP – $FW not described in DT"
162+
log_info "Writing to file $RES_FILE"
162163
echo "$TESTNAME SKIP" >"$RES_FILE"
163164
exit 0
164165
fi
@@ -168,6 +169,7 @@ log_info "DT indicates $FW is present"
168169
entries="$(get_remoteproc_by_firmware "$FW" "" all 2>/dev/null)" || entries=""
169170
if [ -z "$entries" ]; then
170171
log_fail "$FW present in DT but no /sys/class/remoteproc entry found"
172+
log_info "Writing to file $RES_FILE"
171173
echo "$TESTNAME FAIL" >"$RES_FILE"
172174
exit 1
173175
fi
@@ -208,6 +210,7 @@ while IFS='|' read -r rpath rstate rfirm rname; do
208210
if [ "$FATAL_ON_UNSUSPENDED" -eq 1 ]; then
209211
if [ "$(audio_pm_snapshot_ok)" -ne 1 ]; then
210212
log_fail "Audio not in suspended/unsupported state after ${PRE_STOP_DELAY}s (FATAL_ON_UNSUSPENDED=1); aborting before stop"
213+
log_info "Writing to file $RES_FILE"
211214
echo "$TESTNAME FAIL" >"$RES_FILE"
212215
exit 1
213216
fi
@@ -283,10 +286,12 @@ log_info "Instance results:$RESULT_LINES"
283286

284287
if [ "$inst_fail" -gt 0 ]; then
285288
log_fail "One or more $FW instance(s) failed ($inst_fail/$count_instances)"
289+
log_info "Writing to file $RES_FILE"
286290
echo "$TESTNAME FAIL" >"$RES_FILE"
287291
exit 1
288292
fi
289293

290294
log_pass "All $count_instances $FW instance(s) passed"
295+
log_info "Writing to file $RES_FILE"
291296
echo "$TESTNAME PASS" >"$RES_FILE"
292297
exit 0

Runner/suites/Kernel/Baseport/cdsp_remoteproc/run.sh

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,8 @@ if dt_has_remoteproc_fw "$FW"; then
5252
log_info "DT indicates $FW is present"
5353
else
5454
log_skip "$TESTNAME SKIP – $FW not described in DT"
55-
echo "${TESTNAME} SKIP" >"$RES_FILE"
55+
log_info "Writing to $RES_FILE"
56+
echo "${TESTNAME} SKIP" > "$RES_FILE"
5657
exit 0
5758
fi
5859

@@ -161,10 +162,12 @@ log_info "Instance results:$RESULT_LINES"
161162

162163
if [ "$inst_fail" -gt 0 ]; then
163164
log_fail "One or more $FW instance(s) failed ($inst_fail/$count_instances)"
164-
echo "$TESTNAME FAIL" >"$RES_FILE"
165+
log_info "Writing to $RES_FILE"
166+
echo "$TESTNAME FAIL" > "$RES_FILE"
165167
exit 1
166168
fi
167169

168170
log_pass "All $count_instances $FW instance(s) passed"
169-
echo "$TESTNAME PASS" >"$RES_FILE"
171+
log_info "Writing to $RES_FILE"
172+
echo "$TESTNAME PASS" > "$RES_FILE"
170173
exit 0

Runner/suites/Kernel/Baseport/iommu/run.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,19 +56,22 @@ pass=true
5656
CONFIGS="CONFIG_IOMMU_SUPPORT CONFIG_QCOM_IOMMU CONFIG_ARM_SMMU"
5757
check_kernel_config "$CONFIGS" || {
5858
log_fail "Kernel config validation failed."
59+
log_info "Writing to file $res_file"
5960
echo "$TESTNAME FAIL" > "$res_file"
6061
exit 1
6162
}
6263
LOADED_MODULES="msm_iommu arm_smmu"
6364
check_driver_loaded "$LOADED_MODULES" || {
6465
log_fail "Failed to load required driver modules"
66+
log_info "Writing to file $res_file"
6567
echo "$TESTNAME FAIL" > "$res_file"
6668
exit 1
6769
}
6870

6971
DT_NODES="/proc/device-tree/soc@0/iommu@15000000 /proc/device-tree/soc/iommu@15000000"
7072
check_dt_nodes "$DT_NODES" || {
7173
log_fail "Device tree validation failed."
74+
log_info "Writing to file $res_file"
7275
echo "$TESTNAME FAIL" > "$res_file"
7376
exit 1
7477
}
@@ -77,10 +80,12 @@ check_runtime_behavior || pass=false
7780

7881
if $pass; then
7982
log_pass "$TESTNAME : Test Passed"
83+
log_info "Writing to file $res_file"
8084
echo "$TESTNAME PASS" > "$res_file"
8185
exit 0
8286
else
8387
log_fail "$TESTNAME : Test Failed"
88+
log_info "Writing to file $res_file"
8489
echo "$TESTNAME FAIL" > "$res_file"
8590
exit 1
8691
fi

Runner/suites/Kernel/Baseport/kaslr/run.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,12 @@ value=$(echo $output | awk '{print $1}')
4545

4646
if [ $value != "0000000000000000" ]; then
4747
log_pass "$TESTNAME : Test Passed"
48+
log_info "Writing to file $res_file"
4849
echo "$TESTNAME PASS" > "$res_file"
4950
exit 0
5051
else
5152
log_fail "$TESTNAME : Test Failed"
53+
log_info "Writing to file $res_file"
5254
echo "$TESTNAME FAIL" > "$res_file"
5355
exit 1
5456
fi

Runner/suites/Kernel/Baseport/pinctrl/run.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,10 +47,12 @@ output=$(ls /sys/kernel/debug/pinctrl)
4747
# Print overall test result
4848
if [ -z "$output" ]; then
4949
log_fail "$TESTNAME : Test Failed"
50+
log_info "Writing to file $res_file"
5051
echo "$TESTNAME FAIL" > "$res_file"
5152
exit 1
5253
else
5354
log_pass "$TESTNAME : Test Passed"
55+
log_info "Writing to file $res_file"
5456
echo "$TESTNAME PASS" > "$res_file"
5557
exit 0
5658
fi

Runner/suites/Kernel/Baseport/smmu/run.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,12 @@ OUTPUT=$(dmesg | grep iommu)
4545
# Check if the output is null
4646
if [ -z "$OUTPUT" ]; then
4747
log_fail "$TESTNAME : Test Failed"
48+
log_info "Writing to $res_file"
4849
echo "$TESTNAME FAIL" > "$res_file"
4950
exit 1
5051
else
5152
log_pass "$TESTNAME : Test Passed"
53+
log_info "Writing to $res_file"
5254
echo "$TESTNAME PASS" > "$res_file"
5355
exit 0
5456
fi

0 commit comments

Comments
 (0)