Hi! > Signed-off-by: Wanlong Gao <[email protected]> > --- > testcases/kernel/syscalls/getrusage/getrusage03.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/testcases/kernel/syscalls/getrusage/getrusage03.c > b/testcases/kernel/syscalls/getrusage/getrusage03.c > index 202356c..2670316 100644 > --- a/testcases/kernel/syscalls/getrusage/getrusage03.c > +++ b/testcases/kernel/syscalls/getrusage/getrusage03.c > @@ -71,6 +71,13 @@ int main(int argc, char *argv[]) > int lc; > char *msg; > > + /* Disable test if the version of the kernel is less than 2.6.32 */ > + if ((tst_kvercmp(2, 6, 32)) < 0) { > + tst_resm(TWARN, "This ru_maxrss behaviors is not supported"); > + tst_resm(TWARN, "before kernel 2.6.32"); > + exit(0);
Also why exit() and not tst_exit()? And the usuall place where these checks are added is setup() (not that having it before parse_opts is wrong, just unusuall). > + } > + > msg = parse_opts(argc, argv, NULL, NULL); > if (msg != NULL) > tst_brkm(TBROK, NULL, "OPTION PARSING ERROR - %s", msg); > -- > 1.8.3.1.448.gfb7dfaa > -- Cyril Hrubis [email protected] ------------------------------------------------------------------------------ This SF.net email is sponsored by Windows: Build for Windows Store. http://p.sf.net/sfu/windows-dev2dev _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
