Messages in this thread | | | Date | Mon, 17 Dec 2012 19:50:01 +0900 | From | Hiraku Toyooka <> | Subject | Re: [PATCH v2 -tip 3/4] tracing: make a snapshot feature available from userspace |
| |
(12/15/2012 02:08 AM), Steven Rostedt wrote: > On Fri, 2012-12-07 at 11:07 +0900, Hiraku Toyooka wrote: >> Hi, Steven, >> >> (2012/11/30 23:17), Steven Rostedt wrote: >> [snip] >> > >> > Actually, I would have: >> > >> > status\input | 0 | 1 | else | >> > --------------+------------+------------+------------+ >> > not allocated |(do nothing)| alloc+swap | EINVAL | >> > --------------+------------+------------+------------+ >> > allocated | free | swap | clear | >> > --------------+------------+------------+------------+ >> > >> > Perhaps we don't need to do the clear on swap, just let the trace >> > continue where it left off? But in case we should swap... >> > >> >> I think we don't need the clear on swap too. >> I'll update my patches like this table. >> >> > There's a fast way to clear the tracer. Look at what the wakeup tracer >> > does. We can make that generic. If you want, I can write that code up >> > too. Hmm, maybe I'll do that, as it will speed things up for >> > everyone :-) >> > >> > > BTW, any update on this? I really like to get this into 3.9. >
I'm applying your review comment and rebasing now. So, I'll post the updated patch series in a few days.
Thanks, Hiraku Toyooka
| |