[LTP] [PATCH v2] mem: Convert shmt02 to new API

[Ricardo B. Marlière]

From: Ricardo B. Marlière <rbm@suse.com>

Signed-off-by: Ricardo B. Marlière <rbm@suse.com>
---
Changes in v2:
- Use tst_rand_data and TST_EXP_EQ_STRN()
- Link to v1: https://lore.kernel.org/r/20250618-conversions-shmt-v1-1-5b935f7859ad@suse.com
---
 testcases/kernel/mem/shmt/shmt02.c | 120 +++++++------------------------------
 1 file changed, 21 insertions(+), 99 deletions(-)

diff --git a/testcases/kernel/mem/shmt/shmt02.c b/testcases/kernel/mem/shmt/shmt02.c
index b33a042f93625cf44af1c3de6354248919f21522..e944acece9d76356488a745333754619daedf603 100644
--- a/testcases/kernel/mem/shmt/shmt02.c
+++ b/testcases/kernel/mem/shmt/shmt02.c
@@ -1,120 +1,42 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
 /*
- *
- *   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
+ * Copyright (c) International Business Machines  Corp., 2002
+ *	12/20/2002	Port to LTP	robbiew@us.ibm.com
+ *	06/30/2001	Port to Linux	nsharoff@us.ibm.com
+ * Copyright (c) 2025 SUSE LLC Ricardo B. Marlière <rbm@suse.com>
  */
 
-/* 12/20/2002		 Port to LTP		 robbiew@us.ibm.com */
-/* 06/30/2001		 Port to Linux		 nsharoff@us.ibm.com */
-
-/*
- * NAME
- *		 shmt02
- *
- * CALLS
- *		 shmctl(2) shmget(2)
- *
- * ALGORITHM
+/*\
  * Create and attach a shared memory segment, write to it
- * and then remove it.		  Verify that the shared memory segment
+ * and then remove it. Verify that the shared memory segment
  * is accessible as long as the process is still alive.
- *
  */
 
-#include <stdio.h>
-#include <sys/types.h>
-#include <sys/ipc.h>
-#include <sys/shm.h>
-#include <sys/utsname.h>
-#include <errno.h>
-
-/** LTP Port **/
-#include "test.h"
-
-char *TCID = "shmt02";		/* Test program identifier.    */
-int TST_TOTAL = 3;		/* Total number of test cases. */
-
-/**************/
+#include "tst_test.h"
+#include "tst_safe_sysv_ipc.h"
+#include "tst_rand_data.h"
 
-#define K_1 1024
+#define SHMSIZE 16
 
-static int rm_shm(int);
-
-int main(void)
+static void run(void)
 {
 	register int shmid;
 	char *cp;
 	key_t key;
 
 	errno = 0;
-	key = (key_t) getpid();
-
-/*----------------------------------------------------------------*/
-
-	if ((shmid = shmget(key, 16 * K_1, IPC_CREAT | 0666)) < 0) {
-		perror("shmget");
-		tst_brkm(TFAIL, NULL,
-			 "shmget Failed: shmid = %d, errno = %d",
-			 shmid, errno);
-	}
-
-	tst_resm(TPASS, "shmget");
+	key = (key_t)getpid();
 
-/*----------------------------------------------------------------*/
+	shmid = SAFE_SHMGET(key, SHMSIZE, IPC_CREAT | 0666);
 
-	cp = shmat(shmid, NULL, 0);
+	cp = SAFE_SHMAT(shmid, NULL, 0);
+	memcpy(cp, tst_rand_data, SHMSIZE);
 
-	if (cp == (char *)-1) {
-		perror("shmat");
-		tst_resm(TFAIL, "shmat Failed: shmid = %d, errno = %d",
-			 shmid, errno);
-		rm_shm(shmid);
-		tst_exit();
-	}
+	SAFE_SHMCTL(shmid, IPC_RMID, NULL);
 
-	*cp = '1';
-	*(cp + 1) = '2';
-
-	tst_resm(TPASS, "shmat");
-
-/*----------------------------------------------------------------*/
-
-	rm_shm(shmid);
-
-	if (*cp != '1' || *(cp + 1) != '2') {
-		tst_resm(TFAIL,
-			 "Error in shared memory contents: shmid = %d",
-			 shmid);
-	}
-
-	tst_resm(TPASS, "Correct shared memory contents");
-
-/*------------------------------------------------------------------*/
-
-	tst_exit();
+	TST_EXP_EQ_STRN(cp, tst_rand_data, SHMSIZE);
 }
 
-static int rm_shm(int shmid)
-{
-	if (shmctl(shmid, IPC_RMID, NULL) == -1) {
-		perror("shmctl");
-		tst_brkm(TFAIL,
-			 NULL,
-			 "shmctl Failed to remove: shmid = %d, errno = %d",
-			 shmid, errno);
-	}
-	return (0);
-}
+static struct tst_test test = {
+	.test_all = run,
+};

---
base-commit: df591113afeb31107bc45bd5ba28a99b556d1028
change-id: 20250618-conversions-shmt-5ea5a5ebe55e

Best regards,
-- 
Ricardo B. Marlière <rbm@suse.com>