From e0b0cd61d2ffde48d7fc69623c4a59005a4c58af Mon Sep 17 00:00:00 2001 From: Keir Fraser Date: Wed, 3 Feb 2010 09:36:37 +0000 Subject: xentrace: Disable tracing, then read records one more time. When interrupted, first disable tracing, then read through the records one last time. Without this patch, it's possible to get traces which interact (such as runstate changes) on processors with higher numbers, while missing the corresponding traces generated on lower-numbered processors. Signed-off-by: George Dunlap --- tools/xentrace/xentrace.c | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) (limited to 'tools/xentrace') diff --git a/tools/xentrace/xentrace.c b/tools/xentrace/xentrace.c index 7f63e7fc9d..18283f3e97 100644 --- a/tools/xentrace/xentrace.c +++ b/tools/xentrace/xentrace.c @@ -634,6 +634,8 @@ static int monitor_tbufs(void) unsigned long data_size; + int last_read = 1; + /* prepare to listen for VIRQ_TBUF */ event_init(); @@ -712,14 +714,22 @@ static int monitor_tbufs(void) } if ( interrupted ) - break; + { + if ( last_read ) + { + /* Disable tracing, then read through all the buffers one last time */ + if ( opts.disable_tracing ) + disable_tbufs(); + last_read = 0; + continue; + } + else + break; + } wait_for_event_or_timeout(opts.poll_sleep); } - if ( opts.disable_tracing ) - disable_tbufs(); - if ( opts.memory_buffer ) membuf_dump(); -- cgit v1.2.3