Skip to content

Commit 37fb5cc

Browse files
committed
fix: tools: sof_perf_analyzer: ignore errors when decoding mtrace file
Ignore faulty lines in mtrace file when analyzing it with sof_perf_analyzer.py script Signed-off-by: Emilia Kurdybelska <emiliax.kurdybelska@intel.com>
1 parent 53d0819 commit 37fb5cc

File tree

3 files changed

+4
-7
lines changed

3 files changed

+4
-7
lines changed

case-lib/hijack.sh

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -169,10 +169,7 @@ function func_exit_handler()
169169

170170
# Only do performance analysis on test passed
171171
[ "x$exit_status" != "x0" ] || {
172-
# On performance analysis return error, set exit code
173-
if ! perf_analyze; then
174-
exit_status=1
175-
fi
172+
perf_analyze || dlogw 'perf_analyze failed in exit handler'
176173
}
177174

178175
# Only run when changing topology was initiated during the test

case-lib/lib.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1496,11 +1496,11 @@ perf_analyze()
14961496
fi
14971497
dlogc "$perf_cmd"
14981498
eval "$perf_cmd" || {
1499-
dloge "SOF component performace analysis tool exit with error"
1499+
dlogw "SOF component performace analysis tool exit with error"
15001500
return 1
15011501
}
15021502
else
1503-
dloge "Firmware trace file not found"
1503+
dlogw "Firmware trace file not found"
15041504
return 1
15051505
fi
15061506
}

tools/sof_perf_analyzer.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ def process_trace_file():
173173
#
174174
# pylint: disable=C0103
175175
ts_shift = 0
176-
with open(args.filename, 'r', encoding='utf8') as file:
176+
with open(args.filename, 'r', encoding='utf8', errors='ignore') as file:
177177
trace_item_gen = make_trace_item(file)
178178
trace_prev = None
179179
try:

0 commit comments

Comments
 (0)