Messages in this thread | | | From | Namhyung Kim <> | Subject | Re: [PATCH] perf tools: Use tid for finding thread | Date | Mon, 12 May 2014 09:50:01 +0900 |
| |
On Sun, 11 May 2014 15:15:42 +0200, Jiri Olsa wrote: > On Wed, May 07, 2014 at 11:26:06AM +0900, Namhyung Kim wrote: >> I believe that passing pid (instead of tid) as the 3rd arg of the >> machine__find*_thread() was to find a main thread so that it can >> search proper map group for symbols. However with the map sharing >> patch applied, it now can do it in any thread. >> >> It fixes a bug when each thread has different name, it only reports a >> main thread for samples in other threads. > > and breaks tests 14 and 26 ;-) > > ... > 14: Test matching and linking multiple hists : FAILED! > 15: Try 'use perf' in python, checking link problems : Ok > 16: Test breakpoint overflow signal handler : Ok > 17: Test breakpoint overflow sampling : Ok > 18: Test number of exit event of a simple workload : Ok > 19: Test software clock events have valid period values : Ok > 20: Test converting perf time to TSC : Ok > 21: Test object code reading : Ok > 22: Test sample parsing : Ok > 23: Test using a dummy software event to keep tracking : Ok > 24: Test parsing with no sample_id_all bit set : Ok > 25: Test dwarf unwind : Ok > 26: Test filtering hist entries : FAILED! > ...
Forgot to set sample->tid for those cases. Will fix.
Thanks, Namhyung
| |