Cyril, Garrett

Any objection?

Gui

Gui Jianfeng wrote:
> Hi Cyril,
> 
> Would you pick up this patch?
> 
> Thanks,
> Gui
> 
> Gui Jianfeng wrote:
>> Some testcases are not subsystem specific, So there's no need
>> to mount all subsystems at "cgroup/". Furthre more, if some
>> subsystem is already mounted, it would fail if we're trying
>> to mount all subsystems at a mountpoint.
>> This patch creates a hierarchy with no bound subsystems.
>>
>> Signed-off-by: Gui Jianfeng <[email protected]>
>> ---
>> diff --git a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh 
>> b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
>> index 5527afc..878f91a 100755
>> --- a/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
>> +++ b/testcases/kernel/controllers/cgroup/cgroup_regression_test.sh
>> @@ -97,7 +97,7 @@ test_1()
>>      ./fork_processes &
>>      sleep 1
>>  
>> -    mount -t cgroup xxx cgroup/
>> +    mount -t cgroup -o none,name=foo cgroup cgroup/
>>      if [ $? -ne 0 ]; then
>>              tst_resm TFAIL "failed to mount cgroup filesystem"
>>              failed=1
>> @@ -124,7 +124,7 @@ test_1()
>>  #---------------------------------------------------------------------------
>>  test_2()
>>  {
>> -    mount -t cgroup xxx cgroup/
>> +    mount -t cgroup -o none,name=foo cgroup cgroup/
>>      if [ $? -ne 0 ]; then
>>              tst_resm TFAIL "Failed to mount cgroup filesystem"
>>              failed=1
>> @@ -220,7 +220,7 @@ test_4()
>>              return
>>      fi
>>  
>> -    mount -t cgroup xxx cgroup/
>> +    mount -t cgroup -o none,name=foo cgroup cgroup/
>>      mkdir cgroup/0
>>      rmdir cgroup/0
>>      umount cgroup/
>> @@ -360,7 +360,7 @@ test_7_1()
>>  
>>  test_7_2()
>>  {
>> -    mount -t cgroup xxx cgroup/
>> +    mount -t cgroup -o none,name=foo cgroup cgroup/
>>      if [ $? -ne 0 ]; then
>>              tst_resm TFAIL "failed to mount $subsys"
>>              failed=1
>> @@ -429,7 +429,7 @@ test_7()
>>  #---------------------------------------------------------------------------
>>  test_8()
>>  {
>> -    mount -t cgroup xxx cgroup/
>> +    mount -t cgroup -o none,name=foo cgroup cgroup/
>>      if [ $? -ne 0 ]; then
>>              tst_resm TFAIL "failed to mount cgroup filesystem"
>>              failed=1
>>
>> ------------------------------------------------------------------------------
>> Xperia(TM) PLAY
>> It's a major breakthrough. An authentic gaming
>> smartphone on the nation's most reliable network.
>> And it wants your games.
>> http://p.sf.net/sfu/verizon-sfdev
>> _______________________________________________
>> Ltp-list mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/ltp-list
>>
> 

-- 
Regards
Gui Jianfeng

------------------------------------------------------------------------------
What Every C/C++ and Fortran developer Should Know!
Read this article and learn how Intel has extended the reach of its 
next-generation tools to help Windows* and Linux* C/C++ and Fortran 
developers boost performance applications - including clusters. 
http://p.sf.net/sfu/intel-dev2devmay
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to