[PATCH] trace: create tracers menu - Kernel

This is a discussion on [PATCH] trace: create tracers menu - Kernel ; Subject: trace: create tracers menu From: Peter Zijlstra Date: Tue Oct 21 16:26:36 CEST 2008 We seem to have plenty tracers, lets create a menu and not clutter the already cluttered debug menu more. Signed-off-by: Peter Zijlstra --- kernel/trace/Kconfig | ...

+ Reply to Thread
Results 1 to 6 of 6

Thread: [PATCH] trace: create tracers menu

  1. [PATCH] trace: create tracers menu

    Subject: trace: create tracers menu
    From: Peter Zijlstra
    Date: Tue Oct 21 16:26:36 CEST 2008

    We seem to have plenty tracers, lets create a menu and not clutter
    the already cluttered debug menu more.

    Signed-off-by: Peter Zijlstra
    ---
    kernel/trace/Kconfig | 4 ++++
    1 file changed, 4 insertions(+)

    Index: linux-2.6/kernel/trace/Kconfig
    ================================================== =================
    --- linux-2.6.orig/kernel/trace/Kconfig
    +++ linux-2.6/kernel/trace/Kconfig
    @@ -29,6 +29,8 @@ config TRACING
    select STACKTRACE
    select TRACEPOINTS

    +menu "Tracers"
    +
    config FUNCTION_TRACER
    bool "Kernel Function Tracer"
    depends on HAVE_FUNCTION_TRACER
    @@ -191,3 +193,5 @@ config FTRACE_STARTUP_TEST
    a series of tests are made to verify that the tracer is
    functioning properly. It will do tests on all the configured
    tracers of ftrace.
    +
    +endmenu

    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  2. Re: [PATCH] trace: create tracers menu

    On Tue, 2008-10-21 at 16:37 +0200, Frédéric Weisbecker wrote:
    > 2008/10/21 Frédéric Weisbecker :
    > > 2008/10/21 Peter Zijlstra :
    > >> Subject: trace: create tracers menu
    > >> From: Peter Zijlstra
    > >> Date: Tue Oct 21 16:26:36 CEST 2008
    > >>
    > >> We seem to have plenty tracers, lets create a menu and not clutter
    > >> the already cluttered debug menu more.

    >
    > But...you just included the function tracer inside?


    Problem being? That's a tracer too, right? And it doesn't seem required
    by the other tracers (except the max stack tracer).

    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  3. Re: [PATCH] trace: create tracers menu


    On Tue, 21 Oct 2008, Fr?d?ric Weisbecker wrote:

    > 2008/10/21 Fr?d?ric Weisbecker :
    > > 2008/10/21 Peter Zijlstra :
    > >> Subject: trace: create tracers menu
    > >> From: Peter Zijlstra
    > >> Date: Tue Oct 21 16:26:36 CEST 2008
    > >>
    > >> We seem to have plenty tracers, lets create a menu and not clutter
    > >> the already cluttered debug menu more.

    >
    > But...you just included the function tracer inside?


    Sure, why not? The function tracer is its own tracer itself.

    -- Steve

    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  4. Re: [PATCH] trace: create tracers menu


    On Tue, 21 Oct 2008, Peter Zijlstra wrote:

    > Subject: trace: create tracers menu
    > From: Peter Zijlstra
    > Date: Tue Oct 21 16:26:36 CEST 2008
    >
    > We seem to have plenty tracers, lets create a menu and not clutter
    > the already cluttered debug menu more.
    >
    > Signed-off-by: Peter Zijlstra


    Acked-by: Steven Rostedt

    -- Steve

    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  5. Re: [PATCH] trace: create tracers menu

    2008/10/21 Peter Zijlstra :
    > Problem being? That's a tracer too, right? And it doesn't seem required
    > by the other tracers (except the max stack tracer).



    Sorry I didn't want to write "just" but "only". I misread the patch
    and I believed
    there only was the function tracer inside.
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  6. Re: [PATCH] trace: create tracers menu


    * Steven Rostedt wrote:

    > On Tue, 21 Oct 2008, Peter Zijlstra wrote:
    >
    > > Subject: trace: create tracers menu
    > > From: Peter Zijlstra
    > > Date: Tue Oct 21 16:26:36 CEST 2008
    > >
    > > We seem to have plenty tracers, lets create a menu and not clutter
    > > the already cluttered debug menu more.
    > >
    > > Signed-off-by: Peter Zijlstra

    >
    > Acked-by: Steven Rostedt


    applied to tip/tracing/urgent, thanks!

    Ingo
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

+ Reply to Thread