On Wed, Mar 9, 2011 at 6:17 AM, Carsten Emde <C.Emde@xxxxxxxxx> wrote: > On 03/05/2011 02:18 AM, Geunsik Lim wrote: >> >> When we try to run ftrace with cyclictest command of rt-test, >> We view the error according to base kernel version. >> We need to modify this hard coded interface. >> >> Signed-off-by: Geunsik Lim<geunsik.lim@xxxxxxxxxxx> >> --- >> Âsrc/cyclictest/cyclictest.c |  Â3 ++- >> Â1 files changed, 2 insertions(+), 1 deletions(-) >> [..] >> -        setkernvar("latency_hist/wakeup_latency/reset", "1"); >> +        setkernvar("latency_hist/wakeup/reset", "1"); > > This is correct. It considers a directory name change made longtime ago. > >> [..] >>        Âsetkernvar("trace_enabled", "1"); >> +        setkernvar("latency_hist/wakeup_latency/reset", "1"); > > This is not correct - probably an unintended insertion of the line that was > removed above. > > Would you mind to resubmit the patch with the first part only? Ok. I will resubmit this patch after modification. Thank you for your review. > >    Â-Carsten. > -- Regards, Geunsik Lim ( Samsung Electronics ) Blog : http://blog.naver.com/invain/ e-Mail: geunsik.lim@xxxxxxxxxxx     Â leemgs@xxxxxxxxx , leemgs1@xxxxxxxxx -- 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/ -- To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html