Re: Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-29 Thread Pekka Enberg
On Tue, Oct 29, 2013 at 4:51 PM, Mark Wielaard wrote: > A nice user case to think about when designing this interface might be > the java hotspot jvm (libjvm.so). It has SDT markers with the same name > that might occur at multiple addresses depending on code path taken or > compiler optimization.

Re: Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-29 Thread Mark Wielaard
On Tue, 2013-10-29 at 12:19 +0900, Masami Hiramatsu wrote: > (2013/10/29 2:48), Pekka Enberg wrote: > > For the 32-bit and 64-bit libc case, why cannot 'perf list' > > by default print out something like: > > > > $ perf list > > > >libc:setjmp [SDT marker group] > > > > and provide a '--full

Re: Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-28 Thread Masami Hiramatsu
(2013/10/29 2:48), Pekka Enberg wrote: > On 10/28/13 7:31 PM, Srikar Dronamraju wrote: But what if a system has both 32 bit libc and 64 bit libc? Wont we could end up with 2 libc:setjmp? Should we give some more intelligence into perf to choose the 64 bit libc over 32 bit one? >

Re: Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-28 Thread Masami Hiramatsu
(2013/10/29 2:31), Srikar Dronamraju wrote: >>> >>> But what if a system has both 32 bit libc and 64 bit libc? >>> Wont we could end up with 2 libc:setjmp? >>> Should we give some more intelligence into perf to choose the 64 bit >>> libc over 32 bit one? >> >> You can just trace both of them by def

Re: Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-28 Thread Masami Hiramatsu
(2013/10/25 21:38), Pekka Enberg wrote: > Hello Hemant, > > On Wed, Oct 23, 2013 at 7:05 AM, Hemant Kumar > wrote: >> This allows perf to probe into the sdt markers/notes present in >> the libraries and executables. We try to find the associated location >> and handle prelinking (since, stapsdt

Re: Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-28 Thread Masami Hiramatsu
(2013/10/26 18:50), Ingo Molnar wrote: > > * Srikar Dronamraju wrote: > >> Hi Pekka, >> You can now use it in all perf tools, such as: perf record -e libc:my_event -aR sleep 1 >>> >>> Is there a technical reason why 'perf list' could not show all the >>> available SDT mar