mirror of
https://gitlab.com/qemu-project/qemu
synced 2024-10-15 15:32:51 +00:00
simpletrace: fix process() argument count
The simpletrace.process() function invokes analyzer methods with the wrong number of arguments if a timestamp should be included. This patch fixes the issue so that trace analysis scripts can make use of timestamps. Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
This commit is contained in:
parent
47f08d7a9d
commit
d8e8ef4ee0
|
@ -102,10 +102,10 @@ def build_fn(analyzer, event):
|
||||||
fn_argcount = len(inspect.getargspec(fn)[0]) - 1
|
fn_argcount = len(inspect.getargspec(fn)[0]) - 1
|
||||||
if fn_argcount == event_argcount + 1:
|
if fn_argcount == event_argcount + 1:
|
||||||
# Include timestamp as first argument
|
# Include timestamp as first argument
|
||||||
return lambda _, rec: fn(*rec[1:2 + fn_argcount])
|
return lambda _, rec: fn(*rec[1:2 + event_argcount])
|
||||||
else:
|
else:
|
||||||
# Just arguments, no timestamp
|
# Just arguments, no timestamp
|
||||||
return lambda _, rec: fn(*rec[2:2 + fn_argcount])
|
return lambda _, rec: fn(*rec[2:2 + event_argcount])
|
||||||
|
|
||||||
analyzer.begin()
|
analyzer.begin()
|
||||||
fn_cache = {}
|
fn_cache = {}
|
||||||
|
|
Loading…
Reference in a new issue