On Tue, Jan 13, 2015 at 12:47:59PM +0000, Zhiqiang Zhang wrote:
> From:Jiri Olsa <[email protected]>
> 
> commit f61ff6c06dc8f32c7036013ad802c899ec590607 upstream
> 
> Linus reported perf report command being interrupted due to processing
> of 'out of order' event, with following error:
> 
> Timestamp below last timeslice flush
> 0x5733a8 [0x28]: failed to process type: 3
> 
> I could reproduce the issue and in my case it was caused by one CPU
> (mmap) being behind during record and userspace mmap reader seeing the
> data after other CPUs data were already stored.
> 
> This is expected under some circumstances because we need to limit the
> number of events that we queue for reordering when we receive a
> PERF_RECORD_FINISHED_ROUND or when we force flush due to memory
> pressure.
> 
> Reported-by: Linus Torvalds <[email protected]>
> Signed-off-by: Jiri Olsa <[email protected]>
> Acked-by: Ingo Molnar <[email protected]>
> Cc: Andi Kleen <[email protected]>
> Cc: Corey Ashford <[email protected]>
> Cc: David Ahern <[email protected]>
> Cc: Frederic Weisbecker <[email protected]>
> Cc: Ingo Molnar <[email protected]>
> Cc: Linus Torvalds <[email protected]>
> Cc: Matt Fleming <[email protected]>
> Cc: Namhyung Kim <[email protected]>
> Cc: Paul Mackerras <[email protected]>
> Cc: Peter Zijlstra <[email protected]>
> Cc: Stephane Eranian <[email protected]>
> Link: 
> http://lkml.kernel.org/r/[email protected]
> Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
> 
> [zhangzhiqiang: backport to 3.10:
>  - adjust context
>  - commit f61ff6c06d struct events_stats was defined in 
> tools/perf/util/event.h
>    while 3.10 stable defined in tools/perf/util/hist.h.
>  - 3.10 stable there is no pr_oe_time() which used for debug.
>  - After the above adjustments, becomes same to the original patch:
>    
> https://github.com/torvalds/linux/commit/f61ff6c06dc8f32c7036013ad802c899ec590607

seems ok

jirka
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to