Garrett Cooper wrote: > On Wed, Apr 6, 2011 at 10:24 PM, Gui Jianfeng > <[email protected]> wrote: >> Hi Cyril >> >> Any objection of this fix, if no, would you push it? >> >> 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 > > Tried this out in my Fedora 13 VM and it looked ok. Would be nice if > these options were properly documented in mount(8).
Thanks for testing. It's a good idea to add "cgroup" annotation when introducing the "-t" option. But "-o" option is too specific for each file-system. So for more details about "-o", we'd better the check out the document of individual system. Thanks, Gui > -Garrett > ------------------------------------------------------------------------------ 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
