diff options
| author | Ingo Molnar <mingo@elte.hu> | 2009-09-07 02:19:51 -0400 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2009-09-07 02:19:51 -0400 |
| commit | a1922ed661ab2c1637d0b10cde933bd9cd33d965 (patch) | |
| tree | 0f1777542b385ebefd30b3586d830fd8ed6fda5b /kernel/trace/trace_stat.c | |
| parent | 75e33751ca8bbb72dd6f1a74d2810ddc8cbe4bdf (diff) | |
| parent | d28daf923ac5e4a0d7cecebae56f3e339189366b (diff) | |
Merge branch 'tracing/core' into tracing/hw-breakpoints
Conflicts:
arch/Kconfig
kernel/trace/trace.h
Merge reason: resolve the conflicts, plus adopt to the new
ring-buffer APIs.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/trace/trace_stat.c')
| -rw-r--r-- | kernel/trace/trace_stat.c | 53 |
1 files changed, 33 insertions, 20 deletions
diff --git a/kernel/trace/trace_stat.c b/kernel/trace/trace_stat.c index c00643733f4c..a4bb239eb987 100644 --- a/kernel/trace/trace_stat.c +++ b/kernel/trace/trace_stat.c | |||
| @@ -49,7 +49,8 @@ static struct dentry *stat_dir; | |||
| 49 | * but it will at least advance closer to the next one | 49 | * but it will at least advance closer to the next one |
| 50 | * to be released. | 50 | * to be released. |
| 51 | */ | 51 | */ |
| 52 | static struct rb_node *release_next(struct rb_node *node) | 52 | static struct rb_node *release_next(struct tracer_stat *ts, |
| 53 | struct rb_node *node) | ||
| 53 | { | 54 | { |
| 54 | struct stat_node *snode; | 55 | struct stat_node *snode; |
| 55 | struct rb_node *parent = rb_parent(node); | 56 | struct rb_node *parent = rb_parent(node); |
| @@ -67,26 +68,35 @@ static struct rb_node *release_next(struct rb_node *node) | |||
| 67 | parent->rb_right = NULL; | 68 | parent->rb_right = NULL; |
| 68 | 69 | ||
| 69 | snode = container_of(node, struct stat_node, node); | 70 | snode = container_of(node, struct stat_node, node); |
| 71 | if (ts->stat_release) | ||
| 72 | ts->stat_release(snode->stat); | ||
| 70 | kfree(snode); | 73 | kfree(snode); |
| 71 | 74 | ||
| 72 | return parent; | 75 | return parent; |
| 73 | } | 76 | } |
| 74 | } | 77 | } |
| 75 | 78 | ||
| 76 | static void reset_stat_session(struct stat_session *session) | 79 | static void __reset_stat_session(struct stat_session *session) |
| 77 | { | 80 | { |
| 78 | struct rb_node *node = session->stat_root.rb_node; | 81 | struct rb_node *node = session->stat_root.rb_node; |
| 79 | 82 | ||
| 80 | while (node) | 83 | while (node) |
| 81 | node = release_next(node); | 84 | node = release_next(session->ts, node); |
| 82 | 85 | ||
| 83 | session->stat_root = RB_ROOT; | 86 | session->stat_root = RB_ROOT; |
| 84 | } | 87 | } |
| 85 | 88 | ||
| 89 | static void reset_stat_session(struct stat_session *session) | ||
| 90 | { | ||
| 91 | mutex_lock(&session->stat_mutex); | ||
| 92 | __reset_stat_session(session); | ||
| 93 | mutex_unlock(&session->stat_mutex); | ||
| 94 | } | ||
| 95 | |||
| 86 | static void destroy_session(struct stat_session *session) | 96 | static void destroy_session(struct stat_session *session) |
| 87 | { | 97 | { |
| 88 | debugfs_remove(session->file); | 98 | debugfs_remove(session->file); |
| 89 | reset_stat_session(session); | 99 | __reset_stat_session(session); |
| 90 | mutex_destroy(&session->stat_mutex); | 100 | mutex_destroy(&session->stat_mutex); |
| 91 | kfree(session); | 101 | kfree(session); |
| 92 | } | 102 | } |
| @@ -150,7 +160,7 @@ static int stat_seq_init(struct stat_session *session) | |||
| 150 | int i; | 160 | int i; |
| 151 | 161 | ||
| 152 | mutex_lock(&session->stat_mutex); | 162 | mutex_lock(&session->stat_mutex); |
| 153 | reset_stat_session(session); | 163 | __reset_stat_session(session); |
| 154 | 164 | ||
| 155 | if (!ts->stat_cmp) | 165 | if (!ts->stat_cmp) |
| 156 | ts->stat_cmp = dummy_cmp; | 166 | ts->stat_cmp = dummy_cmp; |
| @@ -183,7 +193,7 @@ exit: | |||
| 183 | return ret; | 193 | return ret; |
| 184 | 194 | ||
| 185 | exit_free_rbtree: | 195 | exit_free_rbtree: |
| 186 | reset_stat_session(session); | 196 | __reset_stat_session(session); |
| 187 | mutex_unlock(&session->stat_mutex); | 197 | mutex_unlock(&session->stat_mutex); |
| 188 | return ret; | 198 | return ret; |
| 189 | } | 199 | } |
| @@ -193,23 +203,23 @@ static void *stat_seq_start(struct seq_file *s, loff_t *pos) | |||
| 193 | { | 203 | { |
| 194 | struct stat_session *session = s->private; | 204 | struct stat_session *session = s->private; |
| 195 | struct rb_node *node; | 205 | struct rb_node *node; |
| 206 | int n = *pos; | ||
| 196 | int i; | 207 | int i; |
| 197 | 208 | ||
| 198 | /* Prevent from tracer switch or rbtree modification */ | 209 | /* Prevent from tracer switch or rbtree modification */ |
| 199 | mutex_lock(&session->stat_mutex); | 210 | mutex_lock(&session->stat_mutex); |
| 200 | 211 | ||
| 201 | /* If we are in the beginning of the file, print the headers */ | 212 | /* If we are in the beginning of the file, print the headers */ |
| 202 | if (!*pos && session->ts->stat_headers) { | 213 | if (session->ts->stat_headers) { |
| 203 | (*pos)++; | 214 | if (n == 0) |
| 204 | return SEQ_START_TOKEN; | 215 | return SEQ_START_TOKEN; |
| 216 | n--; | ||
| 205 | } | 217 | } |
| 206 | 218 | ||
| 207 | node = rb_first(&session->stat_root); | 219 | node = rb_first(&session->stat_root); |
| 208 | for (i = 0; node && i < *pos; i++) | 220 | for (i = 0; node && i < n; i++) |
| 209 | node = rb_next(node); | 221 | node = rb_next(node); |
| 210 | 222 | ||
| 211 | (*pos)++; | ||
| 212 | |||
| 213 | return node; | 223 | return node; |
| 214 | } | 224 | } |
| 215 | 225 | ||
| @@ -254,16 +264,21 @@ static const struct seq_operations trace_stat_seq_ops = { | |||
| 254 | static int tracing_stat_open(struct inode *inode, struct file *file) | 264 | static int tracing_stat_open(struct inode *inode, struct file *file) |
| 255 | { | 265 | { |
| 256 | int ret; | 266 | int ret; |
| 257 | 267 | struct seq_file *m; | |
| 258 | struct stat_session *session = inode->i_private; | 268 | struct stat_session *session = inode->i_private; |
| 259 | 269 | ||
| 270 | ret = stat_seq_init(session); | ||
| 271 | if (ret) | ||
| 272 | return ret; | ||
| 273 | |||
| 260 | ret = seq_open(file, &trace_stat_seq_ops); | 274 | ret = seq_open(file, &trace_stat_seq_ops); |
| 261 | if (!ret) { | 275 | if (ret) { |
| 262 | struct seq_file *m = file->private_data; | 276 | reset_stat_session(session); |
| 263 | m->private = session; | 277 | return ret; |
| 264 | ret = stat_seq_init(session); | ||
| 265 | } | 278 | } |
| 266 | 279 | ||
| 280 | m = file->private_data; | ||
| 281 | m->private = session; | ||
| 267 | return ret; | 282 | return ret; |
| 268 | } | 283 | } |
| 269 | 284 | ||
| @@ -274,11 +289,9 @@ static int tracing_stat_release(struct inode *i, struct file *f) | |||
| 274 | { | 289 | { |
| 275 | struct stat_session *session = i->i_private; | 290 | struct stat_session *session = i->i_private; |
| 276 | 291 | ||
| 277 | mutex_lock(&session->stat_mutex); | ||
| 278 | reset_stat_session(session); | 292 | reset_stat_session(session); |
| 279 | mutex_unlock(&session->stat_mutex); | ||
| 280 | 293 | ||
| 281 | return 0; | 294 | return seq_release(i, f); |
| 282 | } | 295 | } |
| 283 | 296 | ||
| 284 | static const struct file_operations tracing_stat_fops = { | 297 | static const struct file_operations tracing_stat_fops = { |
