diff mbox series

[3/4] correct order of parameters in SAFE_WRITE()

Message ID 90a93e9b87fda1811a00a35f8a549f2a1125ba0d.1664979933.git.jstancek@redhat.com
State Accepted, archived
Headers show
Series [1/4] mmapstress04: use SAFE_WRITE_RETRY from LTP library | expand

Commit Message

Jan Stancek Oct. 5, 2022, 2:25 p.m. UTC
And use new enum.

Signed-off-by: Jan Stancek <jstancek@redhat.com>
---
 testcases/kernel/containers/pidns/pidns13.c   | 2 +-
 testcases/kernel/containers/userns/userns08.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Cyril Hrubis Oct. 5, 2022, 3:53 p.m. UTC | #1
Hi!
Good catch.

Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
diff mbox series

Patch

diff --git a/testcases/kernel/containers/pidns/pidns13.c b/testcases/kernel/containers/pidns/pidns13.c
index bdc1097fd7d3..39fd6df7f4c4 100644
--- a/testcases/kernel/containers/pidns/pidns13.c
+++ b/testcases/kernel/containers/pidns/pidns13.c
@@ -96,7 +96,7 @@  static void child_fn(unsigned int cinit_no)
 		SAFE_CLOSE(pipe_fd[0]);
 
 		TST_CHECKPOINT_WAIT(1);
-		SAFE_WRITE(pipe_fd[1], 1, "test\n", 5);
+		SAFE_WRITE(SAFE_WRITE_ALL, pipe_fd[1], "test\n", 5);
 	}
 
 	exit(0);
diff --git a/testcases/kernel/containers/userns/userns08.c b/testcases/kernel/containers/userns/userns08.c
index bde944f224ba..0f1cb54a9dfe 100644
--- a/testcases/kernel/containers/userns/userns08.c
+++ b/testcases/kernel/containers/userns/userns08.c
@@ -118,7 +118,7 @@  static void setup(void)
 {
 	int fd = SAFE_OPEN("restricted", O_CREAT | O_WRONLY, 0700);
 
-	SAFE_WRITE(fd, 1, "\n", 1);
+	SAFE_WRITE(SAFE_WRITE_ALL, fd, "\n", 1);
 	SAFE_CLOSE(fd);
 
 	SAFE_TRY_FILE_PRINTF("/proc/sys/user/max_user_namespaces", "%d", 10);