[LTP] [PATCH v7 4/5] Use termios instead of legacy struct in ioctl02.c
Marius Kittler
mkittler@suse.de
Mon Oct 23 18:02:41 CEST 2023
Signed-off-by: Marius Kittler <mkittler@suse.de>
---
testcases/kernel/syscalls/ioctl/ioctl02.c | 24 +++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/testcases/kernel/syscalls/ioctl/ioctl02.c b/testcases/kernel/syscalls/ioctl/ioctl02.c
index c66473a8d..5111d7178 100644
--- a/testcases/kernel/syscalls/ioctl/ioctl02.c
+++ b/testcases/kernel/syscalls/ioctl/ioctl02.c
@@ -8,17 +8,17 @@
/*\
* [Description]
*
- * Testcase to test the TCGETA, and TCSETA ioctl implementations for
+ * Testcase to test the TCGETS, and TCSETS ioctl implementations for
* the tty driver
*
* In this test, the parent and child open the parentty and the childtty
* respectively. After opening the childtty the child flushes the stream
* and wakes the parent (thereby asking it to continue its testing). The
- * parent, then starts the testing. It issues a TCGETA ioctl to get all
+ * parent, then starts the testing. It issues a TCGETS ioctl to get all
* the tty parameters. It then changes them to known values by issuing a
- * TCSETA ioctl. Then the parent issues a TCGETA ioctl again and compares
+ * TCSETS ioctl. Then the parent issues a TCGETS ioctl again and compares
* the received values with what it had set earlier. The test fails if
- * TCGETA or TCSETA fails, or if the received values don't match those
+ * TCGETS or TCSETS fails, or if the received values don't match those
* that were set. The parent does all the testing, the requirement of the
* child process is to moniter the testing done by the parent, and hence
* the child just waits for the parent.
@@ -38,7 +38,7 @@
#include "tst_test.h"
#include "tst_safe_macros.h"
-static struct termio termio, save_io;
+static struct termios termio, save_io;
static char *parenttty, *childtty;
static int parentfd = -1, childfd = -1;
@@ -82,7 +82,7 @@ static void verify_ioctl(void)
/*
* run_ptest() - setup the various termio structure values and issue
- * the TCSETA ioctl call with the TEST macro.
+ * the TCSETS ioctl call with the TEST macro.
*/
static void run_ptest(void)
{
@@ -111,10 +111,10 @@ static void run_ptest(void)
/* Set output modes. */
termio.c_oflag = OPOST | OLCUC | ONLCR | ONOCR;
- SAFE_IOCTL(parentfd, TCSETA, &termio);
+ SAFE_IOCTL(parentfd, TCSETS, &termio);
/* Get termio and see if all parameters actually got set */
- SAFE_IOCTL(parentfd, TCGETA, &termio);
+ SAFE_IOCTL(parentfd, TCGETS, &termio);
chk_tty_parms();
}
@@ -175,10 +175,10 @@ static void chk_tty_parms(void)
}
if (flag != 0)
- tst_res(TFAIL, "TCGETA/TCSETA tests FAILED with "
+ tst_res(TFAIL, "TCGETS/TCSETS tests FAILED with "
"%d %s", flag, flag > 1 ? "errors" : "error");
else
- tst_res(TPASS, "TCGETA/TCSETA tests SUCCEEDED");
+ tst_res(TPASS, "TCGETS/TCSETS tests SUCCEEDED");
}
static int do_child_setup(void)
@@ -202,7 +202,7 @@ static void setup(void)
int fd = SAFE_OPEN(device, O_RDWR, 0777);
/* Save the current device information - to be restored in cleanup() */
- SAFE_IOCTL(fd, TCGETA, &save_io);
+ SAFE_IOCTL(fd, TCGETS, &save_io);
/* Close the device */
SAFE_CLOSE(fd);
@@ -211,7 +211,7 @@ static void setup(void)
static void cleanup(void)
{
if (parentfd >= 0) {
- SAFE_IOCTL(parentfd, TCSETA, &save_io);
+ SAFE_IOCTL(parentfd, TCSETS, &save_io);
SAFE_CLOSE(parentfd);
}
}
--
2.42.0
More information about the ltp
mailing list