[PATCH 07/12] perf, sched migration: Parameterize cpu height and spacing

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Frederic Weisbecker
Date: Sunday, August 1, 2010 - 5:13 pm

From: Nikhil Rao <ncrao@google.com>

Without vertical zoom, it is not possible to see all CPUs in a trace
taken on a larger machine. This patch parameterizes the height and
spacing of CPUs so that you can fit more cpus into the screen.

Ideally we should dynamically size/space the CPU rectangles with some
minimum threshold. Until then, this patch is a stop-gap.

Signed-off-by: Nikhil Rao <ncrao@google.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Tom Zanussi <tzanussi@gmail.com>
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
---
 tools/perf/scripts/python/sched-migration.py |   23 ++++++++++++++---------
 1 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/tools/perf/scripts/python/sched-migration.py b/tools/perf/scripts/python/sched-migration.py
index 8b8fb7c..d902668 100644
--- a/tools/perf/scripts/python/sched-migration.py
+++ b/tools/perf/scripts/python/sched-migration.py
@@ -27,6 +27,11 @@ from perf_trace_context import *
 from Core import *
 
 class RootFrame(wx.Frame):
+	Y_OFFSET = 100
+	CPU_HEIGHT = 100
+	CPU_SPACE = 50
+	EVENT_MARKING_WIDTH = 5
+
 	def __init__(self, timeslices, parent = None, id = -1, title = "Migration"):
 		wx.Frame.__init__(self, parent, id, title)
 
@@ -97,8 +102,8 @@ class RootFrame(wx.Frame):
 		if width_px == 0:
 			return
 
-		offset_py = 100 + (cpu * 150)
-		width_py = 100
+		offset_py = RootFrame.Y_OFFSET + (cpu * (RootFrame.CPU_HEIGHT + RootFrame.CPU_SPACE))
+		width_py = RootFrame.CPU_HEIGHT
 
 		if cpu in slice.event_cpus:
 			rgb = rq.event.color()
@@ -107,9 +112,9 @@ class RootFrame(wx.Frame):
 				color = wx.Colour(r, g, b)
 				brush = wx.Brush(color, wx.SOLID)
 				dc.SetBrush(brush)
-				dc.DrawRectangle(offset_px, offset_py, width_px, 5)
-				width_py -= 5
-				offset_py += 5
+				dc.DrawRectangle(offset_px, offset_py, width_px, RootFrame.EVENT_MARKING_WIDTH)
+				width_py -= RootFrame.EVENT_MARKING_WIDTH
+				offset_py += RootFrame.EVENT_MARKING_WIDTH
 
 		red_power = int(0xff - (0xff * load_rate))
 		color = wx.Colour(0xff, red_power, red_power)
@@ -154,11 +159,11 @@ class RootFrame(wx.Frame):
 		self.update_rectangles(dc, start, end)
 
 	def cpu_from_ypixel(self, y):
-		y -= 100
-		cpu = y / 150
-		height = y % 150
+		y -= RootFrame.Y_OFFSET
+		cpu = y / (RootFrame.CPU_HEIGHT + RootFrame.CPU_SPACE)
+		height = y % (RootFrame.CPU_HEIGHT + RootFrame.CPU_SPACE)
 
-		if cpu < 0 or cpu > self.max_cpu or height > 100:
+		if cpu < 0 or cpu > self.max_cpu or height > RootFrame.CPU_HEIGHT:
 			return -1
 
 		return cpu
-- 
1.6.2.3

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[GIT PULL] instrumentation updates, Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 02/12] tracing: Drop cpparg() macro, Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 03/12] perf: New migration tool overview, Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 04/12] perf, sched migration: Handle ignored migrat ..., Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 05/12] perf, sched migration: Ignore unhandled task ..., Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 06/12] perf, sched migration: Fix key bindings, Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 07/12] perf, sched migration: Parameterize cpu heig ..., Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 08/12] perf, sched migration: Make it vertically sc ..., Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 09/12] perf, sched migration: Make the GUI class cl ..., Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 10/12] perf, sched migration: Librarize the GUI class, Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 11/12] perf, sched migration: Librarize task states ..., Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
[PATCH 12/12] x86,mmiotrace: Add support for tracing STOS ..., Frederic Weisbecker, (Sun Aug 1, 5:13 pm)
Re: [GIT PULL] instrumentation updates, Ingo Molnar, (Sun Aug 1, 11:28 pm)