diff options
author | 2023-06-07 23:18:12 -0700 | |
---|---|---|
committer | 2023-06-14 18:19:06 -0300 | |
commit | 701677b95764c06bb058c92be11c3a4ad25ab5f2 (patch) | |
tree | 3fb3b56c1fa8a39221f42aad9d8bf47e5ee22a69 /tools/perf/scripts/python/export-to-postgresql.py | |
parent | tools api: Add simple timeout to io read (diff) | |
download | wireguard-linux-701677b95764c06bb058c92be11c3a4ad25ab5f2.tar.xz wireguard-linux-701677b95764c06bb058c92be11c3a4ad25ab5f2.zip |
perf srcline: Add a timeout to reading from addr2line
addr2line may fail to send expected values causing perf to wait
indefinitely. Add a 1 second timeout (twice the timeout for reading from
/proc/pid/maps) so that such reads don't cause perf to appear to lock
up.
There are already checks that the file for addr2line contains a debug
section but this isn't always sufficient. The problem was observed when
a valid elf file would set the configuration for binutils addr2line,
then a later read of vmlinux with ELF debug sections would cause a
failing write/read which would block indefinitely.
As a service to future readers, if the io hits eof or an error, cleanup
the addr2line process.
Signed-off-by: Ian Rogers <irogers@google.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Yang Jihong <yangjihong1@huawei.com>
Link: https://lore.kernel.org/r/20230608061812.3715566-2-irogers@google.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/scripts/python/export-to-postgresql.py')
0 files changed, 0 insertions, 0 deletions