Hi,
We have found out a cgroup test case issue with
'ltp-full-20100331/testcases/kernel/controllers/memcg/stress/memcg_stress_test.sh'.
The sniff of test case function is as below:-
Note: - Below code sniff is taken from latest LTP August month
release.
run_stress()
{
do_mount;
for i in $(seq 0 $(($1-1)))
do
mkdir /dev/memcg/$i 2> /dev/null
./memcg_process_stress $2 $3 &
eval pid$i=$!
eval echo \$pid$i > /dev/memcg/$i/tasks
done
for i in $(seq 0 $(($1-1)))
do
eval /bin/kill -s SIGUSR1 \$pid$i 2> /dev/null
done
sleep $4
for i in $(seq 0 $(($1-1)))
do
eval
/bin/kill -s SIGINT \$pid$i 2> /dev/null <----send
SIGINT signal for a pid
eval wait \$pid$i
rmdir /dev/memcg/$i 2> /dev/null
done
cleanup;
}
Basically, the test case is not doing a proper cleanup using
SIGINT.It seems that the current test runs for much longer time than
expected, thus affecting test performance. During this course of
time, other dependent tests were also not proceeding further and
system seemed to be in a process hang state. However we have found
that SIGKILL does a better cleanup job compared to SIGINT, to
overcome this situation.
So, I have created a patch to fix above test case issue. Kindly
review this patch and acknowledge. I think this patch can be merged
into latest LTP code to avoid this problem.
Please let me know if you need any clarifications.
Regards,
Shubham