diff options
author | Andy Lutomirski <luto@amacapital.net> | 2014-06-04 13:31:15 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2014-06-05 06:09:50 -0400 |
commit | dfc68f29ae67f2a6e799b44e6a4eb3417dffbfcd (patch) | |
tree | 3c91426549e3e5e5c04104f083150e8aa7994cdc | |
parent | 84c407084137d4e491b07ea5ff8665d19106a5ac (diff) |
sched, trace: Add a tracepoint for IPI-less remote wakeups
Remote wakeups of polling CPUs are a valuable performance
improvement; add a tracepoint to make it much easier to verify that
they're working.
Signed-off-by: Andy Lutomirski <luto@amacapital.net>
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Cc: nicolas.pitre@linaro.org
Cc: daniel.lezcano@linaro.org
Cc: umgwanakikbuti@gmail.com
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: linux-kernel@vger.kernel.org
Link: http://lkml.kernel.org/r/16205aee116772aa686814f9b13bccb562108047.1401902905.git.luto@amacapital.net
Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r-- | include/trace/events/sched.h | 20 | ||||
-rw-r--r-- | kernel/sched/core.c | 4 |
2 files changed, 24 insertions, 0 deletions
diff --git a/include/trace/events/sched.h b/include/trace/events/sched.h index 67e1bbf83695..0a68d5ae584e 100644 --- a/include/trace/events/sched.h +++ b/include/trace/events/sched.h | |||
@@ -530,6 +530,26 @@ TRACE_EVENT(sched_swap_numa, | |||
530 | __entry->dst_pid, __entry->dst_tgid, __entry->dst_ngid, | 530 | __entry->dst_pid, __entry->dst_tgid, __entry->dst_ngid, |
531 | __entry->dst_cpu, __entry->dst_nid) | 531 | __entry->dst_cpu, __entry->dst_nid) |
532 | ); | 532 | ); |
533 | |||
534 | /* | ||
535 | * Tracepoint for waking a polling cpu without an IPI. | ||
536 | */ | ||
537 | TRACE_EVENT(sched_wake_idle_without_ipi, | ||
538 | |||
539 | TP_PROTO(int cpu), | ||
540 | |||
541 | TP_ARGS(cpu), | ||
542 | |||
543 | TP_STRUCT__entry( | ||
544 | __field( int, cpu ) | ||
545 | ), | ||
546 | |||
547 | TP_fast_assign( | ||
548 | __entry->cpu = cpu; | ||
549 | ), | ||
550 | |||
551 | TP_printk("cpu=%d", __entry->cpu) | ||
552 | ); | ||
533 | #endif /* _TRACE_SCHED_H */ | 553 | #endif /* _TRACE_SCHED_H */ |
534 | 554 | ||
535 | /* This part must be outside protection */ | 555 | /* This part must be outside protection */ |
diff --git a/kernel/sched/core.c b/kernel/sched/core.c index 5976ca579d3e..e4c0ddd3db8e 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c | |||
@@ -564,6 +564,8 @@ void resched_task(struct task_struct *p) | |||
564 | 564 | ||
565 | if (set_nr_and_not_polling(p)) | 565 | if (set_nr_and_not_polling(p)) |
566 | smp_send_reschedule(cpu); | 566 | smp_send_reschedule(cpu); |
567 | else | ||
568 | trace_sched_wake_idle_without_ipi(cpu); | ||
567 | } | 569 | } |
568 | 570 | ||
569 | void resched_cpu(int cpu) | 571 | void resched_cpu(int cpu) |
@@ -647,6 +649,8 @@ static void wake_up_idle_cpu(int cpu) | |||
647 | smp_mb(); | 649 | smp_mb(); |
648 | if (!tsk_is_polling(rq->idle)) | 650 | if (!tsk_is_polling(rq->idle)) |
649 | smp_send_reschedule(cpu); | 651 | smp_send_reschedule(cpu); |
652 | else | ||
653 | trace_sched_wake_idle_without_ipi(cpu); | ||
650 | } | 654 | } |
651 | 655 | ||
652 | static bool wake_up_full_nohz_cpu(int cpu) | 656 | static bool wake_up_full_nohz_cpu(int cpu) |