[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20081004083242.GD27624@elte.hu>
Date: Sat, 4 Oct 2008 10:32:42 +0200
From: Ingo Molnar <mingo@...e.hu>
To: Steven Rostedt <rostedt@...dmis.org>
Cc: Arjan van de Ven <arjan@...radead.org>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ftrace: Add a C-state tracer to help power optimization
* Steven Rostedt <rostedt@...dmis.org> wrote:
>
> On Fri, 3 Oct 2008, Arjan van de Ven wrote:
>
> > On Fri, 3 Oct 2008 20:32:51 -0400 (EDT)
> > Steven Rostedt <rostedt@...dmis.org> wrote:
> >
> > >
> > > Hi Arjan,
> > >
> > > Very nice! I just have a few comments below.
> >
> > since this is my first tracer.... I would suggest having a "golden
> > example" one that people can copy (it seems I copied a less than
> > perfect one ;-)... maybe a trace_example.c ?
>
> Yeah, that would make sense to have. People are starting to come
> up with so many different tracers, I'm not sure what a golden
> example would look like :-/
i think starting with kernel/trace/trace_nop.c is the simplest: copy
that into a new plugin and make it show up in make oldconfig and make it
build. Then add ftrace_printk() lines initially - only specialize the
trace entries to that plugin later on.
Ingo
--
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