diff options
author | Stefan Hajnoczi <stefanha@redhat.com> | 2014-06-22 21:46:04 +0800 |
---|---|---|
committer | Stefan Hajnoczi <stefanha@redhat.com> | 2014-08-12 14:26:11 +0100 |
commit | a76ccf3c1cb06576af091c5ac8bc264515b1bb7f (patch) | |
tree | 1abca48701be45aa32e8e13e4ef9b1637b8800af /scripts/tracetool | |
parent | 2d591ce2aeebf9620ff527c7946844a3122afeec (diff) | |
download | hqemu-a76ccf3c1cb06576af091c5ac8bc264515b1bb7f.zip hqemu-a76ccf3c1cb06576af091c5ac8bc264515b1bb7f.tar.gz |
trace: extract stap_escape() function for reuse
SystemTap reserved words sometimes conflict with QEMU variable names.
We escape them to prevent conflicts.
Move escaping into its own function so the next patch can reuse it.
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'scripts/tracetool')
-rw-r--r-- | scripts/tracetool/format/stap.py | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/scripts/tracetool/format/stap.py b/scripts/tracetool/format/stap.py index e24abf7..9e780f1 100644 --- a/scripts/tracetool/format/stap.py +++ b/scripts/tracetool/format/stap.py @@ -27,6 +27,13 @@ RESERVED_WORDS = ( ) +def stap_escape(identifier): + # Append underscore to reserved keywords + if identifier in RESERVED_WORDS: + return identifier + '_' + return identifier + + def generate(events, backend): events = [e for e in events if "disable" not in e.properties] @@ -45,9 +52,7 @@ def generate(events, backend): i = 1 if len(e.args) > 0: for name in e.args.names(): - # Append underscore to reserved keywords - if name in RESERVED_WORDS: - name += '_' + name = stap_escape(name) out(' %s = $arg%d;' % (name, i)) i += 1 |