Re: [PATCH RESEND] cyclictest: Fixed incorrect wakeup reset interface

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux