| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1269060320.28658.7.camel@localhost> Date: Sat, 20 Mar 2010 00:45:20 -0400 From: Steven Rostedt <rostedt@...dmis.org> To: Andi Kleen <andi@...stfloor.org> Cc: linux-kernel@...r.kernel.org, robert.richter@....com Subject: Re: [PATCH] oprofile: remove double ring buffering 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@...ux.intel.com> > 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@...dmis.org> -- Steve > > Signed-off-by: Andi Kleen <ak@...ux.intel.com> > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists