qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] tracetool: use relative paths for '#line' preprocessor directives
@ 2023-04-06  8:00 Thomas De Schampheleire
  2023-04-09 12:50 ` Stefan Hajnoczi
  2023-04-11 10:44 ` Philippe Mathieu-Daudé
  0 siblings, 2 replies; 3+ messages in thread
From: Thomas De Schampheleire @ 2023-04-06  8:00 UTC (permalink / raw)
  To: qemu-devel; +Cc: Stefan Hajnoczi, Thomas De Schampheleire

The event filename is an absolute path. Convert it to a relative path when
writing '#line' directives, to preserve reproducibility of the generated
output when different base paths are used.

Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
---
 scripts/tracetool/backend/ftrace.py | 4 +++-
 scripts/tracetool/backend/log.py    | 4 +++-
 scripts/tracetool/backend/syslog.py | 4 +++-
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/scripts/tracetool/backend/ftrace.py b/scripts/tracetool/backend/ftrace.py
index 5fa30ccc08..baed2ae61c 100644
--- a/scripts/tracetool/backend/ftrace.py
+++ b/scripts/tracetool/backend/ftrace.py
@@ -12,6 +12,8 @@
 __email__      = "stefanha@redhat.com"
 
 
+import os.path
+
 from tracetool import out
 
 
@@ -45,7 +47,7 @@ def generate_h(event, group):
         args=event.args,
         event_id="TRACE_" + event.name.upper(),
         event_lineno=event.lineno,
-        event_filename=event.filename,
+        event_filename=os.path.relpath(event.filename),
         fmt=event.fmt.rstrip("\n"),
         argnames=argnames)
 
diff --git a/scripts/tracetool/backend/log.py b/scripts/tracetool/backend/log.py
index 17ba1cd90e..de27b7e62e 100644
--- a/scripts/tracetool/backend/log.py
+++ b/scripts/tracetool/backend/log.py
@@ -12,6 +12,8 @@
 __email__      = "stefanha@redhat.com"
 
 
+import os.path
+
 from tracetool import out
 
 
@@ -53,7 +55,7 @@ def generate_h(event, group):
         '    }',
         cond=cond,
         event_lineno=event.lineno,
-        event_filename=event.filename,
+        event_filename=os.path.relpath(event.filename),
         name=event.name,
         fmt=event.fmt.rstrip("\n"),
         argnames=argnames)
diff --git a/scripts/tracetool/backend/syslog.py b/scripts/tracetool/backend/syslog.py
index 5a3a00fe31..012970f6cc 100644
--- a/scripts/tracetool/backend/syslog.py
+++ b/scripts/tracetool/backend/syslog.py
@@ -12,6 +12,8 @@
 __email__      = "stefanha@redhat.com"
 
 
+import os.path
+
 from tracetool import out
 
 
@@ -41,7 +43,7 @@ def generate_h(event, group):
         '    }',
         cond=cond,
         event_lineno=event.lineno,
-        event_filename=event.filename,
+        event_filename=os.path.relpath(event.filename),
         name=event.name,
         fmt=event.fmt.rstrip("\n"),
         argnames=argnames)
-- 
2.39.2



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] tracetool: use relative paths for '#line' preprocessor directives
  2023-04-06  8:00 [PATCH] tracetool: use relative paths for '#line' preprocessor directives Thomas De Schampheleire
@ 2023-04-09 12:50 ` Stefan Hajnoczi
  2023-04-11 10:44 ` Philippe Mathieu-Daudé
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Hajnoczi @ 2023-04-09 12:50 UTC (permalink / raw)
  To: Thomas De Schampheleire; +Cc: qemu-devel, Stefan Hajnoczi

On Thu, 6 Apr 2023 at 09:22, Thomas De Schampheleire
<thomas.de_schampheleire@nokia.com> wrote:
>
> The event filename is an absolute path. Convert it to a relative path when
> writing '#line' directives, to preserve reproducibility of the generated
> output when different base paths are used.
>
> Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
> ---
>  scripts/tracetool/backend/ftrace.py | 4 +++-
>  scripts/tracetool/backend/log.py    | 4 +++-
>  scripts/tracetool/backend/syslog.py | 4 +++-
>  3 files changed, 9 insertions(+), 3 deletions(-)

Thanks, applied to my block-next tree:
https://gitlab.com/stefanha/qemu/-/tree/block-next

qemu.git/master is currently frozen for the 8.0 release. The latest
release candidate is -rc3 and the final release is imminent. I don't
want to risk breaking and delaying the release at this stage, so I
have queued up this patch for the 8.1 release cycle.

The timing is unlucky, I would have liked to still include this in the
8.0 release. Thank you for the patch!

Stefan


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] tracetool: use relative paths for '#line' preprocessor directives
  2023-04-06  8:00 [PATCH] tracetool: use relative paths for '#line' preprocessor directives Thomas De Schampheleire
  2023-04-09 12:50 ` Stefan Hajnoczi
@ 2023-04-11 10:44 ` Philippe Mathieu-Daudé
  1 sibling, 0 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2023-04-11 10:44 UTC (permalink / raw)
  To: Thomas De Schampheleire, qemu-devel; +Cc: Stefan Hajnoczi

On 6/4/23 10:00, Thomas De Schampheleire wrote:
> The event filename is an absolute path. Convert it to a relative path when
> writing '#line' directives, to preserve reproducibility of the generated
> output when different base paths are used.
> 
> Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
> ---
>   scripts/tracetool/backend/ftrace.py | 4 +++-
>   scripts/tracetool/backend/log.py    | 4 +++-
>   scripts/tracetool/backend/syslog.py | 4 +++-
>   3 files changed, 9 insertions(+), 3 deletions(-)

Nice.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-04-11 10:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-04-06  8:00 [PATCH] tracetool: use relative paths for '#line' preprocessor directives Thomas De Schampheleire
2023-04-09 12:50 ` Stefan Hajnoczi
2023-04-11 10:44 ` Philippe Mathieu-Daudé

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).