diff options
author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2010-05-04 14:36:56 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-05-07 05:28:17 -0400 |
commit | 27a9da6538ee18046d7bff8e36a9f783542c54c3 (patch) | |
tree | 9f9944ca8742ec52f7e4a674f7475fc0378abdb8 /kernel | |
parent | 48652ced1533c3372f996a0d83b6e73b1f1c9381 (diff) |
sched: Remove rq argument to the tracepoints
struct rq isn't visible outside of sched.o so its near useless to
expose the pointer, also there are no users of it, so remove it.
Acked-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <1272997616.1642.207.camel@laptop>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/sched.c | 8 | ||||
-rw-r--r-- | kernel/trace/ftrace.c | 3 | ||||
-rw-r--r-- | kernel/trace/trace_sched_switch.c | 5 | ||||
-rw-r--r-- | kernel/trace/trace_sched_wakeup.c | 5 |
4 files changed, 9 insertions, 12 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index 4956ed092838..11ac0eb0bce7 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -2168,7 +2168,7 @@ unsigned long wait_task_inactive(struct task_struct *p, long match_state) | |||
2168 | * just go back and repeat. | 2168 | * just go back and repeat. |
2169 | */ | 2169 | */ |
2170 | rq = task_rq_lock(p, &flags); | 2170 | rq = task_rq_lock(p, &flags); |
2171 | trace_sched_wait_task(rq, p); | 2171 | trace_sched_wait_task(p); |
2172 | running = task_running(rq, p); | 2172 | running = task_running(rq, p); |
2173 | on_rq = p->se.on_rq; | 2173 | on_rq = p->se.on_rq; |
2174 | ncsw = 0; | 2174 | ncsw = 0; |
@@ -2439,7 +2439,7 @@ out_activate: | |||
2439 | success = 1; | 2439 | success = 1; |
2440 | 2440 | ||
2441 | out_running: | 2441 | out_running: |
2442 | trace_sched_wakeup(rq, p, success); | 2442 | trace_sched_wakeup(p, success); |
2443 | check_preempt_curr(rq, p, wake_flags); | 2443 | check_preempt_curr(rq, p, wake_flags); |
2444 | 2444 | ||
2445 | p->state = TASK_RUNNING; | 2445 | p->state = TASK_RUNNING; |
@@ -2613,7 +2613,7 @@ void wake_up_new_task(struct task_struct *p, unsigned long clone_flags) | |||
2613 | 2613 | ||
2614 | rq = task_rq_lock(p, &flags); | 2614 | rq = task_rq_lock(p, &flags); |
2615 | activate_task(rq, p, 0); | 2615 | activate_task(rq, p, 0); |
2616 | trace_sched_wakeup_new(rq, p, 1); | 2616 | trace_sched_wakeup_new(p, 1); |
2617 | check_preempt_curr(rq, p, WF_FORK); | 2617 | check_preempt_curr(rq, p, WF_FORK); |
2618 | #ifdef CONFIG_SMP | 2618 | #ifdef CONFIG_SMP |
2619 | if (p->sched_class->task_woken) | 2619 | if (p->sched_class->task_woken) |
@@ -2833,7 +2833,7 @@ context_switch(struct rq *rq, struct task_struct *prev, | |||
2833 | struct mm_struct *mm, *oldmm; | 2833 | struct mm_struct *mm, *oldmm; |
2834 | 2834 | ||
2835 | prepare_task_switch(rq, prev, next); | 2835 | prepare_task_switch(rq, prev, next); |
2836 | trace_sched_switch(rq, prev, next); | 2836 | trace_sched_switch(prev, next); |
2837 | mm = next->mm; | 2837 | mm = next->mm; |
2838 | oldmm = prev->active_mm; | 2838 | oldmm = prev->active_mm; |
2839 | /* | 2839 | /* |
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 2404b59b3097..aa3a92b511e2 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c | |||
@@ -3212,8 +3212,7 @@ free: | |||
3212 | } | 3212 | } |
3213 | 3213 | ||
3214 | static void | 3214 | static void |
3215 | ftrace_graph_probe_sched_switch(struct rq *__rq, struct task_struct *prev, | 3215 | ftrace_graph_probe_sched_switch(struct task_struct *prev, struct task_struct *next) |
3216 | struct task_struct *next) | ||
3217 | { | 3216 | { |
3218 | unsigned long long timestamp; | 3217 | unsigned long long timestamp; |
3219 | int index; | 3218 | int index; |
diff --git a/kernel/trace/trace_sched_switch.c b/kernel/trace/trace_sched_switch.c index 5fca0f51fde4..a55fccfede5d 100644 --- a/kernel/trace/trace_sched_switch.c +++ b/kernel/trace/trace_sched_switch.c | |||
@@ -50,8 +50,7 @@ tracing_sched_switch_trace(struct trace_array *tr, | |||
50 | } | 50 | } |
51 | 51 | ||
52 | static void | 52 | static void |
53 | probe_sched_switch(struct rq *__rq, struct task_struct *prev, | 53 | probe_sched_switch(struct task_struct *prev, struct task_struct *next) |
54 | struct task_struct *next) | ||
55 | { | 54 | { |
56 | struct trace_array_cpu *data; | 55 | struct trace_array_cpu *data; |
57 | unsigned long flags; | 56 | unsigned long flags; |
@@ -109,7 +108,7 @@ tracing_sched_wakeup_trace(struct trace_array *tr, | |||
109 | } | 108 | } |
110 | 109 | ||
111 | static void | 110 | static void |
112 | probe_sched_wakeup(struct rq *__rq, struct task_struct *wakee, int success) | 111 | probe_sched_wakeup(struct task_struct *wakee, int success) |
113 | { | 112 | { |
114 | struct trace_array_cpu *data; | 113 | struct trace_array_cpu *data; |
115 | unsigned long flags; | 114 | unsigned long flags; |
diff --git a/kernel/trace/trace_sched_wakeup.c b/kernel/trace/trace_sched_wakeup.c index 0271742abb8d..8052446ceeaa 100644 --- a/kernel/trace/trace_sched_wakeup.c +++ b/kernel/trace/trace_sched_wakeup.c | |||
@@ -107,8 +107,7 @@ static void probe_wakeup_migrate_task(struct task_struct *task, int cpu) | |||
107 | } | 107 | } |
108 | 108 | ||
109 | static void notrace | 109 | static void notrace |
110 | probe_wakeup_sched_switch(struct rq *rq, struct task_struct *prev, | 110 | probe_wakeup_sched_switch(struct task_struct *prev, struct task_struct *next) |
111 | struct task_struct *next) | ||
112 | { | 111 | { |
113 | struct trace_array_cpu *data; | 112 | struct trace_array_cpu *data; |
114 | cycle_t T0, T1, delta; | 113 | cycle_t T0, T1, delta; |
@@ -200,7 +199,7 @@ static void wakeup_reset(struct trace_array *tr) | |||
200 | } | 199 | } |
201 | 200 | ||
202 | static void | 201 | static void |
203 | probe_wakeup(struct rq *rq, struct task_struct *p, int success) | 202 | probe_wakeup(struct task_struct *p, int success) |
204 | { | 203 | { |
205 | struct trace_array_cpu *data; | 204 | struct trace_array_cpu *data; |
206 | int cpu = smp_processor_id(); | 205 | int cpu = smp_processor_id(); |