Hi!
> +setup()
> +{
> +     tst_require_root
> +
> +     tst_check_cmds df
> +
> +     tst_tmpdir
> +
> +     dd if=/dev/zero of=test.img bs=1M count=20 >/dev/null 2>&1
> +     if [ $? -ne 0 ];then

Unless we set cleanup we need to do tst_rmdir here and below.

> +             tst_brkm TBROK "dd failed."
> +     fi
> +
> +     device_name=$(/sbin/losetup -f)
> +     if [ $? -ne 0 ];then
> +             tst_brkm TBROK "device_name failed."
> +     fi
> +
> +     losetup ${device_name} test.img
> +     if [ $? -ne 0 ];then
> +             tst_brkm TBROK "losetup failed."
> +     fi

Hmm, we may need tst_acquire_device and tst_release_device for shell as
well. Because as this is we cannot make use of the device passed to the
runltp script.

> +     mkfs.ext2 ${device_name} >/dev/null 2>&1
> +     if [ $? -ne 0 ];then

If we end up here because, for example, mkfs.ext2 is not installed, the
device will still be attached. We need either create a cleanup that is
able to handle to cleanup interrupted setup or just detach the device
and tst_rmdir() here and in the steps below.

> +             tst_brkm TBROK "mkfs.ext2 failed."
> +     fi
> +
> +     mkdir mntpoint
> +     if [ $? -ne 0 ];then
> +             tst_brkm TBROK "mkdir testdir failed."
> +     fi

This can be moved before the losetup when the only cleanup we need to do
is tst_rmdir()

> +     mount ${device_name} mntpoint
> +     if [ $? -ne 0 ];then
> +             tst_brkm TBROK "mount failed."
> +     fi

Can we use the ROD and ROD_SILENT here instead? :)

> +}
> +
> +cleanup()
> +{
> +     umount -d mntpoint
> +     if [ $? -ne 0 ];then
> +             tst_resm TINFO "umount error."
> +     fi

You need to detach the loop device here with losetup -d

> +     tst_rmdir
> +}
> +
> +df_test()
> +{
> +     $1 >${TCID}.temp 2>&1
> +     if [ $? -ne 0 ]; then
> +             grep -q -E "unrecognized option | invalid option" ${TCID}.temp
> +             if [ $? -eq 0 ]; then
> +                     tst_resm TCONF "'$1' not supported."
> +                     return
> +             else
> +                     tst_resm TFAIL "'$1' failed."
> +                     return
> +             fi
> +     fi
> +
> +     grep ${device_name} ${TCID}.temp | grep mntpoint | grep -q $2
> +     if [ $? -eq 0 ]; then
> +             tst_resm TPASS "'$1' passed."
> +     else
> +             tst_resm TFAIL "'$1' failed."
> +     fi
> +}
> +
> +test1()
> +{
> +     df_test "df" "19827"
> +}
> +
> +test2()
> +{
> +     df_test "df -a" "19827"
> +}
> +
> +test3()
> +{
> +     df_test "df -h" "20M"
> +}
> +
> +test4()
> +{
> +     df_test "df -H" "21M"
> +}
> +
> +test5()
> +{
> +     df_test "df -i" "5136"
> +}
> +
> +test6()
> +{
> +     df_test "df -k" "19827"
> +}
> +
> +test7()
> +{
> +     df_test "df -m" "20"
> +}
> +
> +test8()
> +{
> +     df_test "df -t ext2" "19827"
> +}
> +
> +test9()
> +{
> +     df_test "df -T" "19827"
> +}
> +
> +test10()
> +{
> +     df_test "df -v mntpoint" "19827"
> +}
> +
> +test11()
> +{
> +     df -x ext2 >${TCID}.temp 2>&1
> +     if [ $? -ne 0 ]; then
> +             grep -q -E "unrecognized option | invalid option" ${TCID}.temp
> +             if [ $? -eq 0 ]; then
> +                     tst_resm TCONF "'df -x ext2' not supported."
> +                     return
> +             else
> +                     tst_resm TFAIL "'df -x ext2' failed."
> +                     return
> +             fi
> +     fi
> +
> +     grep ${device_name} ${TCID}.temp | grep -q mntpoint
> +     if [ $? -ne 0 ]; then
> +             tst_resm TPASS "'df -x ext2' passed."
> +     else
> +             tst_resm TFAIL "'df -x ext2' failed."
> +     fi
> +}
> +
> +test12()
> +{
> +     df --version >${TCID}.temp 2>&1
> +     if [ $? -ne 0 ]; then
> +             grep -q -E "unrecognized option | invalid option" ${TCID}.temp
> +             if [ $? -eq 0 ]; then
> +                     tst_resm TCONF "'df --version' not supported."
> +                     return
> +             else
> +                     tst_resm TFAIL "'df --version' failed."
> +                     return
> +             fi
> +     else
> +             tst_resm TPASS "'df --version' passed."
> +     fi
> +}
> +
> +setup
> +TST_CLEANUP="cleanup"
> +
> +for i in $(seq 1 ${TST_TOTAL})
> +do
> +     test$i
> +done
> +
> +tst_exit

-- 
Cyril Hrubis
chru...@suse.cz

------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to