tracetool: show trace-events filename/lineno in fmt string errors
commit7fb48c0ee1bbf5cc4c905e900b054096250e9f39
authorStefan Hajnoczi <stefanha@redhat.com>
Thu, 27 Aug 2020 14:29:15 +0000 (27 15:29 +0100)
committerStefan Hajnoczi <stefanha@redhat.com>
Mon, 4 Jan 2021 14:24:58 +0000 (4 14:24 +0000)
tree612ce599994ea35c2a4dc9a6b490668f6743030b
parent4e66c9ef643b5f6bc01de33eac6fa0335e4c0753
tracetool: show trace-events filename/lineno in fmt string errors

The compiler encounters trace event format strings in generated code.
Format strings are error-prone and therefore clear compiler errors are
important.

Use the #line directive to show the trace-events filename and line
number in format string errors:
https://gcc.gnu.org/onlinedocs/gcc-10.2.0/cpp/Line-Control.html

For example, if the cpu_in trace event's %u is changed to %p the
following error is reported:

  trace-events:29:18: error: format ‘%p’ expects argument of type ‘void *’, but argument 7 has type ‘unsigned int’ [-Werror=format=]

Line 29 in trace-events is where cpu_in is defined. This works for any
trace-events file in the QEMU source tree and the correct path is
displayed.

Unfortunately there does not seem to be a way to set the column, so "18"
is not the right character on that line.

Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20200827142915.108730-5-stefanha@redhat.com>
scripts/tracetool/backend/ftrace.py
scripts/tracetool/backend/log.py
scripts/tracetool/backend/syslog.py