diff options
| author | Arjan van de Ven <arjan@infradead.org> | 2009-02-10 11:42:26 -0800 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2009-02-11 10:18:04 +0100 |
| commit | ad0b0fd554dfc126b5750d14908dccc3bbf602be (patch) | |
| tree | 35e178387e36ac62672e2076231a608c571de0c1 /include/linux/latencytop.h | |
| parent | f437e8b53eab92a5829e65781e29aed23d8ffd0c (diff) | |
| download | lwn-ad0b0fd554dfc126b5750d14908dccc3bbf602be.tar.gz lwn-ad0b0fd554dfc126b5750d14908dccc3bbf602be.zip | |
sched, latencytop: incorporate review feedback from Andrew Morton
Andrew had some suggestions for the latencytop file; this patch takes care
of most of these:
* Add documentation
* Turn account_scheduler_latency into an inline function
* Don't report negative values to userspace
* Make the file operations struct const
* Fix a few checkpatch.pl warnings
Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'include/linux/latencytop.h')
| -rw-r--r-- | include/linux/latencytop.h | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/include/linux/latencytop.h b/include/linux/latencytop.h index 901c2d6377a8..b0e99898527c 100644 --- a/include/linux/latencytop.h +++ b/include/linux/latencytop.h @@ -9,6 +9,7 @@ #ifndef _INCLUDE_GUARD_LATENCYTOP_H_ #define _INCLUDE_GUARD_LATENCYTOP_H_ +#include <linux/compiler.h> #ifdef CONFIG_LATENCYTOP #define LT_SAVECOUNT 32 @@ -24,7 +25,14 @@ struct latency_record { struct task_struct; -void account_scheduler_latency(struct task_struct *task, int usecs, int inter); +extern int latencytop_enabled; +void __account_scheduler_latency(struct task_struct *task, int usecs, int inter); +static inline void +account_scheduler_latency(struct task_struct *task, int usecs, int inter) +{ + if (unlikely(latencytop_enabled)) + __account_scheduler_latency(task, usecs, inter); +} void clear_all_latency_tracing(struct task_struct *p); |