adulau/aha
1
0
Fork 0
mirror of https://github.com/adulau/aha.git synced 2024-12-29 12:16:20 +00:00
Code Issues Projects Releases Packages Wiki Activity

ring-buffer: use internal time stamp function

Browse source 
The ring_buffer_time_stamp that is exported adds a little more overhead
than is needed for using it internally. This patch adds an internal
timestamp function that can be inlined (a single line function)
and used internally for the ring buffer.

[ Impact: a little less overhead to the ring buffer ]

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
This commit is contained in:
Steven Rostedt 2009-05-11 16:28:23 -04:00 committed by Steven Rostedt
parent 0f0c85fc80
commit 88eb012536
1 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
kernel/trace/ring_buffer.c
View file

@ -454,13 +454,18 @@ struct ring_buffer_iter {
/* Up this if you want to test the TIME_EXTENTS and normalization */ /* Up this if you want to test the TIME_EXTENTS and normalization */
#define DEBUG_SHIFT 0 #define DEBUG_SHIFT 0
static inline u64 rb_time_stamp(struct ring_buffer *buffer, int cpu)
{
/* shift to debug/test normalization and TIME_EXTENTS */
return buffer->clock() << DEBUG_SHIFT;
}
u64 ring_buffer_time_stamp(struct ring_buffer *buffer, int cpu) u64 ring_buffer_time_stamp(struct ring_buffer *buffer, int cpu)
{ {
u64 time; u64 time;
preempt_disable_notrace(); preempt_disable_notrace();
/* shift to debug/test normalization and TIME_EXTENTS */ time = rb_time_stamp(buffer, cpu);
time = buffer->clock() << DEBUG_SHIFT;
preempt_enable_no_resched_notrace(); preempt_enable_no_resched_notrace();
return time; return time;
@ -1247,7 +1252,7 @@ rb_move_tail(struct ring_buffer_per_cpu *cpu_buffer,
cpu_buffer->tail_page = next_page; cpu_buffer->tail_page = next_page;
/* reread the time stamp */ /* reread the time stamp */
*ts = ring_buffer_time_stamp(buffer, cpu_buffer->cpu); *ts = rb_time_stamp(buffer, cpu_buffer->cpu);
cpu_buffer->tail_page->page->time_stamp = *ts; cpu_buffer->tail_page->page->time_stamp = *ts;
} }
@ -1413,7 +1418,7 @@ rb_reserve_next_event(struct ring_buffer_per_cpu *cpu_buffer,
if (RB_WARN_ON(cpu_buffer, ++nr_loops > 1000)) if (RB_WARN_ON(cpu_buffer, ++nr_loops > 1000))
return NULL; return NULL;
ts = ring_buffer_time_stamp(cpu_buffer->buffer, cpu_buffer->cpu); ts = rb_time_stamp(cpu_buffer->buffer, cpu_buffer->cpu);
/* /*
* Only the first commit can update the timestamp. * Only the first commit can update the timestamp.