On 11/14/2012 03:01 PM, Li, Jin wrote:
>
>
> On 11/14/2012 02:29 PM, Wanlong Gao wrote:
>> On 11/12/2012 03:34 PM, [email protected] wrote:
>>> From: Jin Li<[email protected]>
>>>
>>> Fix test case cpuhotplug03 as follows:
>>> 1. Avoid to check whether the cpu0 is on.
>>> 2. Check the cpu before turn it on
>>>
>>> Signed-off-by: Jin Li<[email protected]>
>>> ---
>>> .../hotplug/cpu_hotplug/functional/cpuhotplug03.sh | 21
>>> ++++++++++++++-------
>>> 1 file changed, 14 insertions(+), 7 deletions(-)
>>>
>>> diff --git
>>> a/testcases/kernel/hotplug/cpu_hotplug/functional/cpuhotplug03.sh
>>> b/testcases/kernel/hotplug/cpu_hotplug/functional/cpuhotplug03.sh
>>> index ca636d1..516a21f 100755
>>> --- a/testcases/kernel/hotplug/cpu_hotplug/functional/cpuhotplug03.sh
>>> +++ b/testcases/kernel/hotplug/cpu_hotplug/functional/cpuhotplug03.sh
>>> @@ -60,15 +60,22 @@ until [ $TST_COUNT -gt $TST_TOTAL ]; do
>>>
>>> # Turns on all CPUs and saves their states
>>> for i in $( get_all_cpus ); do
>>> - if ! online_cpu $1; then
>>> - : $(( cpu += 1 ))
>>> - eval "on_${cpu}=$i"
>>> - fi
>>> + if [ "$i" = "cpu0" ]; then
>>> + continue
>>> + fi
>>> + if ! cpu_is_online $i; then
>>> + if ! online_cpu $i; then
>>> + tst_resm TFAIL "Could not online cpu $i"
>>> + exit_clean 1
>>> + fi
>>> + : $(( cpu += 1 ))
>>> + eval "on_${cpu}=$i"
>>> + fi
>>> : $(( number_of_cpus += 1 ))
>>> done
>>>
>>> if ! offline_cpu ${CPU_TO_TEST} ; then
>>> - tst_resm TBAIL "CPU${CPU_TO_TEST} cannot be offlined"
>>> + tst_resm TFAIL "CPU${CPU_TO_TEST} cannot be offlined"
>>> exit_clean 1
>>> fi
>>>
>>> @@ -90,7 +97,7 @@ until [ $TST_COUNT -gt $TST_TOTAL ]; do
>>> tst_resm TINFO "Onlining CPU ${CPU_TO_TEST}"
>>> online_cpu ${CPU_TO_TEST}
>>> RC=$?
>>> - if [ $RC -eq 0 ]; then
>>> + if [ $RC -ne 0 ]; then
>>> tst_resm TFAIL "CPU${CPU_TO_TEST} cannot be onlined"
>
> Looks strange. It's just what I fixed.
>
> Could you please help to confirm that the 0002-patch is applied correctly?
I tested again and it passed. Maybe caused by some cache.
So, can you resend your 1/3 patch and I can apply all these three.
Thanks,
Wanlong Gao
>
> Thanks,
>
> Jin
>
>>
>> I test with your patch and always failed here,
>> Running tests.......
>> <<<test_start>>>
>> tag=cpuhotplug03 stime=1352874488
>> cmdline="export CPU_TO_TEST=1;
>> $LTPROOT/testcases/bin/cpu_hotplug/functional/cpuhotplug03.sh"
>> contacts=""
>> analysis=exit
>> <<<test_output>>>
>> incrementing stop
>> Name: cpuhotplug03
>> Date: Wed Nov 14 14:28:08 CST 2012
>> Desc: Do tasks get scheduled to a newly on-lined CPU?
>>
>> CPU is 1
>> USER PID %CPU %MEM VSZ RSS TTY STAT START TIME COMMAND
>> root 31631 3.0 0.0 134116 1556 pts/1 R 14:28 0:00 /bin/sh
>> /opt/ltp/testcases/bin/cpu_hotplug/tools/do_spin_loop
>> root 31632 0.0 0.0 134116 1560 pts/1 R 14:28 0:00 /bin/sh
>> /opt/ltp/testcases/bin/cpu_hotplug/tools/do_spin_loop
>> root 31634 0.0 0.0 134116 1560 pts/1 R 14:28 0:00 /bin/sh
>> /opt/ltp/testcases/bin/cpu_hotplug/tools/do_spin_loop
>> root 31635 0.0 0.0 134116 1560 pts/1 R 14:28 0:00 /bin/sh
>> /opt/ltp/testcases/bin/cpu_hotplug/tools/do_spin_loop
>> root 31636 0.0 0.0 134116 1560 pts/1 R 14:28 0:00 /bin/sh
>> /opt/ltp/testcases/bin/cpu_hotplug/tools/do_spin_loop
>> root 31637 0.0 0.0 134116 1560 pts/1 R 14:28 0:00 /bin/sh
>> /opt/ltp/testcases/bin/cpu_hotplug/tools/do_spin_loop
>> root 31638 2.0 0.0 134116 1560 pts/1 R 14:28 0:00 /bin/sh
>> /opt/ltp/testcases/bin/cpu_hotplug/tools/do_spin_loop
>> root 31639 1.0 0.0 134116 1560 pts/1 R 14:28 0:00 /bin/sh
>> /opt/ltp/testcases/bin/cpu_hotplug/tools/do_spin_loop
>> root 31643 0.0 0.0 133488 1304 pts/1 S 14:28 0:00 grep
>> do_spin_loop
>> cpuhotplug03 0 TINFO : Onlining CPU 1
>> cpuhotplug03 1 TFAIL : CPU1 cannot be onlined
>> Wed Nov 14 14:28:08 CST 2012
>> <<<execution_status>>>
>> initiation_status="ok"
>> duration=0 termination_type=exited termination_id=1 corefile=no
>> cutime=37 cstime=4
>> <<<test_end>>>
>> INFO: ltp-pan reported some tests FAIL
>> LTP Version: 20120903-133-gb631790
>>
>> ###############################################################
>>
>> Done executing testcases.
>> LTP Version: 20120903-133-gb631790
>> ###############################################################
>>
>>
>> Thanks,
>> Wanlong Gao
>>
>>> exit_clean 1
>>> fi
>>> @@ -101,7 +108,7 @@ until [ $TST_COUNT -gt $TST_TOTAL ]; do
>>> ps -o psr -o command --no-headers -C do_spin_loop
>>> RC=$?
>>> NUM=`ps -o psr -o command --no-headers -C do_spin_loop | sed -e "s/^
>>> *//" | cut -d' ' -f 1 | grep "^${CPU_TO_TEST}$" | wc -l`
>>> - if [ $RC -eq 0 ]; then
>>> + if [ $RC -ne 0 ]; then
>>> tst_resm TBROK "No do_spin_loop processes found on any processor"
>>> elif [ $NUM -lt 1 ]; then
>>> tst_resm TFAIL "No do_spin_loop processes found on
>>> CPU${CPU_TO_TEST}"
>>>
>>
>
------------------------------------------------------------------------------
Monitor your physical, virtual and cloud infrastructure from a single
web console. Get in-depth insight into apps, servers, databases, vmware,
SAP, cloud infrastructure, etc. Download 30-day Free Trial.
Pricing starts from $795 for 25 servers or applications!
http://p.sf.net/sfu/zoho_dev2dev_nov
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
