OpenVZ Forum: Devel » [PATCH 0/7] Profiling sleep times (v3)

Home » Mailing lists » Devel » [PATCH 0/7] Profiling sleep times (v3)

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

Re: [PATCH 0/7] Profiling sleep times (v3) [message #44405 is a reply to message #44255]

Tue, 06 December 2011 07:15

Andrey Vagin
Messages: 9
Registered: December 2011

Junior Member

Hello Arnaldo,

I found, that you commit patches for perf. Could you review this part?

Arun Sharma said, that the second versions of patches works ok for him.
(Arun is the first user of this functionality after me.)

Thanks.

On 11/28/2011 01:03 PM, Andrew Vagin wrote:
> Do you want to know where your code waits locks for a long time?
> Yes! It's for you. This feature helps you to find bottlenecks.
>
> It's not artificial task. Once one of my colleague was investigating a
> scalability problem. He pressed sysrq-t some times and tried to merge
> call-chains by hand. But perf can do that.
>
> Problem:
> The problem is that events sched_stat_* contain call-chains of
> non-target tasks.
> About month ago I sent series of patches:
>
> [PATCH 0/3] trace: add ability to collect call chains of non current task.
>
> Peter and Frederic explained me, that this solve isn't good and will be
> better to make it in userspace.
>
> Now it's in userspace. This series expands "perf inject" to be able to
> merge sched_switch events and sched_stat* events. sched_switch events
> contain correct call-chains and sched_stat contains a correct time
> slices.
>
> v2:
> * Removed all known issues. Now it works completely.
> * Improved usability of sched-stat scripts according with Arun's comments.
> v3: fixed accoding to comments from David Ahem
>
> Andrew Vagin (7):
> perf: use event_name() to get an event name
> perf: add ability to record event period
> perf: add ability to change event according to sample (v2)
> perf: teach "perf inject" to work with files
> perf: teach perf inject to merge sched_stat_* and sched_switch events
> perf: add scripts for profiling sleep times (v2)
> event: add tracepoint for accounting block time
>
> include/trace/events/sched.h | 7 ++
> kernel/sched_fair.c | 2 +
> tools/perf/builtin-inject.c | 120 ++++++++++++++++++++++-
> tools/perf/builtin-record.c | 5 +
> tools/perf/scripts/python/bin/sched-stat-record | 65 ++++++++++++
> tools/perf/scripts/python/bin/sched-stat-report | 5 +
> tools/perf/util/event.h | 2 +
> tools/perf/util/evsel.c | 74 ++++++++++++++
> tools/perf/util/header.c | 2 +-
> tools/perf/util/session.h | 9 ++
> 10 files changed, 288 insertions(+), 3 deletions(-)
> create mode 100644 tools/perf/scripts/python/bin/sched-stat-record
> create mode 100644 tools/perf/scripts/python/bin/sched-stat-report
>

Report message to a moderator

[Message index]

		[PATCH 0/7] Profiling sleep times (v3) By: Andrew Vagin on Mon, 28 November 2011 09:03
		[PATCH 3/7] perf: add ability to change event according to sample (v2) By: Andrew Vagin on Mon, 28 November 2011 09:03
		Re: [PATCH 3/7] perf: add ability to change event according to sample (v2) By: Arnaldo Carvalho de M[2] on Tue, 06 December 2011 15:06
		Re: [PATCH 3/7] perf: add ability to change event according to sample (v2) By: Arnaldo Carvalho de M[2] on Tue, 06 December 2011 14:19
		Re: [PATCH 3/7] perf: add ability to change event according to sample (v2) By: Arnaldo Carvalho de M[2] on Tue, 06 December 2011 14:24
		Re: [PATCH 3/7] perf: add ability to change event according to sample (v2) By: David Ahern on Tue, 06 December 2011 14:57
		[PATCH 2/7] perf: add ability to record event period By: Andrew Vagin on Mon, 28 November 2011 09:03
		Re: [PATCH 2/7] perf: add ability to record event period By: Arnaldo Carvalho de M[2] on Tue, 06 December 2011 14:02
		[PATCH 5/7] perf: teach perf inject to merge sched_stat_* and sched_switch events By: Andrew Vagin on Mon, 28 November 2011 09:03
		[PATCH 6/7] perf: add scripts for profiling sleep times (v2) By: Andrew Vagin on Mon, 28 November 2011 09:03
		[PATCH 4/7] perf: teach "perf inject" to work with files By: Andrew Vagin on Mon, 28 November 2011 09:03
		[PATCH 7/7] event: add tracepoint for accounting block time By: Andrew Vagin on Mon, 28 November 2011 09:03
		Re: [PATCH 7/7] event: add tracepoint for accounting block time By: Peter Zijlstra on Mon, 28 November 2011 11:42
		Re: [PATCH 7/7] event: add tracepoint for accounting block time By: Arjan van de Ven on Mon, 28 November 2011 14:02
		Re: [PATCH 7/7] event: add tracepoint for accounting block time By: Arnaldo Carvalho de M[2] on Mon, 28 November 2011 14:31
		Re: [PATCH 7/7] event: add tracepoint for accounting block time By: Arjan van de Ven on Mon, 28 November 2011 14:43
		Re: [PATCH 7/7] event: add tracepoint for accounting block time By: Arnaldo Carvalho de M[2] on Mon, 28 November 2011 14:59
		Re: [PATCH 0/7] Profiling sleep times (v3) By: Andrey Vagin on Tue, 06 December 2011 07:15
		Re: [PATCH 0/7] Profiling sleep times (v3) By: Ingo Molnar on Tue, 06 December 2011 08:30
		Re: [PATCH 0/7] Profiling sleep times (v3) By: Arnaldo Carvalho de M[2] on Tue, 06 December 2011 13:45
		Re: [PATCH 0/7] Profiling sleep times (v3) By: Arun Sharma on Wed, 07 December 2011 20:33

Previous Topic:	[GIT PULL 0/2] perf/urgent fixes
Next Topic:	Re: [GIT PULL 0/2] perf/urgent fixes

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Sat Jan 24 03:38:41 GMT 2026

Total time taken to generate the page: 0.27206 seconds