Messages in this thread | | | Subject | Re: [PATCH v7 1/3] trace: Make removal of ring buffer pages atomic | From | Steven Rostedt <> | Date | Wed, 09 May 2012 13:54:18 -0400 |
| |
On Wed, 2012-05-09 at 10:46 -0700, Vaibhav Nagarnaik wrote: > On Wed, May 9, 2012 at 7:29 AM, Steven Rostedt <rostedt@goodmis.org> wrote: > > This is far from a hot path. In fact, it's quite slow. Lets not uglify > > code just to optimize something that's not time critical. > > > > Please combine these two into a single loop. The wait for completion is > > fine as a separate loop. > > > > Sure, I will update the patch with that change. Have you found any > other issues in testing the patch? >
Nope, not yet. Sorry for the spotty checks, I'm doing several other things in between.
-- Steve
| |