From: Wanlong Gao <gaowanlong@cn.fujitsu.com>
To: DAN LI <li.dan@cn.fujitsu.com>
Cc: LTP list <ltp-list@lists.sourceforge.net>
Subject: Re: [LTP] [PATCH 1/2] syscalls/getitimer: cleanup
Date: Wed, 24 Apr 2013 15:40:52 +0800 [thread overview]
Message-ID: <51778C84.1050500@cn.fujitsu.com> (raw)
In-Reply-To: <5177866B.8040606@cn.fujitsu.com>
On 04/24/2013 03:14 PM, DAN LI wrote:
>
> cleanup syscalls/getitimer01.c getitimer02.c
>
> Signed-off-by: DAN LI <li.dan@cn.fujitsu.com>
> ---
> testcases/kernel/syscalls/getitimer/getitimer01.c | 106 ++++++----------------
> testcases/kernel/syscalls/getitimer/getitimer02.c | 96 +++++---------------
> 2 files changed, 53 insertions(+), 149 deletions(-)
>
> diff --git a/testcases/kernel/syscalls/getitimer/getitimer01.c b/testcases/kernel/syscalls/getitimer/getitimer01.c
> index 56c56e0..54aa5bb 100644
> --- a/testcases/kernel/syscalls/getitimer/getitimer01.c
> +++ b/testcases/kernel/syscalls/getitimer/getitimer01.c
> @@ -1,55 +1,28 @@
> /*
> + * Copyright (c) International Business Machines Corp., 2001
> *
> - * Copyright (c) International Business Machines Corp., 2001
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
> + * the GNU General Public License for more details.
> *
> - * This program is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
> - * the GNU General Public License for more details.
> - *
> - * You should have received a copy of the GNU General Public License
> - * along with this program; if not, write to the Free Software
> - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> */
>
> /*
> - * NAME
> - * getitimer01.c
> - *
> - * DESCRIPTION
> - * getitimer01 - check that a correct call to getitimer() succeeds
> - *
> - * ALGORITHM
> - * loop if that option was specified
> - * allocate needed space
> - * issue the system call
> - * check the errno value
> - * issue a PASS message if we get zero
> - * otherwise, the tests fails
> - * issue a FAIL message
> - * break any remaining tests
> - * call cleanup
> - *
> - * USAGE: <for command-line>
> - * getitimer01 [-c n] [-f] [-i n] [-I x] [-P x] [-t]
> - * where, -c n : Run n copies concurrently.
> - * -f : Turn off functionality Testing.
> - * -i n : Execute test n times.
> - * -I x : Execute test for x seconds.
> - * -P x : Pause for x seconds between iterations.
> - * -t : Turn on syscall timing.
> - *
> - * HISTORY
> - * 03/2001 - Written by Wayne Boyer
> - *
> - * RESTRICTIONS
> - * none
> - */
> + HISTORY
> + 03/2001 - Written by Wayne Boyer
> +
> + TEST ITEMS:
> + Check that a correct call to getitimer() succeeds.
> +*/
>
> #include "test.h"
> #include "usctest.h"
> @@ -57,8 +30,8 @@
> #include <errno.h>
> #include <sys/time.h>
>
> -void cleanup(void);
> -void setup(void);
> +static void cleanup(void);
> +static void setup(void);
>
> char *TCID = "getitimer01";
> int TST_TOTAL = 1;
> @@ -69,35 +42,25 @@ int main(int ac, char **av)
> char *msg;
> struct itimerval *value;
>
> - if ((msg = parse_opts(ac, av, NULL, NULL)) != NULL) {
> + if ((msg = parse_opts(ac, av, NULL, NULL)) != NULL)
ERROR: do not use assignment in if condition
> tst_brkm(TBROK, NULL, "OPTION PARSING ERROR - %s", msg);
> - }
> -
> - setup(); /* global setup */
>
> - /* The following loop checks looping state if -i option given */
> + setup();
>
> for (lc = 0; TEST_LOOPING(lc); lc++) {
> - /* reset tst_count in case we are looping */
> tst_count = 0;
>
> /* allocate space for the itimerval structure */
> -
> - if ((value = (struct itimerval *)malloc((size_t)
> - sizeof(struct
> - itimerval))) ==
> - NULL) {
> + value = (struct itimerval *)malloc(
> + (size_t)sizeof(struct itimerval));
The type cast here is not necessary, right?
Thanks,
Wanlong Gao
> + if (value == NULL)
> tst_brkm(TBROK, cleanup, "value malloc failed");
> - }
> -
> - /* call the system call with the TEST() macro */
>
> TEST(getitimer(ITIMER_REAL, value));
>
> - if (TEST_RETURN != 0) {
> + if (TEST_RETURN != 0)
> tst_resm(TFAIL, "call failed - errno = %d - %s",
> TEST_ERRNO, strerror(TEST_ERRNO));
> - }
>
> if (STD_FUNCTIONAL_TEST) {
>
> @@ -122,27 +85,14 @@ int main(int ac, char **av)
> tst_exit();
> }
>
> -/*
> - * setup() - performs all the ONE TIME setup for this test.
> - */
> -void setup(void)
> +static void setup(void)
> {
> -
> tst_sig(NOFORK, DEF_HANDLER, cleanup);
>
> TEST_PAUSE;
> }
>
> -/*
> - * cleanup() - performs all the ONE TIME cleanup for this test at completion
> - * or premature exit.
> - */
> -void cleanup(void)
> +static void cleanup(void)
> {
> - /*
> - * print timing stats if that option was specified.
> - * print errno log if that option was specified.
> - */
> TEST_CLEANUP;
> -
> }
> diff --git a/testcases/kernel/syscalls/getitimer/getitimer02.c b/testcases/kernel/syscalls/getitimer/getitimer02.c
> index fcb3d6c..eab40ae 100644
> --- a/testcases/kernel/syscalls/getitimer/getitimer02.c
> +++ b/testcases/kernel/syscalls/getitimer/getitimer02.c
> @@ -1,55 +1,30 @@
> /*
> + * Copyright (c) International Business Machines Corp., 2001
> *
> - * Copyright (c) International Business Machines Corp., 2001
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
> + * the GNU General Public License for more details.
> *
> - * This program is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
> - * the GNU General Public License for more details.
> - *
> - * You should have received a copy of the GNU General Public License
> - * along with this program; if not, write to the Free Software
> - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> */
>
> /*
> - * NAME
> - * getitimer02.c
> - *
> - * DESCRIPTION
> - * getitimer02 - check that a getitimer() call fails as expected
> - * with an incorrect second argument.
> - *
> - * ALGORITHM
> - * loop if that option was specified
> - * call getitimer() with an incorrect second argument.
> - * check the errno value
> - * issue a PASS message if we get EFAULT - errno 14
> - * otherwise, the tests fails
> - * issue a FAIL message
> - * break any remaining tests
> - * call cleanup
> - *
> - * USAGE: <for command-line>
> - * getitimer02 [-c n] [-e] [-i n] [-I x] [-P x] [-t]
> - * where, -c n : Run n copies concurrently.
> - * -e : Turn on errno logging.
> - * -i n : Execute test n times.
> - * -I x : Execute test for x seconds.
> - * -P x : Pause for x seconds between iterations.
> - * -t : Turn on syscall timing.
> - *
> - * HISTORY
> - * 03/2001 - Written by Wayne Boyer
> - *
> - * RESTRICTIONS
> - * none
> - */
> + HISTORY
> + 03/2001 - Written by Wayne Boyer
> +
> + TEST ITEMS:
> + Check that a getitimer() call fails as expected
> + with an incorrect second argument.
> +*/
> +
>
> #include "test.h"
> #include "usctest.h"
> @@ -62,8 +37,8 @@ int TST_TOTAL = 1;
>
> #if !defined(UCLINUX)
>
> -void cleanup(void);
> -void setup(void);
> +static void cleanup(void);
> +static void setup(void);
>
> int exp_enos[] = { EFAULT, 0 };
>
> @@ -76,19 +51,11 @@ int main(int ac, char **av)
> tst_brkm(TBROK, NULL, "OPTION PARSING ERROR - %s", msg);
> }
>
> - setup(); /* global setup */
> -
> - /* The following loop checks looping state if -i option given */
> + setup();
>
> for (lc = 0; TEST_LOOPING(lc); lc++) {
> - /* reset tst_count in case we are looping */
> tst_count = 0;
>
> - /*
> - * issue the system call with the TEST() macro
> - * ITIMER_REAL = 0, ITIMER_VIRTUAL = 1 and ITIMER_PROF = 2
> - */
> -
> /* call with a bad address */
> TEST(getitimer(ITIMER_REAL, (struct itimerval *)-1));
>
> @@ -118,12 +85,8 @@ int main(int ac, char **av)
> tst_exit();
> }
>
> -/*
> - * setup() - performs all the ONE TIME setup for this test.
> - */
> -void setup(void)
> +static void setup(void)
> {
> -
> tst_sig(NOFORK, DEF_HANDLER, cleanup);
>
> TEST_EXP_ENOS(exp_enos);
> @@ -131,18 +94,9 @@ void setup(void)
> TEST_PAUSE;
> }
>
> -/*
> - * cleanup() - performs all the ONE TIME cleanup for this test at completion
> - * or premature exit.
> - */
> -void cleanup(void)
> +static void cleanup(void)
> {
> - /*
> - * print timing stats if that option was specified.
> - * print errno log if that option was specified.
> - */
> TEST_CLEANUP;
> -
> }
>
> #else
>
------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_apr
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next prev parent reply other threads:[~2013-04-24 7:40 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-24 7:14 [LTP] [PATCH 1/2] syscalls/getitimer: cleanup DAN LI
2013-04-24 7:19 ` [LTP] [PATCH 2/2] syscalls/getitimer01.c: Test for getting ITIMER_VIRTUAL and ITIMER_PROF DAN LI
2013-04-24 7:40 ` Wanlong Gao [this message]
2013-04-24 9:36 ` [LTP] [PATCH 1/2] syscalls/getitimer: cleanup chrubis
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51778C84.1050500@cn.fujitsu.com \
--to=gaowanlong@cn.fujitsu.com \
--cc=li.dan@cn.fujitsu.com \
--cc=ltp-list@lists.sourceforge.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox