[patch 0/3] timer_list: make debug output more useful - Kernel

This is a discussion on [patch 0/3] timer_list: make debug output more useful - Kernel ; /proc/timer_list resp. sysrq-Q is an important debug helper to analyse timer related problems. Make it more useful. Thanks, tglx -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo ...

+ Reply to Thread
Results 1 to 3 of 3

Thread: [patch 0/3] timer_list: make debug output more useful

  1. [patch 0/3] timer_list: make debug output more useful

    /proc/timer_list resp. sysrq-Q is an important debug helper to analyse
    timer related problems. Make it more useful.

    Thanks,

    tglx



    --
    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. [patch 3/3] timer_list: add base address to clock base

    The base address of a (per cpu) clock base is a useful debug info.
    Add it and bump the version number of timer_lists.

    Signed-off-by: Thomas Gleixner
    ---
    kernel/time/timer_list.c | 3 ++-
    1 file changed, 2 insertions(+), 1 deletion(-)

    Index: linux-2.6-tip/kernel/time/timer_list.c
    ================================================== =================
    --- linux-2.6-tip.orig/kernel/time/timer_list.c
    +++ linux-2.6-tip/kernel/time/timer_list.c
    @@ -110,6 +110,7 @@ next_one:
    static void
    print_base(struct seq_file *m, struct hrtimer_clock_base *base, u64 now)
    {
    + SEQ_printf(m, " .base: %p\n", base);
    SEQ_printf(m, " .index: %d\n",
    base->index);
    SEQ_printf(m, " .resolution: %Lu nsecs\n",
    @@ -249,7 +250,7 @@ static int timer_list_show(struct seq_fi
    u64 now = ktime_to_ns(ktime_get());
    int cpu;

    - SEQ_printf(m, "Timer List Version: v0.3\n");
    + SEQ_printf(m, "Timer List Version: v0.4\n");
    SEQ_printf(m, "HRTIMER_MAX_CLOCK_BASES: %d\n", HRTIMER_MAX_CLOCK_BASES);
    SEQ_printf(m, "now at %Ld nsecs\n", (unsigned long long)now);



    --
    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. [patch 2/3] timer_list: print cpu number of clockevents device

    The per cpu clock events device output of timer_list lacks an
    association of the device to the cpu which is annoying when looking at
    the output of /proc/timer_list from a 128 way system.

    Add the CPU number info and mark the broadcast device in the device
    list printout.

    Signed-off-by: Thomas Gleixner
    ---
    kernel/time/timer_list.c | 10 +++++++---
    1 file changed, 7 insertions(+), 3 deletions(-)

    Index: linux-2.6-tip/kernel/time/timer_list.c
    ================================================== =================
    --- linux-2.6-tip.orig/kernel/time/timer_list.c
    +++ linux-2.6-tip/kernel/time/timer_list.c
    @@ -184,12 +184,16 @@ static void print_cpu(struct seq_file *m

    #ifdef CONFIG_GENERIC_CLOCKEVENTS
    static void
    -print_tickdevice(struct seq_file *m, struct tick_device *td)
    +print_tickdevice(struct seq_file *m, struct tick_device *td, int cpu)
    {
    struct clock_event_device *dev = td->evtdev;

    SEQ_printf(m, "\n");
    SEQ_printf(m, "Tick Device: mode: %d\n", td->mode);
    + if (cpu < 0)
    + SEQ_printf(m, "Broadcast device\n");
    + else
    + SEQ_printf(m, "Per CPU device: %d\n", cpu);

    SEQ_printf(m, "Clock Event Device: ");
    if (!dev) {
    @@ -223,7 +227,7 @@ static void timer_list_show_tickdevices(
    int cpu;

    #ifdef CONFIG_GENERIC_CLOCKEVENTS_BROADCAST
    - print_tickdevice(m, tick_get_broadcast_device());
    + print_tickdevice(m, tick_get_broadcast_device(), -1);
    SEQ_printf(m, "tick_broadcast_mask: %08lx\n",
    tick_get_broadcast_mask()->bits[0]);
    #ifdef CONFIG_TICK_ONESHOT
    @@ -233,7 +237,7 @@ static void timer_list_show_tickdevices(
    SEQ_printf(m, "\n");
    #endif
    for_each_online_cpu(cpu)
    - print_tickdevice(m, tick_get_device(cpu));
    + print_tickdevice(m, tick_get_device(cpu), cpu);
    SEQ_printf(m, "\n");
    }
    #else


    --
    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