diff options
author | Ingo Molnar <mingo@elte.hu> | 2010-05-28 04:24:03 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-05-29 05:30:30 -0400 |
commit | 92af1abde4f2d74c9e5f7b670a1971779ed0b051 (patch) | |
tree | 2a927306d50da15f241ec68cdbab6d811ea9c29c | |
parent | c4fe52a8ee730ed340eba8fe6ccbf26347ebe9aa (diff) | |
parent | 2711ca237a084286ea1c2dcf82ab2aadab23a00d (diff) |
Merge branch 'tip/perf/core' of git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-2.6-trace into perf/urgent
-rw-r--r-- | kernel/trace/ring_buffer.c | 19 | ||||
-rw-r--r-- | kernel/trace/trace.c | 6 |
2 files changed, 17 insertions, 8 deletions
diff --git a/kernel/trace/ring_buffer.c b/kernel/trace/ring_buffer.c index 7f6059c5aa94..1da7b6ea8b85 100644 --- a/kernel/trace/ring_buffer.c +++ b/kernel/trace/ring_buffer.c | |||
@@ -1768,6 +1768,14 @@ rb_reset_tail(struct ring_buffer_per_cpu *cpu_buffer, | |||
1768 | * must fill the old tail_page with padding. | 1768 | * must fill the old tail_page with padding. |
1769 | */ | 1769 | */ |
1770 | if (tail >= BUF_PAGE_SIZE) { | 1770 | if (tail >= BUF_PAGE_SIZE) { |
1771 | /* | ||
1772 | * If the page was filled, then we still need | ||
1773 | * to update the real_end. Reset it to zero | ||
1774 | * and the reader will ignore it. | ||
1775 | */ | ||
1776 | if (tail == BUF_PAGE_SIZE) | ||
1777 | tail_page->real_end = 0; | ||
1778 | |||
1771 | local_sub(length, &tail_page->write); | 1779 | local_sub(length, &tail_page->write); |
1772 | return; | 1780 | return; |
1773 | } | 1781 | } |
@@ -3894,12 +3902,12 @@ int ring_buffer_read_page(struct ring_buffer *buffer, | |||
3894 | ret = read; | 3902 | ret = read; |
3895 | 3903 | ||
3896 | cpu_buffer->lost_events = 0; | 3904 | cpu_buffer->lost_events = 0; |
3905 | |||
3906 | commit = local_read(&bpage->commit); | ||
3897 | /* | 3907 | /* |
3898 | * Set a flag in the commit field if we lost events | 3908 | * Set a flag in the commit field if we lost events |
3899 | */ | 3909 | */ |
3900 | if (missed_events) { | 3910 | if (missed_events) { |
3901 | commit = local_read(&bpage->commit); | ||
3902 | |||
3903 | /* If there is room at the end of the page to save the | 3911 | /* If there is room at the end of the page to save the |
3904 | * missed events, then record it there. | 3912 | * missed events, then record it there. |
3905 | */ | 3913 | */ |
@@ -3907,10 +3915,17 @@ int ring_buffer_read_page(struct ring_buffer *buffer, | |||
3907 | memcpy(&bpage->data[commit], &missed_events, | 3915 | memcpy(&bpage->data[commit], &missed_events, |
3908 | sizeof(missed_events)); | 3916 | sizeof(missed_events)); |
3909 | local_add(RB_MISSED_STORED, &bpage->commit); | 3917 | local_add(RB_MISSED_STORED, &bpage->commit); |
3918 | commit += sizeof(missed_events); | ||
3910 | } | 3919 | } |
3911 | local_add(RB_MISSED_EVENTS, &bpage->commit); | 3920 | local_add(RB_MISSED_EVENTS, &bpage->commit); |
3912 | } | 3921 | } |
3913 | 3922 | ||
3923 | /* | ||
3924 | * This page may be off to user land. Zero it out here. | ||
3925 | */ | ||
3926 | if (commit < BUF_PAGE_SIZE) | ||
3927 | memset(&bpage->data[commit], 0, BUF_PAGE_SIZE - commit); | ||
3928 | |||
3914 | out_unlock: | 3929 | out_unlock: |
3915 | spin_unlock_irqrestore(&cpu_buffer->reader_lock, flags); | 3930 | spin_unlock_irqrestore(&cpu_buffer->reader_lock, flags); |
3916 | 3931 | ||
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index ba0ec81158b2..95d0b1a28f93 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c | |||
@@ -3661,7 +3661,6 @@ tracing_buffers_read(struct file *filp, char __user *ubuf, | |||
3661 | size_t count, loff_t *ppos) | 3661 | size_t count, loff_t *ppos) |
3662 | { | 3662 | { |
3663 | struct ftrace_buffer_info *info = filp->private_data; | 3663 | struct ftrace_buffer_info *info = filp->private_data; |
3664 | unsigned int pos; | ||
3665 | ssize_t ret; | 3664 | ssize_t ret; |
3666 | size_t size; | 3665 | size_t size; |
3667 | 3666 | ||
@@ -3688,11 +3687,6 @@ tracing_buffers_read(struct file *filp, char __user *ubuf, | |||
3688 | if (ret < 0) | 3687 | if (ret < 0) |
3689 | return 0; | 3688 | return 0; |
3690 | 3689 | ||
3691 | pos = ring_buffer_page_len(info->spare); | ||
3692 | |||
3693 | if (pos < PAGE_SIZE) | ||
3694 | memset(info->spare + pos, 0, PAGE_SIZE - pos); | ||
3695 | |||
3696 | read: | 3690 | read: |
3697 | size = PAGE_SIZE - info->read; | 3691 | size = PAGE_SIZE - info->read; |
3698 | if (size > count) | 3692 | if (size > count) |