Dear Thomas & Clark,
This is e-mail about patch to support wakeup & wakeup_rt tracing additionally
at the argument of cyclictest.
Currently, cyclictest v 0.34 support three tracing like PREEMPTOFF ,
IRQSOFF , PREEMPTIRQSOFF.
I want you to append WAKEUP & WAKEUP_RT at the current cyclictest version.
and, cyclictest ver 0.34 have a bug about context switching and event option.
for example,
we can not use "-C" option without --context option.
we can not use "-E" option without --event option.
Belows is result after patching in sequence.
1) adjust "remove-duplicate-h-option.patch" file.
2) and then, adjust
"add-wakeup-wakeuprt-tracing-and-misc-bugfix.patch" file.
Hope this helps. If I mistake, correct me.
Thanks.
After Patching ....
#### [ wakeup test]###############################################
[root@fedora9 rt-tests-temp0]# cat /debug/tracing/available_tracers
syscall blk kmemtrace power branch function_graph mmiotrace wakeup_rt
wakeup preemptirqsoff preemptoff irqsoff function sched_switch
initcall nop
[root@fedora9 rt-tests-temp0]# ./cyclictest -t 2 -b 200 -W -D 5
could not set latency_hist/wakeup_latency/reset to 1
0.16 0.27 0.21 2/308 2521
T: 0 ( 2520) P: 0 I:1000 C: 4288 Min: 16 Act: 27 Avg: 28 Max: 152
T: 1 ( 2521) P: 0 I:1500 C: 2859 Min: 19 Act: 29 Avg: 27 Max: 72
[root@fedora9 rt-tests-temp0]# cat /debug/tracing/trace
# tracer: wakeup
#
# wakeup latency trace v1.1.5 on 2.6.29-rt2-ftrace
# --------------------------------------------------------------------
# latency: 20 us, #3/3, CPU#2 | (M:preempt VP:0, KP:0, SP:0 HP:0 #P:4)
# -----------------
# | task: gnome-terminal-3884 (uid:504 nice:0 policy:0 rt_prio:0)
# -----------------
#
# _------=> CPU#
# / _-----=> irqs-off
# | / _----=> need-resched
# || / _---=> hardirq/softirq
# ||| / _--=> preempt-depth
# |||| /
# ||||| delay
# cmd pid ||||| time | caller
# \ / ||||| \ | /
gnome-te-3884 2d.h. 1us+: irq_enter <-smp_apic_timer_interrupt
gnome-te-3884 2dN.1 20us : irq_exit <-smp_apic_timer_interrupt
gnome-te-3884 2dN.1 21us : trace_preempt_on <-smp_apic_timer_interrupt
#####[ Context Switching Test]#####################################
[root@fedora9 rt-tests-temp0]# ./cyclictest -t 10 -b 100000 -C -m -D 5
could not set latency_hist/wakeup_latency/reset to 1
0.17 0.17 0.17 1/326 2706
T: 0 ( 2697) P: 0 I:1000 C: 4000 Min: 12 Act: 21 Avg: 20 Max: 107
T: 1 ( 2698) P: 0 I:1500 C: 2667 Min: 13 Act: 27 Avg: 22 Max: 138
T: 2 ( 2699) P: 0 I:2000 C: 2000 Min: 12 Act: 13 Avg: 13 Max: 40
T: 3 ( 2700) P: 0 I:2500 C: 1600 Min: 12 Act: 16 Avg: 13 Max: 41
T: 4 ( 2701) P: 0 I:3000 C: 1334 Min: 12 Act: 15 Avg: 13 Max: 39
T: 5 ( 2702) P: 0 I:3500 C: 1143 Min: 12 Act: 28 Avg: 16 Max: 61
T: 6 ( 2703) P: 0 I:4000 C: 1000 Min: 12 Act: 13 Avg: 13 Max: 102
T: 7 ( 2704) P: 0 I:4500 C: 889 Min: 12 Act: 13 Avg: 15 Max: 184
T: 8 ( 2705) P: 0 I:5000 C: 800 Min: 12 Act: 13 Avg: 13 Max: 32
T: 9 ( 2706) P: 0 I:5500 C: 728 Min: 12 Act: 15 Avg: 15 Max: 54
[root@fedora9 rt-tests-temp0]# cat /debug/tracing/trace |head
# tracer: sched_switch
#
# TASK-PID CPU# TIMESTAMP FUNCTION
# | | | | |
##### CPU 0 buffer started ####
<idle>-0 [000] 1107.917632: 0:140:R + [000]
10: 49:D sirq-sched/0
<idle>-0 [000] 1107.917635: 0:140:R + [000]
5: 49:D sirq-timer/0
<idle>-0 [000] 1107.917642: 0:140:R ==> [000]
10: 49:R sirq-sched/0
sirq-sched/0-10 [000] 1107.917655: 10: 49:D ==> [000]
5: 49:R sirq-timer/0
sirq-timer/0-5 [000] 1107.917660: 5: 49:D ==> [000]
0:140:R <idle>
[root@fedora9 rt-tests-temp0]#
#################################################################################
2009/4/6 GeunSik Lim <leemgs1@xxxxxxxxx>:
> Dear Clark,
>
> This is trivial contents about option of cyclictest.
>
> Cyclictest utiltiy (ver 0.34) display the information with duplicate
> "-h" options,
> when a user run "cyclictest --help" command.
> User will confuse exact understanding because of duplicate
> description of -h option.
> How about you remove "-h H_MAX *****" line?
> Thanks reading.
>
> Signed-off-by: GeunSik Lim <leemgs1@xxxxxxxxx>
>
> diff --git a/src/cyclictest/cyclictest.c b/src/cyclictest/cyclictest.c
> index f878217..cedc36f 100644
> --- a/src/cyclictest/cyclictest.c
> +++ b/src/cyclictest/cyclictest.c
> @@ -688,7 +688,8 @@ static void display_help(void)
> "-d DIST --distance=DIST distance of thread
> intervals in us default=500\n"
> "-E --event event tracing (used with -b)\n"
> "-f --ftrace function trace (when -b is active)\n"
> - "-h H_MAX latency histogram size in
> us default 0 (off)\n"
> + "-h --histogram=US dump a latency histogram to
> stdout after the run\n"
> + " US is the max time to be be
> tracked in microseconds\n"
> "-i INTV --interval=INTV base interval of thread in
> us default=1000\n"
> "-I --irqsoff Irqsoff tracing (used with -b)\n"
> "-l LOOPS --loops=LOOPS number of loops:
> default=0(endless)\n"
> @@ -710,8 +711,6 @@ static void display_help(void)
> "-D --duration=t specify a length for the test run\n"
> " default is in seconds, but
> 'm', 'h', or 'd' maybe added\n"
> " to modify value to minutes,
> hours or days\n"
> - "-h --histogram=US dump a latency histogram to
> stdout after the run\n"
> - " US is the max time to be be
> tracked in microseconds\n"
> );
> exit(0);
> }
>
>
> --
> Regards,
> GeunSik Lim
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
--
Regards,
GeunSik Lim
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Attachment:
add-wakeup-wakeuprt-tracing-and-misc-bugfix.patch
Description: Binary data
