| Submitter | Stefan Hajnoczi |
|---|---|
| Date | Sept. 1, 2011, 8:06 a.m. |
| Message ID | <1314864386-14202-16-git-send-email-stefanha@linux.vnet.ibm.com> |
| Download | mbox | patch |
| Permalink | /patch/112832/ |
| State | New |
| Headers | show |
Comments
Patch
diff --git a/scripts/simpletrace.py b/scripts/simpletrace.py index 2ad5699..f55e5e6 100755 --- a/scripts/simpletrace.py +++ b/scripts/simpletrace.py @@ -102,10 +102,10 @@ def process(events, log, analyzer): fn_argcount = len(inspect.getargspec(fn)[0]) - 1 if fn_argcount == event_argcount + 1: # Include timestamp as first argument - return lambda _, rec: fn(*rec[1:2 + fn_argcount]) + return lambda _, rec: fn(*rec[1:2 + event_argcount]) else: # Just arguments, no timestamp - return lambda _, rec: fn(*rec[2:2 + fn_argcount]) + return lambda _, rec: fn(*rec[2:2 + event_argcount]) analyzer.begin() fn_cache = {}
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> --- scripts/simpletrace.py | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)