diff options
| author | Witold Baryluk <baryluk@smp.if.uj.edu.pl> | 2009-03-17 16:15:44 -0400 |
|---|---|---|
| committer | Steven Rostedt <srostedt@redhat.com> | 2009-03-17 23:10:43 -0400 |
| commit | 97e7e4f391cac2b00417b581b432533d245d4fd0 (patch) | |
| tree | d51d4776962f5838f4cb4cc804d2c94db185527d /include/linux | |
| parent | af4617bdba34aa556272b34c3986b0a4d588f568 (diff) | |
tracing: optimization of branch tracer
Impact: better performance for if branch tracer
Use an array to count the hit and misses of a conditional instead
of using another conditional. This cuts down on saturation of branch
predictions and increases performance of modern pipelined architectures.
Signed-off-by: Witold Baryluk <baryluk@smp.if.uj.edu.pl>
Signed-off-by: Steven Rostedt <srostedt@redhat.com>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/compiler.h | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/include/linux/compiler.h b/include/linux/compiler.h index d95da1020f1c..6faa7e549de4 100644 --- a/include/linux/compiler.h +++ b/include/linux/compiler.h | |||
| @@ -68,6 +68,7 @@ struct ftrace_branch_data { | |||
| 68 | unsigned long miss; | 68 | unsigned long miss; |
| 69 | unsigned long hit; | 69 | unsigned long hit; |
| 70 | }; | 70 | }; |
| 71 | unsigned long miss_hit[2]; | ||
| 71 | }; | 72 | }; |
| 72 | }; | 73 | }; |
| 73 | 74 | ||
| @@ -125,10 +126,7 @@ void ftrace_likely_update(struct ftrace_branch_data *f, int val, int expect); | |||
| 125 | .line = __LINE__, \ | 126 | .line = __LINE__, \ |
| 126 | }; \ | 127 | }; \ |
| 127 | ______r = !!(cond); \ | 128 | ______r = !!(cond); \ |
| 128 | if (______r) \ | 129 | ______f.miss_hit[______r]++; \ |
| 129 | ______f.hit++; \ | ||
| 130 | else \ | ||
| 131 | ______f.miss++; \ | ||
| 132 | ______r; \ | 130 | ______r; \ |
| 133 | })) | 131 | })) |
| 134 | #endif /* CONFIG_PROFILE_ALL_BRANCHES */ | 132 | #endif /* CONFIG_PROFILE_ALL_BRANCHES */ |
