On 06/24/2013 06:30 PM, [email protected] wrote: > 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).
Yeah, done in V2. Thanks, Wanlong Gao > >> + } >> + >> 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 >> > ------------------------------------------------------------------------------ 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
