From: "M. Mohan Kumar" <mohan@in.ibm.com>
To: subrata@linux.vnet.ibm.com
Cc: ltp-list@lists.sf.net, sukadev@linux.vnet.ibm.com, sachinp@in.ibm.com
Subject: [LTP] Remove pidns14 test case
Date: Wed, 22 Jul 2009 11:59:05 +0530 [thread overview]
Message-ID: <20090722062905.GA3113@in.ibm.com> (raw)
[PATCH] Remove pidns14 test case
Container-init may be immune to unhandled fatal signals (like SIGUSR1)
even if they are from ancestor namespace. SIGKILL/SIGSTOP are the only
reliable signals to a container-init from ancestor namespace. So remove
test case pidns14 from container test suite. Please check git commit id
b3bfa0cba867f23365b81658b47efd906830879b for the kernel changes.
Signed-off-by: M. Mohan Kumar <mohan@in.ibm.com>
---
testcases/kernel/containers/pidns/pidns14.c | 122 ---------------------
testcases/kernel/containers/pidns/runpidnstest.sh | 6 -
2 files changed, 0 insertions(+), 128 deletions(-)
delete mode 100644 testcases/kernel/containers/pidns/pidns14.c
diff --git a/testcases/kernel/containers/pidns/pidns14.c b/testcases/kernel/containers/pidns/pidns14.c
deleted file mode 100644
index e95bf95..0000000
--- a/testcases/kernel/containers/pidns/pidns14.c
+++ /dev/null
@@ -1,122 +0,0 @@
-/*
-* Copyright (c) International Business Machines Corp., 2008
-* 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.
-* 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., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
-*
-***************************************************************************
-* * Assertion:
-* * kill -USR1 container_init from outside a container
-* * $ Where init has not defined a custom handler for USR1
-* * $ Should kill the container
-* * $ else the test fails.
-* *
-* * Description:
-* * This testcase creates container and waits till it is awakened by parent.
-* * The parent process sends the SIGUSR1 to container init.
-* * This would invoke default handler & terminate the container init.
-* * In parent process verify, if the SIGUSR1 signal is passed to c-init.
-* * If yes then Test is Passed else Test is Failed.
-* *
-* * History:
-* * DATE NAME DESCRIPTION
-* * 14/11/08 Veerendra C <vechandr@in.ibm.com> Verifying kill -USR1 in pidns
-*
-******************************************************************************/
-#include "config.h"
-
-#define _GNU_SOURCE 1
-#include <sys/wait.h>
-#include <sys/types.h>
-#include <signal.h>
-#include <stdlib.h>
-#include <unistd.h>
-#include <usctest.h>
-#include <test.h>
-#include <libclone.h>
-
-char *TCID = "pidns14";
-int TST_TOTAL = 1;
-
-int child_fn(void *);
-void cleanup(void);
-
-#define CHILD_PID 1
-#define PARENT_PID 0
-
-/*
- * child_fn() - Inside container
- */
-int child_fn(void *ttype)
-{
- pid_t pid, ppid;
-
- /* Set process id and parent pid */
- pid = getpid();
- ppid = getppid();
-
- if ((pid != CHILD_PID) || (ppid != PARENT_PID)) {
- tst_resm(TBROK, "pidns is not created.");
- cleanup();
- }
- pause();
- tst_resm(TFAIL, "Oops! Container init resumed after receiving SIGUSR1");
- return -1;
-}
-
-/*
- * cleanup() - performs all ONE TIME cleanup for this test at
- * completion or premature exit.
- */
-void cleanup()
-{
- /* Clean the test testcase as LTP wants*/
- TEST_CLEANUP;
- tst_exit();
-
-}
-
-/***********************************************************************
-* M A I N
-***********************************************************************/
-
-int main(int argc, char *argv[])
-{
- int status;
- pid_t cpid;
-
- cpid = do_clone(CLONE_NEWPID | SIGCHLD, child_fn, NULL);
-
- if (cpid < 0) {
- tst_resm(TBROK, "clone() failed.");
- cleanup();
- }
-
- sleep(1);
- /* Passing the SIGUSR1 to the container init */
- if (kill(cpid, SIGUSR1) != 0) {
- tst_resm(TBROK, "kill(SIGUSR1) fails.");
- cleanup();
- }
-
- sleep(1);
- if (waitpid(cpid, &status, 0) < 0)
- tst_resm(TWARN, "waitpid() failed.");
-
- if ((WIFSIGNALED(status)) && (WTERMSIG(status) == SIGUSR1))
- tst_resm(TPASS, "Container init is killed as expected, "
- " when the SIGUSR1 is passed from parent\n");
- else
- tst_resm(TFAIL, "After sending signal kill -USR1, "
- "returned unexpected error\n");
-
- return 0;
-} /* End main */
diff --git a/testcases/kernel/containers/pidns/runpidnstest.sh b/testcases/kernel/containers/pidns/runpidnstest.sh
index 12d3b34..03fed38 100644
--- a/testcases/kernel/containers/pidns/runpidnstest.sh
+++ b/testcases/kernel/containers/pidns/runpidnstest.sh
@@ -85,12 +85,6 @@ if [ $rc -ne 0 ] && [ -z $err_code ]; then
err_code=$rc
fi
-pidns14
-rc=$?
-if [ $rc -ne 0 ] && [ -z $err_code ]; then
- err_code=$rc
-fi
-
pidns16
rc=$?
if [ $rc -ne 0 ] && [ -z $err_code ]; then
--
1.6.2.5
------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next reply other threads:[~2009-07-22 6:44 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-22 6:29 M. Mohan Kumar [this message]
2009-07-30 18:29 ` [LTP] Remove pidns14 test case Subrata Modak
2009-08-03 12:09 ` M. Mohan Kumar
2009-08-04 12:06 ` Subrata Modak
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=20090722062905.GA3113@in.ibm.com \
--to=mohan@in.ibm.com \
--cc=ltp-list@lists.sf.net \
--cc=sachinp@in.ibm.com \
--cc=subrata@linux.vnet.ibm.com \
--cc=sukadev@linux.vnet.ibm.com \
/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