diff mbox series

De-duplication: Remove requirement of needs_tmpdir

Message ID 20210301022550.96228-1-zhaogongyi@huawei.com
State Accepted
Headers show
Series De-duplication: Remove requirement of needs_tmpdir | expand

Commit Message

Zhao Gongyi March 1, 2021, 2:25 a.m. UTC
The flag is not needed to be set when use these flags: '.all_filesystems',
'.format_device', '.mntpoint', '.mount_device' '.needs_checkpoints',
'.needs_device', '.resource_file'. These flags imply creating temporary
directory.

For those:
	fanotify13.c
	setxattr01.c
	fanotify16.c
	close_range01.c
	clone301.c
	fanotify09.c

Signed-off-by: Zhao Gongyi <zhaogongyi@huawei.com>
---
 testcases/kernel/syscalls/clone3/clone301.c           | 1 -
 testcases/kernel/syscalls/close_range/close_range01.c | 1 -
 testcases/kernel/syscalls/fanotify/fanotify09.c       | 1 -
 testcases/kernel/syscalls/fanotify/fanotify13.c       | 1 -
 testcases/kernel/syscalls/fanotify/fanotify16.c       | 1 -
 testcases/kernel/syscalls/setxattr/setxattr01.c       | 1 -
 6 files changed, 6 deletions(-)

--
2.17.1

Comments

Yang Xu March 1, 2021, 2:57 a.m. UTC | #1
Hi Gongyi
> The flag is not needed to be set when use these flags: '.all_filesystems',
> '.format_device', '.mntpoint', '.mount_device' '.needs_checkpoints',
> '.needs_device', '.resource_file'. These flags imply creating temporary
> directory.
Merged, thanks!
diff mbox series

Patch

diff --git a/testcases/kernel/syscalls/clone3/clone301.c b/testcases/kernel/syscalls/clone3/clone301.c
index bf009e940..11787fc7e 100644
--- a/testcases/kernel/syscalls/clone3/clone301.c
+++ b/testcases/kernel/syscalls/clone3/clone301.c
@@ -172,7 +172,6 @@  static struct tst_test test = {
 	.test = run,
 	.setup = setup,
 	.needs_root = 1,
-	.needs_tmpdir = 1,
 	.needs_checkpoints = 1,
 	.bufs = (struct tst_buffers []) {
 		{&args, .size = sizeof(*args)},
diff --git a/testcases/kernel/syscalls/close_range/close_range01.c b/testcases/kernel/syscalls/close_range/close_range01.c
index 56def2426..6ecd4576e 100644
--- a/testcases/kernel/syscalls/close_range/close_range01.c
+++ b/testcases/kernel/syscalls/close_range/close_range01.c
@@ -190,7 +190,6 @@  static void run(unsigned int n)

 static struct tst_test test = {
 	.tcnt = 4,
-	.needs_tmpdir = 1,
 	.forks_child = 1,
 	.mount_device = 1,
 	.mntpoint = "mnt",
diff --git a/testcases/kernel/syscalls/fanotify/fanotify09.c b/testcases/kernel/syscalls/fanotify/fanotify09.c
index 918e40274..089655d29 100644
--- a/testcases/kernel/syscalls/fanotify/fanotify09.c
+++ b/testcases/kernel/syscalls/fanotify/fanotify09.c
@@ -365,7 +365,6 @@  static struct tst_test test = {
 	.cleanup = cleanup,
 	.mount_device = 1,
 	.mntpoint = MOUNT_PATH,
-	.needs_tmpdir = 1,
 	.needs_root = 1,
 	.tags = (const struct tst_tag[]) {
 		{"linux-git", "54a307ba8d3c"},
diff --git a/testcases/kernel/syscalls/fanotify/fanotify13.c b/testcases/kernel/syscalls/fanotify/fanotify13.c
index 6d812cdd1..1fb12ae68 100644
--- a/testcases/kernel/syscalls/fanotify/fanotify13.c
+++ b/testcases/kernel/syscalls/fanotify/fanotify13.c
@@ -289,7 +289,6 @@  static struct tst_test test = {
 	.setup = do_setup,
 	.cleanup = do_cleanup,
 	.needs_root = 1,
-	.needs_tmpdir = 1,
 	.mount_device = 1,
 	.mntpoint = MOUNT_PATH,
 	.all_filesystems = 1,
diff --git a/testcases/kernel/syscalls/fanotify/fanotify16.c b/testcases/kernel/syscalls/fanotify/fanotify16.c
index c4b8a5abc..801d05cc2 100644
--- a/testcases/kernel/syscalls/fanotify/fanotify16.c
+++ b/testcases/kernel/syscalls/fanotify/fanotify16.c
@@ -562,7 +562,6 @@  static struct tst_test test = {
 	.mount_device = 1,
 	.mntpoint = MOUNT_PATH,
 	.all_filesystems = 1,
-	.needs_tmpdir = 1,
 	.needs_root = 1
 };

diff --git a/testcases/kernel/syscalls/setxattr/setxattr01.c b/testcases/kernel/syscalls/setxattr/setxattr01.c
index 8b84dfe0d..8cd2821d0 100644
--- a/testcases/kernel/syscalls/setxattr/setxattr01.c
+++ b/testcases/kernel/syscalls/setxattr/setxattr01.c
@@ -210,7 +210,6 @@  static struct tst_test test = {
 	.mntpoint = MNTPOINT,
 	.mount_device = 1,
 	.all_filesystems = 1,
-	.needs_tmpdir = 1,
 	.needs_root = 1,
 };