Re: [LTP] [PATCH] Add ftrace-stress-test to LTP

Li Zefan
2010-08-25T02:55:01Z

Garrett Cooper wrote:
> On Tue, Aug 24, 2010 at 6:25 PM, Li Zefan <lizf@cn.fujitsu.com> wrote:
>> Subrata Modak wrote:
>>> Thanks. I need to test this on a suitable kernel. But doubtful if i can
>>> add in the default run.
>>>
>> Why? Because it costs 90 secs? This is configurable, and some other
>> test suites like cpuctl cost more time.
>>
>> If a test suite is not run by default, I guess people won't know its
>> existence, probably.
>
> I think that more of the concern stems from the fact that it's a
> performance test, and a lot of people complain if and when their test
> machines slow down to a halt (take hackbench, proc01, for example).
>

It's not a performance test, but a stress test, which aims at revealing
racy corners in ftrace. Though it will slow down the machine, it won't
halt the machine, unless we hit a serious kernel bug.

> There are issues with the code, but it's dealing with the fact that
> it's not really written in a portable way. Give me a bit of time and
> I'll comment on what needs to be changed.
>

Thanks for reviewing the code. I'm not good at writing protable bash
scripts..