diff mbox series

[v2] Add io_read_eof in common.h utilities

Message ID 20211213121417.21825-1-andrea.cervesato@suse.com
State Accepted
Headers show
Series [v2] Add io_read_eof in common.h utilities | expand

Commit Message

Andrea Cervesato Dec. 13, 2021, 12:14 p.m. UTC
Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
---
 testcases/kernel/io/ltp-aiodio/common.h | 30 +++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

Comments

Petr Vorel Dec. 14, 2021, 4:40 p.m. UTC | #1
Hi,

...
> +static inline void io_read_eof(const char *filename, volatile int *run_child)
> +{
> +	char buff[4096];
> +	int fd;
> +	int r;
> +
> +	while ((fd = open(filename, O_RDONLY, 0666)) < 0)
Just for a record [1] this open should be checked for errno == EINVAL
in case of /tmp/ being tmpfs.

Kind regards,
Petr

[1] https://lore.kernel.org/ltp/YbjIqypOcAYAnLbl@pevik/T/#t
> +		usleep(100);
> +

Kind regards,
Petr
Cyril Hrubis Dec. 15, 2021, 1:06 p.m. UTC | #2
Hi!
Pushed, thanks.
diff mbox series

Patch

diff --git a/testcases/kernel/io/ltp-aiodio/common.h b/testcases/kernel/io/ltp-aiodio/common.h
index c9fd0bbaa..c27c3a0ad 100644
--- a/testcases/kernel/io/ltp-aiodio/common.h
+++ b/testcases/kernel/io/ltp-aiodio/common.h
@@ -86,6 +86,36 @@  static inline void io_read(const char *filename, int filesize, volatile int *run
 	SAFE_CLOSE(fd);
 }
 
+static inline void io_read_eof(const char *filename, volatile int *run_child)
+{
+	char buff[4096];
+	int fd;
+	int r;
+
+	while ((fd = open(filename, O_RDONLY, 0666)) < 0)
+		usleep(100);
+
+	tst_res(TINFO, "child %i reading file", getpid());
+
+	while (*run_child) {
+		off_t offset;
+		char *bufoff;
+
+		offset = SAFE_LSEEK(fd, SEEK_END, 0);
+
+		r = SAFE_READ(0, fd, buff, sizeof(buff));
+		if (r > 0) {
+			bufoff = check_zero(buff, r);
+			if (bufoff) {
+				tst_res(TINFO, "non-zero read at offset %p", offset + bufoff);
+				break;
+			}
+		}
+	}
+
+	SAFE_CLOSE(fd);
+}
+
 /*
  * This code tries to create dirty free blocks on
  * the HDD so there is a chance that blocks to be allocated