diff mbox series

syscalls/mkdir09: rewrite in new LTP API

Message ID a0733b841c7867ca6af1feb033b691c9f72ce0ec.1643291588.git.jstancek@redhat.com
State Changes Requested
Headers show
Series syscalls/mkdir09: rewrite in new LTP API | expand

Commit Message

Jan Stancek Jan. 27, 2022, 1:56 p.m. UTC
'jump' variable is not initialized, which I suspect is behind
rare failures of this test. The original test is using longjmp,
processes and signals to sychronize couple processes that
exercise mkdir/rmdir calls.

Rewrite it using threads and new LTP API, drop all parameters,
because no runtest is using them, and make new default test time
just 1 second.

Signed-off-by: Jan Stancek <jstancek@redhat.com>
---
It's based on original test, but I question its usefulness,
since it's just mkdir/rmdir in loop over same directory.
So, I decided to port it with just 1 second test time, but
I'm not opposed to dropping this test entirely. Thoughts?

 testcases/kernel/syscalls/mkdir/Makefile  |   2 +
 testcases/kernel/syscalls/mkdir/mkdir09.c | 629 ++++++----------------
 2 files changed, 171 insertions(+), 460 deletions(-)
 rewrite testcases/kernel/syscalls/mkdir/mkdir09.c (92%)

Comments

Petr Vorel Jan. 27, 2022, 2:04 p.m. UTC | #1
Hi Jan,

> 'jump' variable is not initialized, which I suspect is behind
> rare failures of this test. The original test is using longjmp,
> processes and signals to sychronize couple processes that
> exercise mkdir/rmdir calls.

> Rewrite it using threads and new LTP API, drop all parameters,
> because no runtest is using them, and make new default test time
> just 1 second.

> Signed-off-by: Jan Stancek <jstancek@redhat.com>
> ---
> It's based on original test, but I question its usefulness,
> since it's just mkdir/rmdir in loop over same directory.
> So, I decided to port it with just 1 second test time, but
> I'm not opposed to dropping this test entirely. Thoughts?

Not sure about usefulness. Also how about use .all_filesystems to have smoke
test for all supported filesystems?

Kind regards,
Petr

+++ testcases/kernel/syscalls/mkdir/mkdir09.c
@@ -14,9 +14,10 @@
 #include "tst_test.h"
 #include "tst_safe_pthread.h"
 
+#define MNTPOINT "mntpoint"
 #define MODE_RWX	07770
-#define DIR_NAME	"./X.%d"
-#define DIR_NAME_GROUP	"./X.%d.%d"
+#define DIR_NAME	MNTPOINT "/X.%d"
+#define DIR_NAME_GROUP	MNTPOINT "/X.%d.%d"
 #define NCHILD		3
 
 static int child_groups = 2;
@@ -166,4 +167,7 @@ static struct tst_test test = {
 	.test_all = verify_mkdir,
 	.needs_tmpdir = 1,
 	.setup = setup,
+	.mount_device = 1,
+	.mntpoint = MNTPOINT,
+	.all_filesystems = 1,
 };
Cyril Hrubis Feb. 4, 2022, 12:26 p.m. UTC | #2
Hi!
> It's based on original test, but I question its usefulness,
> since it's just mkdir/rmdir in loop over same directory.
> So, I decided to port it with just 1 second test time, but
> I'm not opposed to dropping this test entirely. Thoughts?

It's pretty basic but I do not think that it's a reason for a removal.

I guess that more comprehensive stress test would be a better
replacement though. For example we do have a test that keeps adding
files until it gets ENOSPC (fs_fill.c) and I guess having something
similar for directories would make sense as well. I guess that we would
have to do it differently though, create a file(s) that eat most of the
available disk space, then do a similar stress test for the directories
as well.

>  testcases/kernel/syscalls/mkdir/Makefile  |   2 +
>  testcases/kernel/syscalls/mkdir/mkdir09.c | 629 ++++++----------------
>  2 files changed, 171 insertions(+), 460 deletions(-)
>  rewrite testcases/kernel/syscalls/mkdir/mkdir09.c (92%)
> 
> diff --git a/testcases/kernel/syscalls/mkdir/Makefile b/testcases/kernel/syscalls/mkdir/Makefile
> index 044619fb8724..881b087c6c1a 100644
> --- a/testcases/kernel/syscalls/mkdir/Makefile
> +++ b/testcases/kernel/syscalls/mkdir/Makefile
> @@ -3,6 +3,8 @@
>  
>  top_srcdir		?= ../../../..
>  
> +mkdir09: CFLAGS += -pthread
> +
>  include $(top_srcdir)/include/mk/testcases.mk
>  
>  include $(top_srcdir)/include/mk/generic_leaf_target.mk
> diff --git a/testcases/kernel/syscalls/mkdir/mkdir09.c b/testcases/kernel/syscalls/mkdir/mkdir09.c
> dissimilarity index 92%
> index 88034d29ed89..3afb19d72ec3 100644
> --- a/testcases/kernel/syscalls/mkdir/mkdir09.c
> +++ b/testcases/kernel/syscalls/mkdir/mkdir09.c
> @@ -1,460 +1,169 @@
> -/*
> - *
> - *   Copyright (c) International Business Machines  Corp., 2002
> - *
> - *   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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> - */
> -
> -/* 06/30/2001	Port to Linux	nsharoff@us.ibm.com */
> -/* 10/30/2002	Port to LTP	dbarrera@us.ibm.com */
> -
> -/*
> - * Stress test of mkdir call.
> - *
> - * ALGORITHM
> - *	Create multiple processes which create subdirectories in the
> - *	same directory multiple times. On exit of all child processes,
> - *	make sure all subdirectories can be removed.
> - *
> - *      USAGE: mkdir09 -c # -t # -d #
> - *              -c = number of children groups
> - *              -t = number of seconds to run test
> - *              -d = number of directories created in test directory
> - *
> - */
> -
> -#include <stdio.h>
> -#include <sys/wait.h>
> -#include <sys/types.h>
> -#include <sys/param.h>
> -#include <sys/stat.h>
> -#include <sys/mman.h>
> -#include <errno.h>
> -#include <signal.h>
> -#include <unistd.h>
> -#include <setjmp.h>
> -#include "test.h"
> -
> -#include <stdlib.h>
> -#include <stdlib.h>
> -#include <string.h>
> -
> -#define NCHILD		3
> -
> -#define MODE_RWX	07770
> -#define DIR_NAME	"./X.%d"
> -
> -char *TCID = "mkdir09";
> -int TST_TOTAL = 1;
> -
> -char testdir[MAXPATHLEN];
> -int parent_pid, sigchld, sigterm, jump;
> -void term(int sig);
> -void chld(int sig);
> -int *pidlist, child_count;
> -jmp_buf env_buf;
> -
> -int getchild(int group, int child, int children);
> -int dochild1(void);
> -int dochild2(void);
> -int dochild3(int group);
> -int massmurder(void);
> -int runtest(void);
> -void setup(void);
> -void cleanup(void);
> -
> -static int child_groups = 2;
> -static int test_time = 5;
> -static int nfiles = 5;
> -
> -static char *opt_child_groups;
> -static char *opt_test_time;
> -static char *opt_nfiles;
> -
> -static option_t options[] = {
> -	{"c:", NULL, &opt_child_groups},
> -	{"t:", NULL, &opt_test_time},
> -	{"d:", NULL, &opt_nfiles},
> -	{NULL, NULL, NULL}
> -};
> -
> -static void usage(void)
> -{
> -	printf("  -c      Child groups\n");
> -	printf("  -t      Test runtime\n");
> -	printf("  -d      Directories\n");
> -}
> -
> -int main(int argc, char *argv[])
> -{
> -	tst_parse_opts(argc, argv, options, usage);
> -
> -	if (opt_child_groups)
> -		child_groups = atoi(opt_child_groups);
> -
> -	if (opt_test_time)
> -		test_time = atoi(opt_test_time);
> -
> -	if (opt_nfiles)
> -		nfiles = atoi(opt_nfiles);
> -
> -	setup();
> -
> -	if (signal(SIGTERM, term) == SIG_ERR) {
> -		tst_brkm(TFAIL, cleanup,
> -			 "Error setting up SIGTERM signal, ERRNO = %d", errno);
> -
> -	}
> -
> -	if (signal(SIGCHLD, chld) == SIG_ERR) {
> -		tst_brkm(TFAIL, cleanup,
> -			 "Error setting up SIGCHLD signal, ERRNO = %d", errno);
> -
> -	}
> -
> -	runtest();
> -	cleanup();
> -	tst_exit();
> -}
> -
> -int runtest(void)
> -{
> -	int i, j;
> -	int count, child, status;
> -	char tmpdir[MAXPATHLEN];
> -
> -	/* Create permanent directories with holes in directory structure */
> -
> -	for (j = 0; j < nfiles; j++) {
> -		sprintf(tmpdir, DIR_NAME, j);
> -		TEST(mkdir(tmpdir, MODE_RWX));
> -
> -		if (TEST_RETURN < 0) {
> -			tst_brkm(TFAIL, cleanup,
> -				 "Error creating permanent directories, ERRNO = %d",
> -				 TEST_ERRNO);
> -		}
> -		if ((j % NCHILD) != 0) {
> -			if (rmdir(tmpdir) < 0) {
> -				tst_brkm(TFAIL, cleanup,
> -					 "Error removing directory, ERRNO = %d",
> -					 errno);
> -			}
> -		}
> -	}
> -
> -	parent_pid = getpid();
> -
> -	/* allocate space for list of child pid's */
> -
> -	if ((pidlist = malloc((child_groups * NCHILD) * sizeof(int))) ==
> -	    NULL) {
> -		tst_brkm(TWARN, NULL,
> -			 "\tMalloc failed (may be OK if under stress)");
> -	}
> -
> -	child_count = 0;
> -	for (j = 0; j < child_groups; j++) {
> -		for (i = 0; i < NCHILD; i++) {
> -			getchild(j, i, child_count);
> -			child_count++;
> -		}
> -	}
> -
> -	/* If signal already received, skip to cleanup */
> -
> -	if (!sigchld && !sigterm) {
> -		if (test_time) {
> -			/* To get out of sleep if signal caught */
> -			if (!setjmp(env_buf)) {
> -				jump++;
> -				sleep(test_time);
> -			}
> -		} else {
> -			pause();
> -		}
> -	}
> -
> -	/* Reset signals since we are about to clean-up and to avoid
> -	 * problem with wait call *               $
> -	 * */
> -
> -	if (signal(SIGTERM, SIG_IGN) == SIG_ERR) {
> -		tst_brkm(TFAIL, cleanup,
> -			 "Error resetting SIGTERM signal, ERRNO = %d", errno);
> -	}
> -	if (signal(SIGCHLD, SIG_DFL) == SIG_ERR) {
> -		tst_brkm(TFAIL, cleanup,
> -			 "Error resetting SIGCHLD signal, ERRNO = %d", errno);
> -	}
> -
> -	if (test_time) {
> -		sleep(test_time);
> -	}
> -
> -	/* Clean up children */
> -	massmurder();
> -	/*
> -	 * Watch children finish and show returns.
> -	 */
> -
> -	count = 0;
> -	while (1) {
> -		if ((child = wait(&status)) > 0) {
> -			if (status != 0) {
> -				tst_brkm(TWARN,
> -					 NULL,
> -					 "\tChild{%d} exited status = %0x",
> -					 child, status);
> -			}
> -			count++;
> -		} else {
> -			if (errno != EINTR) {
> -				break;
> -			}
> -			tst_resm(TINFO, "\tSignal detected during wait");
> -		}
> -	}
> -
> -	/*
> -	 * Make sure correct number of children exited.
> -	 */
> -
> -	if (count != child_count) {
> -		tst_resm(TWARN, "\tWrong number of children waited on!");
> -		tst_brkm(TWARN, NULL, "\tSaw %d, expected %d", count,
> -			 NCHILD);
> -	}
> -
> -	/* Check for core file in test directory. */
> -
> -	if (access("core", 0) == 0) {
> -		tst_brkm(TWARN, NULL, "\tCore file found in test directory.");
> -	}
> -
> -	/* Remove expected files */
> -
> -	for (j = 0; j < nfiles; j += NCHILD) {
> -		sprintf(tmpdir, DIR_NAME, j);
> -		if (rmdir(tmpdir) < 0) {
> -			tst_brkm(TWARN,
> -				 NULL,
> -				 "\tError removing expected directory, ERRNO = %d",
> -				 errno);
> -		}
> -	}
> -
> -	tst_resm(TPASS, "PASS");
> -
> -	return 0;
> -}
> -
> -int getchild(int group, int child, int children)
> -{
> -	int pid;
> -
> -	pid = FORK_OR_VFORK();
> -
> -	if (pid < 0) {
> -
> -		massmurder();	/* kill the kids */
> -		tst_brkm(TBROK, cleanup,
> -			 "\tFork failed (may be OK if under stress)");
> -	} else if (pid == 0) {	/* child does this */
> -		switch (children % NCHILD) {
> -		case 0:
> -			dochild1();	/* create existing directories */
> -			break;	/* so lint won't complain */
> -		case 1:
> -			dochild2();	/* remove nonexistant directories */
> -			break;
> -		case 2:
> -			dochild3(group);	/* create/delete directories */
> -			break;
> -		default:
> -			tst_brkm(TFAIL, cleanup,
> -				 "Test not inplemented for child %d", child);
> -			exit(1);
> -			break;
> -		}
> -		exit(1);	/* If child gets here, something wrong */
> -	}
> -	pidlist[children] = pid;
> -	return 0;
> -}
> -
> -void term(int sig)
> -{
> -	/* Routine to handle SIGTERM signal. */
> -
> -	if (parent_pid == getpid()) {
> -		tst_brkm(TWARN, NULL, "\tsignal SIGTERM received by parent.");
> -	}
> -	sigterm++;
> -	if (jump) {
> -		longjmp(env_buf, 1);
> -	}
> -}
> -
> -void chld(int sig)
> -{
> -	/* Routine to handle SIGCHLD signal. */
> -
> -	sigchld++;
> -	if (jump) {
> -		longjmp(env_buf, 1);
> -	}
> -}
> -
> -int dochild1(void)
> -{
> -	/* Child routine which attempts to create directories in the test
> -	 * directory that already exist. Runs until a SIGTERM signal is
> -	 * received. Will exit with an error if it is able to create the
> -	 * directory or if the expected error is not received.
> -	 */
> -
> -	int j;
> -	char tmpdir[MAXPATHLEN];
> -
> -	while (!sigterm) {
> -		for (j = 0; j < nfiles; j += NCHILD) {
> -			sprintf(tmpdir, DIR_NAME, j);
> -			TEST(mkdir(tmpdir, MODE_RWX));
> -
> -			if (TEST_RETURN < 0) {
> -
> -				if (TEST_ERRNO != EEXIST) {
> -					tst_brkm(TFAIL, cleanup,
> -						 "MKDIR %s, errno = %d; Wrong error detected.",
> -						 tmpdir, TEST_ERRNO);
> -					exit(1);
> -				}
> -			} else {
> -				tst_brkm(TFAIL, cleanup,
> -					 "MKDIR %s succeded when it shoud have failed.",
> -					 tmpdir);
> -				exit(1);
> -			}
> -		}
> -	}
> -	exit(0);
> -}
> -
> -int dochild2(void)
> -{
> -	/* Child routine which attempts to remove directories from the
> -	 * test directory which do not exist. Runs until a SIGTERM
> -	 * signal is received. Exits with an error if the proper
> -	 * error is not detected or if the remove operation is
> -	 * successful.
> -	 */
> -
> -	int j;
> -	char tmpdir[MAXPATHLEN];
> -
> -	while (!sigterm) {
> -		for (j = 1; j < nfiles; j += NCHILD) {
> -			sprintf(tmpdir, DIR_NAME, j);
> -			if (rmdir(tmpdir) < 0) {
> -				if (errno != ENOENT) {
> -					tst_brkm(TFAIL, cleanup,
> -						 "RMDIR %s, errno = %d; Wrong error detected.",
> -						 tmpdir, errno);
> -					exit(1);
> -				}
> -			} else {
> -				tst_brkm(TFAIL, cleanup,
> -					 "RMDIR %s succeded when it should have failed.",
> -					 tmpdir);
> -				exit(1);
> -			}
> -		}
> -	}
> -	exit(0);
> -	return 0;
> -}
> -
> -int dochild3(int group)
> -{
> -	/* Child routine which creates and deletes directories in the
> -	 * test directory. Runs until a SIGTERM signal is received, then
> -	 * cleans up and exits. Detects error if the expected condition
> -	 * is not encountered.
> -	 */
> -
> -	int j;
> -
> -	char tmpdir[MAXPATHLEN];
> -	char tmp[MAXPATHLEN];
> -
> -	while (!sigterm) {
> -		for (j = 2; j < nfiles; j += NCHILD) {
> -			strcpy(tmp, DIR_NAME);
> -			strcat(tmp, ".%d");
> -			sprintf(tmpdir, tmp, j, group);
> -
> -			TEST(mkdir(tmpdir, MODE_RWX));
> -
> -			if (TEST_RETURN < 0) {
> -				tst_brkm(TFAIL, cleanup,
> -					 "MKDIR %s, errno = %d; Wrong error detected.",
> -					 tmpdir, TEST_ERRNO);
> -				exit(1);
> -			}
> -		}
> -		for (j = 2; j < nfiles; j += NCHILD) {
> -			strcpy(tmp, DIR_NAME);
> -			strcat(tmp, ".%d");
> -			sprintf(tmpdir, tmp, j, group);
> -			if (rmdir(tmpdir) < 0) {
> -				tst_brkm(TFAIL, cleanup,
> -					 "RMDIR %s, errno = %d; Wrong error detected.",
> -					 tmpdir, errno);
> -				exit(1);
> -			}
> -		}
> -	}
> -	exit(0);
> -}
> -
> -int massmurder(void)
> -{
> -	register int j;
> -	for (j = 0; j < child_count; j++) {
> -		if (pidlist[j] > 0) {
> -			if (kill(pidlist[j], SIGTERM) < 0) {
> -				tst_brkm(TFAIL, cleanup,
> -					 "Error killing child %d, ERRNO = %d",
> -					 j, errno);
> -			}
> -		}
> -	}
> -	return 0;
> -}
> -
> -void setup(void)
> -{
> -	tst_sig(NOFORK, DEF_HANDLER, cleanup);
> -
> -	TEST_PAUSE;
> -
> -	tst_tmpdir();
> -}
> -
> -void cleanup(void)
> -{
> -	tst_rmdir();
> -}
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +/* Copyright (c) International Business Machines Corp., 2001
> + * Copyright (c) Linux Test Project, 2022
> + */
> +
> +/*
> + * DESCRIPTION

Please change this to a proper docparser comment

/*\
 * [Description]
 *
 * ...

> + *  Create multiple processes which create subdirectories in the
> + *  same directory multiple times within test time.
> + */
> +
> +#include <stdio.h>
> +#include <sys/param.h>
> +#include "tst_test.h"
> +#include "tst_safe_pthread.h"
> +
> +#define MODE_RWX	07770
> +#define DIR_NAME	"./X.%d"
> +#define DIR_NAME_GROUP	"./X.%d.%d"
> +#define NCHILD		3
> +
> +static int child_groups = 2;
> +static int test_time = 1;
> +static int nfiles = 5;
> +static volatile int done;
> +
> +static void test1(int child_num)
> +{
> +	/*
> +	 * Routine which attempts to create directories in the test
> +	 * directory that already exist.
> +	 */

I would put this comment before the function definition.

> +	int j, first_loop = 1;
> +	char tmpdir[MAXPATHLEN];
> +
> +	while (!done) {
> +		for (j = 0; j < nfiles; j += NCHILD) {
> +			sprintf(tmpdir, DIR_NAME, j);
> +			TEST(mkdir(tmpdir, MODE_RWX));
> +
> +			/* use first loop to create dirs, skip checks */
> +			if (first_loop)
> +				continue;

Maybe we should do just separate loop with SAFE_MKDIR() before we start
this loop.

> +
> +			if (TST_RET == 0) {
> +				tst_res(TFAIL, "[%d] mkdir %s succeeded unexpectedly",
> +					child_num, tmpdir);
> +				return;
> +			}
> +
> +			if (TST_ERR != EEXIST) {
> +				tst_res(TFAIL|TERRNO,
> +					"[%d] MKDIR %s expected EEXIST",
> +					child_num, tmpdir);
> +				return;
> +			}

I would say that these checks could have been replaced with
TST_EXP_FAIL_SILENT() but it looks like we don't have this variant yet.
Maybe we should add it first.

> +		}
> +		first_loop = 0;
> +	}
> +	tst_res(TPASS, "[%d] create dirs that already exist", child_num);
> +}
> +
> +static void test2(int child_num)
> +{
> +	/*
> +	 * Child routine which attempts to remove directories from the
> +	 * test directory which do not exist.
> +	 */
> +
> +	int j;
> +	char tmpdir[MAXPATHLEN];
> +
> +	while (!done) {
> +		for (j = 1; j < nfiles; j += NCHILD) {
> +			sprintf(tmpdir, DIR_NAME, j);
> +			TEST(rmdir(tmpdir));
> +
> +			if (TST_RET == 0) {
> +				tst_res(TFAIL, "[%d] rmdir %s succeeded unexpectedly",
> +					 child_num, tmpdir);
> +				return;
> +			}
> +
> +			if (TST_ERR != ENOENT) {
> +				tst_res(TFAIL|TERRNO, "[%d] rmdir %s",
> +					child_num, tmpdir);
> +				return;
> +			}
> +		}
> +	}
> +	tst_res(TPASS, "[%d] create dirs that dont exist", child_num);
> +}

The same applies here as well.

> +static void test3(int child_num)
> +{
> +	/*
> +	 * Child routine which creates and deletes directories in the
> +	 * test directory.
> +	 */
> +
> +	int j;
> +	char tmpdir[MAXPATHLEN];
> +
> +	while (!done) {
> +		for (j = 2; j < nfiles; j += NCHILD) {
> +			sprintf(tmpdir, DIR_NAME_GROUP, j, child_num / NCHILD);
> +
> +			TEST(mkdir(tmpdir, MODE_RWX));
> +			if (TST_RET < 0) {
> +				tst_res(TFAIL | TERRNO, "[%d] mkdir %s",
> +					child_num, tmpdir);
> +				return;
> +			}

TST_EXP_PASS_SILENT()?

> +		}
> +		for (j = 2; j < nfiles; j += NCHILD) {
> +			sprintf(tmpdir, DIR_NAME_GROUP, j, child_num / NCHILD);
> +			TEST(rmdir(tmpdir));
> +			if (TST_RET < 0) {
> +				tst_res(TFAIL | TERRNO, "[%d] rmdir %s",
> +					child_num, tmpdir);
> +				return;
> +			}

Here as well.

> +		}
> +	}
> +	tst_res(TPASS, "[%d] create/remove dirs", child_num);
> +}
> +
> +
> +static void *child_thread_func(void *arg)
> +{
> +	void (*tests[NCHILD])(int) = { test1, test2, test3 };
> +	int child_num = (long)arg;
> +
> +	tests[child_num % NCHILD](child_num);
> +
> +	/* if any thread failed, make other finish as well */
> +	done = 1;
> +
> +	return NULL;
> +}
> +
> +
> +static void verify_mkdir(void)
> +{
> +	pthread_t child_thread[NCHILD * child_groups];
> +	long i;
> +
> +	done = 0;
> +	for (i = 0; i < child_groups * NCHILD; i++) {
> +		SAFE_PTHREAD_CREATE(&child_thread[i], NULL,
> +			child_thread_func, (void *)i);
> +	}
> +
> +	sleep(test_time);
> +	done = 1;
> +
> +	for (i = 0; i < child_groups * NCHILD; i++)
> +		SAFE_PTHREAD_JOIN(child_thread[i], NULL);
> +}
> +
> +static void setup(void)
> +{
> +}

Just remove this.

> +static struct tst_test test = {
> +	.test_all = verify_mkdir,
> +	.needs_tmpdir = 1,
> +	.setup = setup,
> +};
> -- 
> 2.27.0
> 
> 
> -- 
> Mailing list info: https://lists.linux.it/listinfo/ltp
Cyril Hrubis Feb. 4, 2022, 12:39 p.m. UTC | #3
Hi!
> > +static void test2(int child_num)
> > +{
> > +	/*
> > +	 * Child routine which attempts to remove directories from the
> > +	 * test directory which do not exist.
> > +	 */
> > +
> > +	int j;
> > +	char tmpdir[MAXPATHLEN];
> > +
> > +	while (!done) {
> > +		for (j = 1; j < nfiles; j += NCHILD) {
> > +			sprintf(tmpdir, DIR_NAME, j);
> > +			TEST(rmdir(tmpdir));
> > +
> > +			if (TST_RET == 0) {
> > +				tst_res(TFAIL, "[%d] rmdir %s succeeded unexpectedly",
> > +					 child_num, tmpdir);
> > +				return;
> > +			}
> > +
> > +			if (TST_ERR != ENOENT) {
> > +				tst_res(TFAIL|TERRNO, "[%d] rmdir %s",
> > +					child_num, tmpdir);
> > +				return;
> > +			}
> > +		}
> > +	}
> > +	tst_res(TPASS, "[%d] create dirs that dont exist", child_num);
                             ^
			     remove dirs
diff mbox series

Patch

diff --git a/testcases/kernel/syscalls/mkdir/Makefile b/testcases/kernel/syscalls/mkdir/Makefile
index 044619fb8724..881b087c6c1a 100644
--- a/testcases/kernel/syscalls/mkdir/Makefile
+++ b/testcases/kernel/syscalls/mkdir/Makefile
@@ -3,6 +3,8 @@ 
 
 top_srcdir		?= ../../../..
 
+mkdir09: CFLAGS += -pthread
+
 include $(top_srcdir)/include/mk/testcases.mk
 
 include $(top_srcdir)/include/mk/generic_leaf_target.mk
diff --git a/testcases/kernel/syscalls/mkdir/mkdir09.c b/testcases/kernel/syscalls/mkdir/mkdir09.c
dissimilarity index 92%
index 88034d29ed89..3afb19d72ec3 100644
--- a/testcases/kernel/syscalls/mkdir/mkdir09.c
+++ b/testcases/kernel/syscalls/mkdir/mkdir09.c
@@ -1,460 +1,169 @@ 
-/*
- *
- *   Copyright (c) International Business Machines  Corp., 2002
- *
- *   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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-/* 06/30/2001	Port to Linux	nsharoff@us.ibm.com */
-/* 10/30/2002	Port to LTP	dbarrera@us.ibm.com */
-
-/*
- * Stress test of mkdir call.
- *
- * ALGORITHM
- *	Create multiple processes which create subdirectories in the
- *	same directory multiple times. On exit of all child processes,
- *	make sure all subdirectories can be removed.
- *
- *      USAGE: mkdir09 -c # -t # -d #
- *              -c = number of children groups
- *              -t = number of seconds to run test
- *              -d = number of directories created in test directory
- *
- */
-
-#include <stdio.h>
-#include <sys/wait.h>
-#include <sys/types.h>
-#include <sys/param.h>
-#include <sys/stat.h>
-#include <sys/mman.h>
-#include <errno.h>
-#include <signal.h>
-#include <unistd.h>
-#include <setjmp.h>
-#include "test.h"
-
-#include <stdlib.h>
-#include <stdlib.h>
-#include <string.h>
-
-#define NCHILD		3
-
-#define MODE_RWX	07770
-#define DIR_NAME	"./X.%d"
-
-char *TCID = "mkdir09";
-int TST_TOTAL = 1;
-
-char testdir[MAXPATHLEN];
-int parent_pid, sigchld, sigterm, jump;
-void term(int sig);
-void chld(int sig);
-int *pidlist, child_count;
-jmp_buf env_buf;
-
-int getchild(int group, int child, int children);
-int dochild1(void);
-int dochild2(void);
-int dochild3(int group);
-int massmurder(void);
-int runtest(void);
-void setup(void);
-void cleanup(void);
-
-static int child_groups = 2;
-static int test_time = 5;
-static int nfiles = 5;
-
-static char *opt_child_groups;
-static char *opt_test_time;
-static char *opt_nfiles;
-
-static option_t options[] = {
-	{"c:", NULL, &opt_child_groups},
-	{"t:", NULL, &opt_test_time},
-	{"d:", NULL, &opt_nfiles},
-	{NULL, NULL, NULL}
-};
-
-static void usage(void)
-{
-	printf("  -c      Child groups\n");
-	printf("  -t      Test runtime\n");
-	printf("  -d      Directories\n");
-}
-
-int main(int argc, char *argv[])
-{
-	tst_parse_opts(argc, argv, options, usage);
-
-	if (opt_child_groups)
-		child_groups = atoi(opt_child_groups);
-
-	if (opt_test_time)
-		test_time = atoi(opt_test_time);
-
-	if (opt_nfiles)
-		nfiles = atoi(opt_nfiles);
-
-	setup();
-
-	if (signal(SIGTERM, term) == SIG_ERR) {
-		tst_brkm(TFAIL, cleanup,
-			 "Error setting up SIGTERM signal, ERRNO = %d", errno);
-
-	}
-
-	if (signal(SIGCHLD, chld) == SIG_ERR) {
-		tst_brkm(TFAIL, cleanup,
-			 "Error setting up SIGCHLD signal, ERRNO = %d", errno);
-
-	}
-
-	runtest();
-	cleanup();
-	tst_exit();
-}
-
-int runtest(void)
-{
-	int i, j;
-	int count, child, status;
-	char tmpdir[MAXPATHLEN];
-
-	/* Create permanent directories with holes in directory structure */
-
-	for (j = 0; j < nfiles; j++) {
-		sprintf(tmpdir, DIR_NAME, j);
-		TEST(mkdir(tmpdir, MODE_RWX));
-
-		if (TEST_RETURN < 0) {
-			tst_brkm(TFAIL, cleanup,
-				 "Error creating permanent directories, ERRNO = %d",
-				 TEST_ERRNO);
-		}
-		if ((j % NCHILD) != 0) {
-			if (rmdir(tmpdir) < 0) {
-				tst_brkm(TFAIL, cleanup,
-					 "Error removing directory, ERRNO = %d",
-					 errno);
-			}
-		}
-	}
-
-	parent_pid = getpid();
-
-	/* allocate space for list of child pid's */
-
-	if ((pidlist = malloc((child_groups * NCHILD) * sizeof(int))) ==
-	    NULL) {
-		tst_brkm(TWARN, NULL,
-			 "\tMalloc failed (may be OK if under stress)");
-	}
-
-	child_count = 0;
-	for (j = 0; j < child_groups; j++) {
-		for (i = 0; i < NCHILD; i++) {
-			getchild(j, i, child_count);
-			child_count++;
-		}
-	}
-
-	/* If signal already received, skip to cleanup */
-
-	if (!sigchld && !sigterm) {
-		if (test_time) {
-			/* To get out of sleep if signal caught */
-			if (!setjmp(env_buf)) {
-				jump++;
-				sleep(test_time);
-			}
-		} else {
-			pause();
-		}
-	}
-
-	/* Reset signals since we are about to clean-up and to avoid
-	 * problem with wait call *               $
-	 * */
-
-	if (signal(SIGTERM, SIG_IGN) == SIG_ERR) {
-		tst_brkm(TFAIL, cleanup,
-			 "Error resetting SIGTERM signal, ERRNO = %d", errno);
-	}
-	if (signal(SIGCHLD, SIG_DFL) == SIG_ERR) {
-		tst_brkm(TFAIL, cleanup,
-			 "Error resetting SIGCHLD signal, ERRNO = %d", errno);
-	}
-
-	if (test_time) {
-		sleep(test_time);
-	}
-
-	/* Clean up children */
-	massmurder();
-	/*
-	 * Watch children finish and show returns.
-	 */
-
-	count = 0;
-	while (1) {
-		if ((child = wait(&status)) > 0) {
-			if (status != 0) {
-				tst_brkm(TWARN,
-					 NULL,
-					 "\tChild{%d} exited status = %0x",
-					 child, status);
-			}
-			count++;
-		} else {
-			if (errno != EINTR) {
-				break;
-			}
-			tst_resm(TINFO, "\tSignal detected during wait");
-		}
-	}
-
-	/*
-	 * Make sure correct number of children exited.
-	 */
-
-	if (count != child_count) {
-		tst_resm(TWARN, "\tWrong number of children waited on!");
-		tst_brkm(TWARN, NULL, "\tSaw %d, expected %d", count,
-			 NCHILD);
-	}
-
-	/* Check for core file in test directory. */
-
-	if (access("core", 0) == 0) {
-		tst_brkm(TWARN, NULL, "\tCore file found in test directory.");
-	}
-
-	/* Remove expected files */
-
-	for (j = 0; j < nfiles; j += NCHILD) {
-		sprintf(tmpdir, DIR_NAME, j);
-		if (rmdir(tmpdir) < 0) {
-			tst_brkm(TWARN,
-				 NULL,
-				 "\tError removing expected directory, ERRNO = %d",
-				 errno);
-		}
-	}
-
-	tst_resm(TPASS, "PASS");
-
-	return 0;
-}
-
-int getchild(int group, int child, int children)
-{
-	int pid;
-
-	pid = FORK_OR_VFORK();
-
-	if (pid < 0) {
-
-		massmurder();	/* kill the kids */
-		tst_brkm(TBROK, cleanup,
-			 "\tFork failed (may be OK if under stress)");
-	} else if (pid == 0) {	/* child does this */
-		switch (children % NCHILD) {
-		case 0:
-			dochild1();	/* create existing directories */
-			break;	/* so lint won't complain */
-		case 1:
-			dochild2();	/* remove nonexistant directories */
-			break;
-		case 2:
-			dochild3(group);	/* create/delete directories */
-			break;
-		default:
-			tst_brkm(TFAIL, cleanup,
-				 "Test not inplemented for child %d", child);
-			exit(1);
-			break;
-		}
-		exit(1);	/* If child gets here, something wrong */
-	}
-	pidlist[children] = pid;
-	return 0;
-}
-
-void term(int sig)
-{
-	/* Routine to handle SIGTERM signal. */
-
-	if (parent_pid == getpid()) {
-		tst_brkm(TWARN, NULL, "\tsignal SIGTERM received by parent.");
-	}
-	sigterm++;
-	if (jump) {
-		longjmp(env_buf, 1);
-	}
-}
-
-void chld(int sig)
-{
-	/* Routine to handle SIGCHLD signal. */
-
-	sigchld++;
-	if (jump) {
-		longjmp(env_buf, 1);
-	}
-}
-
-int dochild1(void)
-{
-	/* Child routine which attempts to create directories in the test
-	 * directory that already exist. Runs until a SIGTERM signal is
-	 * received. Will exit with an error if it is able to create the
-	 * directory or if the expected error is not received.
-	 */
-
-	int j;
-	char tmpdir[MAXPATHLEN];
-
-	while (!sigterm) {
-		for (j = 0; j < nfiles; j += NCHILD) {
-			sprintf(tmpdir, DIR_NAME, j);
-			TEST(mkdir(tmpdir, MODE_RWX));
-
-			if (TEST_RETURN < 0) {
-
-				if (TEST_ERRNO != EEXIST) {
-					tst_brkm(TFAIL, cleanup,
-						 "MKDIR %s, errno = %d; Wrong error detected.",
-						 tmpdir, TEST_ERRNO);
-					exit(1);
-				}
-			} else {
-				tst_brkm(TFAIL, cleanup,
-					 "MKDIR %s succeded when it shoud have failed.",
-					 tmpdir);
-				exit(1);
-			}
-		}
-	}
-	exit(0);
-}
-
-int dochild2(void)
-{
-	/* Child routine which attempts to remove directories from the
-	 * test directory which do not exist. Runs until a SIGTERM
-	 * signal is received. Exits with an error if the proper
-	 * error is not detected or if the remove operation is
-	 * successful.
-	 */
-
-	int j;
-	char tmpdir[MAXPATHLEN];
-
-	while (!sigterm) {
-		for (j = 1; j < nfiles; j += NCHILD) {
-			sprintf(tmpdir, DIR_NAME, j);
-			if (rmdir(tmpdir) < 0) {
-				if (errno != ENOENT) {
-					tst_brkm(TFAIL, cleanup,
-						 "RMDIR %s, errno = %d; Wrong error detected.",
-						 tmpdir, errno);
-					exit(1);
-				}
-			} else {
-				tst_brkm(TFAIL, cleanup,
-					 "RMDIR %s succeded when it should have failed.",
-					 tmpdir);
-				exit(1);
-			}
-		}
-	}
-	exit(0);
-	return 0;
-}
-
-int dochild3(int group)
-{
-	/* Child routine which creates and deletes directories in the
-	 * test directory. Runs until a SIGTERM signal is received, then
-	 * cleans up and exits. Detects error if the expected condition
-	 * is not encountered.
-	 */
-
-	int j;
-
-	char tmpdir[MAXPATHLEN];
-	char tmp[MAXPATHLEN];
-
-	while (!sigterm) {
-		for (j = 2; j < nfiles; j += NCHILD) {
-			strcpy(tmp, DIR_NAME);
-			strcat(tmp, ".%d");
-			sprintf(tmpdir, tmp, j, group);
-
-			TEST(mkdir(tmpdir, MODE_RWX));
-
-			if (TEST_RETURN < 0) {
-				tst_brkm(TFAIL, cleanup,
-					 "MKDIR %s, errno = %d; Wrong error detected.",
-					 tmpdir, TEST_ERRNO);
-				exit(1);
-			}
-		}
-		for (j = 2; j < nfiles; j += NCHILD) {
-			strcpy(tmp, DIR_NAME);
-			strcat(tmp, ".%d");
-			sprintf(tmpdir, tmp, j, group);
-			if (rmdir(tmpdir) < 0) {
-				tst_brkm(TFAIL, cleanup,
-					 "RMDIR %s, errno = %d; Wrong error detected.",
-					 tmpdir, errno);
-				exit(1);
-			}
-		}
-	}
-	exit(0);
-}
-
-int massmurder(void)
-{
-	register int j;
-	for (j = 0; j < child_count; j++) {
-		if (pidlist[j] > 0) {
-			if (kill(pidlist[j], SIGTERM) < 0) {
-				tst_brkm(TFAIL, cleanup,
-					 "Error killing child %d, ERRNO = %d",
-					 j, errno);
-			}
-		}
-	}
-	return 0;
-}
-
-void setup(void)
-{
-	tst_sig(NOFORK, DEF_HANDLER, cleanup);
-
-	TEST_PAUSE;
-
-	tst_tmpdir();
-}
-
-void cleanup(void)
-{
-	tst_rmdir();
-}
+// SPDX-License-Identifier: GPL-2.0-or-later
+/* Copyright (c) International Business Machines Corp., 2001
+ * Copyright (c) Linux Test Project, 2022
+ */
+
+/*
+ * DESCRIPTION
+ *  Create multiple processes which create subdirectories in the
+ *  same directory multiple times within test time.
+ */
+
+#include <stdio.h>
+#include <sys/param.h>
+#include "tst_test.h"
+#include "tst_safe_pthread.h"
+
+#define MODE_RWX	07770
+#define DIR_NAME	"./X.%d"
+#define DIR_NAME_GROUP	"./X.%d.%d"
+#define NCHILD		3
+
+static int child_groups = 2;
+static int test_time = 1;
+static int nfiles = 5;
+static volatile int done;
+
+static void test1(int child_num)
+{
+	/*
+	 * Routine which attempts to create directories in the test
+	 * directory that already exist.
+	 */
+	int j, first_loop = 1;
+	char tmpdir[MAXPATHLEN];
+
+	while (!done) {
+		for (j = 0; j < nfiles; j += NCHILD) {
+			sprintf(tmpdir, DIR_NAME, j);
+			TEST(mkdir(tmpdir, MODE_RWX));
+
+			/* use first loop to create dirs, skip checks */
+			if (first_loop)
+				continue;
+
+			if (TST_RET == 0) {
+				tst_res(TFAIL, "[%d] mkdir %s succeeded unexpectedly",
+					child_num, tmpdir);
+				return;
+			}
+
+			if (TST_ERR != EEXIST) {
+				tst_res(TFAIL|TERRNO,
+					"[%d] MKDIR %s expected EEXIST",
+					child_num, tmpdir);
+				return;
+			}
+		}
+		first_loop = 0;
+	}
+	tst_res(TPASS, "[%d] create dirs that already exist", child_num);
+}
+
+static void test2(int child_num)
+{
+	/*
+	 * Child routine which attempts to remove directories from the
+	 * test directory which do not exist.
+	 */
+
+	int j;
+	char tmpdir[MAXPATHLEN];
+
+	while (!done) {
+		for (j = 1; j < nfiles; j += NCHILD) {
+			sprintf(tmpdir, DIR_NAME, j);
+			TEST(rmdir(tmpdir));
+
+			if (TST_RET == 0) {
+				tst_res(TFAIL, "[%d] rmdir %s succeeded unexpectedly",
+					 child_num, tmpdir);
+				return;
+			}
+
+			if (TST_ERR != ENOENT) {
+				tst_res(TFAIL|TERRNO, "[%d] rmdir %s",
+					child_num, tmpdir);
+				return;
+			}
+		}
+	}
+	tst_res(TPASS, "[%d] create dirs that dont exist", child_num);
+}
+
+static void test3(int child_num)
+{
+	/*
+	 * Child routine which creates and deletes directories in the
+	 * test directory.
+	 */
+
+	int j;
+	char tmpdir[MAXPATHLEN];
+
+	while (!done) {
+		for (j = 2; j < nfiles; j += NCHILD) {
+			sprintf(tmpdir, DIR_NAME_GROUP, j, child_num / NCHILD);
+
+			TEST(mkdir(tmpdir, MODE_RWX));
+			if (TST_RET < 0) {
+				tst_res(TFAIL | TERRNO, "[%d] mkdir %s",
+					child_num, tmpdir);
+				return;
+			}
+		}
+		for (j = 2; j < nfiles; j += NCHILD) {
+			sprintf(tmpdir, DIR_NAME_GROUP, j, child_num / NCHILD);
+			TEST(rmdir(tmpdir));
+			if (TST_RET < 0) {
+				tst_res(TFAIL | TERRNO, "[%d] rmdir %s",
+					child_num, tmpdir);
+				return;
+			}
+		}
+	}
+	tst_res(TPASS, "[%d] create/remove dirs", child_num);
+}
+
+
+static void *child_thread_func(void *arg)
+{
+	void (*tests[NCHILD])(int) = { test1, test2, test3 };
+	int child_num = (long)arg;
+
+	tests[child_num % NCHILD](child_num);
+
+	/* if any thread failed, make other finish as well */
+	done = 1;
+
+	return NULL;
+}
+
+
+static void verify_mkdir(void)
+{
+	pthread_t child_thread[NCHILD * child_groups];
+	long i;
+
+	done = 0;
+	for (i = 0; i < child_groups * NCHILD; i++) {
+		SAFE_PTHREAD_CREATE(&child_thread[i], NULL,
+			child_thread_func, (void *)i);
+	}
+
+	sleep(test_time);
+	done = 1;
+
+	for (i = 0; i < child_groups * NCHILD; i++)
+		SAFE_PTHREAD_JOIN(child_thread[i], NULL);
+}
+
+static void setup(void)
+{
+}
+
+static struct tst_test test = {
+	.test_all = verify_mkdir,
+	.needs_tmpdir = 1,
+	.setup = setup,
+};