Re: [PATCH] oprofile: remove double ring buffering

[Steven Rostedt]

On Sat, 2010-03-20 at 01:03 +0100, Andi Kleen wrote:
> [This is currently being tested if it fixes the excessive samples
> dropping I have been complaining about for some time. Still wanted
> to post the patch for review.]
> 
> commit 49e66e8ab54f1114237e6dc24dc32ac912870c41
> Author: Andi Kleen <ak@xxxxxxxxxxxxxxx>
> Date:   Fri Mar 19 21:03:48 2010 +0100
> 
>     oprofile: remove double ring buffering
>     
>     oprofile used a double buffer scheme for its cpu event buffer
>     to avoid races on reading with the old lock less ring buffer.

The old ring buffer was not lockless, hence the need for two ring
buffers because of NMIs. The new ring buffer is lockless.

>     
>     But that is obsolete now with the new ring buffer, so simply
>     use a single buffer. This greatly simplifies the code and avoids
>     some situations where samples could be dropped.
>     
>     Based on suggestions from Steven Rostedt

Acked-by: Steven Rostedt <rostedt@xxxxxxxxxxx>

-- Steve

>     
>     Signed-off-by: Andi Kleen <ak@xxxxxxxxxxxxxxx>
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/