Amir Goldstein <amir73il@xxxxxxxxx> wrote: > I know Eryu told you that you can change the default of status to success, > ... what's the benefit of doing that. Simple: All the tests should be carried out, even if one or more fail. The script should not simply abort at that point, but should rather give the cumulative error at the end. This means the default should be success and this altered to failure on the failure of any subtest (ie. we're OR'ing them). There should needs to be a way to report any subtest that was skipped because the kernel or the fs doesn't support it. > A simple way to deal with this is to do > > function check_stat () { > $here/src/stat_test $* || exit > } No. This is the wrong thing to do (see above). Better would be to either set status=1 here (but we're inside a function, so I'm not sure that'll work) or to replace the exit above with "|| echo stat_test failed" and let the output comparator handle it. David