Messages in this thread |  | | | Date | Sat, 17 Jan 2009 20:14:30 +0100 | | From | Jens Axboe <> | | Subject | Re: [RFC][PATCH] ftrace interface for blktrace |
| |
On Sat, Jan 17 2009, Ingo Molnar wrote: > > * Arnaldo Carvalho de Melo <acme@redhat.com> wrote: > > > Hi Jens, > > > > The patch below adds a ftrace interface for blktrace, allowing > > people to use it without any required userspace tools, using sysfs to > > setup the act_mask, pid, start_lba, end_lba. > > Very nice patch! > > Jens, i'm wondering what's your take on this direction is. I think the > consolidation effect is great and the built-in IO tracing capabilities are > very nice.
I like the current patch, from a quick look. I'll look more soonish (monday). I've always liked the concept of being able to mix various traces into the same stream, since it makes it MUCH easier to see what on earth is going wrong. What I didn't like in the patches Acme did was the enable part, I'll need to look into that. Again, from a quick look, what is the BKL doing in there?!
> We can still get the raw events too and do user-space post-processing, > when that is desired - so this does not limit anything that blktrace was > able to do before - it only extends on it. > > Is there any particular blktrace feature you can think of that is not > present in the blktrace ftrace plugin?
Can't answer that yet, I'll give it a more thorough look next week.
-- Jens Axboe
|  |