The simple trace code generator has been rewritten for QEMU 1.1 and now
assigns event numbers only to enabled events.  This means we must skip
disabled events when pretty-printing traces in simpletrace.py.

Note this means old binary traces may be pretty printed incorrectly
since they use a different event numbering when the "disable" keyword is
present in ./trace-events.  It's unfortunate but not easy to avoid at
this stage.  Always use the simpletrace.py that came with the QEMU
binary to ensure correctly pretty-printing.

Reported-by: Juan Quintela <[email protected]>
Signed-off-by: Stefan Hajnoczi <[email protected]>
---
 scripts/simpletrace.py |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/scripts/simpletrace.py b/scripts/simpletrace.py
index f55e5e6..db39b02 100755
--- a/scripts/simpletrace.py
+++ b/scripts/simpletrace.py
@@ -37,8 +37,9 @@ def parse_events(fobj):
             continue
 
         disable, name, args = m.groups()
-        events[event_num] = (name,) + get_argnames(args)
-        event_num += 1
+        if not disable:
+            events[event_num] = (name,) + get_argnames(args)
+            event_num += 1
     return events
 
 def read_record(fobj):
-- 
1.7.10


Reply via email to