Module Name: src
Committed By: riastradh
Date: Sat Aug 10 15:10:17 UTC 2024
Modified Files:
src/tests/lib/libc/c063: t_utimensat.c
Log Message:
t_utimensat: Tidy this up to make failures more obvious.
PR kern/58571: utimensat tests are failing on armv7, aarch64, and
riscv64
To generate a diff of this commit:
cvs rdiff -u -r1.7 -r1.8 src/tests/lib/libc/c063/t_utimensat.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/tests/lib/libc/c063/t_utimensat.c
diff -u src/tests/lib/libc/c063/t_utimensat.c:1.7 src/tests/lib/libc/c063/t_utimensat.c:1.8
--- src/tests/lib/libc/c063/t_utimensat.c:1.7 Wed Jul 10 19:23:56 2024
+++ src/tests/lib/libc/c063/t_utimensat.c Sat Aug 10 15:10:17 2024
@@ -1,4 +1,4 @@
-/* $NetBSD: t_utimensat.c,v 1.7 2024/07/10 19:23:56 rillig Exp $ */
+/* $NetBSD: t_utimensat.c,v 1.8 2024/08/10 15:10:17 riastradh Exp $ */
/*-
* Copyright (c) 2012 The NetBSD Foundation, Inc.
@@ -29,7 +29,7 @@
* POSSIBILITY OF SUCH DAMAGE.
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: t_utimensat.c,v 1.7 2024/07/10 19:23:56 rillig Exp $");
+__RCSID("$NetBSD: t_utimensat.c,v 1.8 2024/08/10 15:10:17 riastradh Exp $");
#include <sys/param.h>
#include <sys/stat.h>
@@ -43,6 +43,8 @@ __RCSID("$NetBSD: t_utimensat.c,v 1.7 20
#include <string.h>
#include <unistd.h>
+#include "h_macros.h"
+
#define DIR "dir"
#define FILE "dir/utimensat"
#define BASEFILE "utimensat"
@@ -50,11 +52,29 @@ __RCSID("$NetBSD: t_utimensat.c,v 1.7 20
#define BASELINK "symlink"
#define FILEERR "dir/symlink"
-const struct timespec tptr[] = {
+static const struct timespec tptr[] = {
{ 0x12345678, 987654321 },
{ 0x15263748, 123456789 },
};
+static void
+checkstattime(const struct stat *st)
+{
+
+ ATF_CHECK_EQ_MSG(st->st_atimespec.tv_sec, tptr[0].tv_sec,
+ "st->st_atimespec.tv_sec=%lld tptr[0].tv_sec=%lld",
+ (long long)st->st_atimespec.tv_sec, (long long)tptr[0].tv_sec);
+ ATF_CHECK_EQ_MSG(st->st_atimespec.tv_nsec, tptr[0].tv_nsec,
+ "st->st_atimespec.tv_nsec=%ld tptr[0].tv_nsec=%ld",
+ (long)st->st_atimespec.tv_nsec, (long)tptr[0].tv_nsec);
+ ATF_CHECK_EQ_MSG(st->st_mtimespec.tv_sec, tptr[1].tv_sec,
+ "st->st_mtimespec.tv_sec=%lld tptr[1].tv_sec=%lld",
+ (long long)st->st_mtimespec.tv_sec, (long long)tptr[1].tv_sec);
+ ATF_CHECK_EQ_MSG(st->st_mtimespec.tv_nsec, tptr[1].tv_nsec,
+ "st->st_mtimespec.tv_nsec=%ld tptr[1].tv_nsec=%ld",
+ (long)st->st_mtimespec.tv_nsec, (long)tptr[1].tv_nsec);
+}
+
ATF_TC(utimensat_fd);
ATF_TC_HEAD(utimensat_fd, tc)
{
@@ -66,77 +86,73 @@ ATF_TC_BODY(utimensat_fd, tc)
int fd;
struct stat st;
- ATF_REQUIRE(mkdir(DIR, 0755) == 0);
- ATF_REQUIRE((fd = open(FILE, O_CREAT|O_RDWR, 0644)) != -1);
- ATF_REQUIRE(close(fd) == 0);
-
- ATF_REQUIRE((dfd = open(DIR, O_RDONLY, 0)) != -1);
- ATF_REQUIRE(utimensat(dfd, BASEFILE, tptr, 0) == 0);
- ATF_REQUIRE(close(dfd) == 0);
-
- ATF_REQUIRE(stat(FILE, &st) == 0);
- ATF_REQUIRE(st.st_atimespec.tv_sec == tptr[0].tv_sec);
- ATF_REQUIRE(st.st_atimespec.tv_nsec == tptr[0].tv_nsec);
- ATF_REQUIRE(st.st_mtimespec.tv_sec == tptr[1].tv_sec);
- ATF_REQUIRE(st.st_mtimespec.tv_nsec == tptr[1].tv_nsec);
+ RL(mkdir(DIR, 0755));
+ RL(fd = open(FILE, O_CREAT|O_RDWR, 0644));
+ RL(close(fd));
+
+ RL(dfd = open(DIR, O_RDONLY, 0));
+ RL(utimensat(dfd, BASEFILE, tptr, 0));
+ RL(close(dfd));
+
+ RL(stat(FILE, &st));
+ checkstattime(&st);
}
ATF_TC(utimensat_fdcwd);
ATF_TC_HEAD(utimensat_fdcwd, tc)
{
- atf_tc_set_md_var(tc, "descr",
- "See that utimensat works with fd as AT_FDCWD");
+ atf_tc_set_md_var(tc, "descr",
+ "See that utimensat works with fd as AT_FDCWD");
}
ATF_TC_BODY(utimensat_fdcwd, tc)
{
int fd;
struct stat st;
- ATF_REQUIRE(mkdir(DIR, 0755) == 0);
- ATF_REQUIRE((fd = open(FILE, O_CREAT|O_RDWR, 0644)) != -1);
- ATF_REQUIRE(close(fd) == 0);
-
- ATF_REQUIRE(chdir(DIR) == 0);
- ATF_REQUIRE(utimensat(AT_FDCWD, BASEFILE, tptr, 0) == 0);
-
- ATF_REQUIRE(stat(BASEFILE, &st) == 0);
- ATF_REQUIRE(st.st_atimespec.tv_sec == tptr[0].tv_sec);
- ATF_REQUIRE(st.st_atimespec.tv_nsec == tptr[0].tv_nsec);
- ATF_REQUIRE(st.st_mtimespec.tv_sec == tptr[1].tv_sec);
- ATF_REQUIRE(st.st_mtimespec.tv_nsec == tptr[1].tv_nsec);
+ RL(mkdir(DIR, 0755));
+ RL(fd = open(FILE, O_CREAT|O_RDWR, 0644));
+ RL(close(fd));
+
+ RL(chdir(DIR));
+ RL(utimensat(AT_FDCWD, BASEFILE, tptr, 0));
+
+ RL(stat(BASEFILE, &st));
+ checkstattime(&st);
}
ATF_TC(utimensat_fdcwderr);
ATF_TC_HEAD(utimensat_fdcwderr, tc)
{
- atf_tc_set_md_var(tc, "descr",
- "See that utimensat fails with fd as AT_FDCWD and bad path");
+ atf_tc_set_md_var(tc, "descr",
+ "See that utimensat fails with fd as AT_FDCWD and bad path");
}
ATF_TC_BODY(utimensat_fdcwderr, tc)
{
- ATF_REQUIRE(mkdir(DIR, 0755) == 0);
- ATF_REQUIRE(utimensat(AT_FDCWD, FILEERR, tptr, 0) == -1);
+ RL(mkdir(DIR, 0755));
+ ATF_CHECK_ERRNO(ENOENT, utimensat(AT_FDCWD, FILEERR, tptr, 0) == -1);
}
ATF_TC(utimensat_fderr1);
ATF_TC_HEAD(utimensat_fderr1, tc)
{
- atf_tc_set_md_var(tc, "descr", "See that utimensat fail with bad path");
+ atf_tc_set_md_var(tc, "descr",
+ "See that utimensat fail with bad path");
}
ATF_TC_BODY(utimensat_fderr1, tc)
{
int dfd;
- ATF_REQUIRE(mkdir(DIR, 0755) == 0);
- ATF_REQUIRE((dfd = open(DIR, O_RDONLY, 0)) != -1);
- ATF_REQUIRE(utimensat(dfd, FILEERR, tptr, 0) == -1);
- ATF_REQUIRE(close(dfd) == 0);
+ RL(mkdir(DIR, 0755));
+ RL(dfd = open(DIR, O_RDONLY, 0));
+ ATF_CHECK_ERRNO(ENOENT, utimensat(dfd, FILEERR, tptr, 0) == -1);
+ RL(close(dfd));
}
ATF_TC(utimensat_fderr2);
ATF_TC_HEAD(utimensat_fderr2, tc)
{
- atf_tc_set_md_var(tc, "descr", "See that utimensat fails with bad fdat");
+ atf_tc_set_md_var(tc, "descr",
+ "See that utimensat fails with bad fdat");
}
ATF_TC_BODY(utimensat_fderr2, tc)
{
@@ -144,29 +160,30 @@ ATF_TC_BODY(utimensat_fderr2, tc)
int fd;
char cwd[MAXPATHLEN];
- ATF_REQUIRE(mkdir(DIR, 0755) == 0);
- ATF_REQUIRE((fd = open(FILE, O_CREAT|O_RDWR, 0644)) != -1);
- ATF_REQUIRE(close(fd) == 0);
-
- ATF_REQUIRE((dfd = open(getcwd(cwd, MAXPATHLEN), O_RDONLY, 0)) != -1);
- ATF_REQUIRE(utimensat(dfd, BASEFILE, tptr, 0) == -1);
- ATF_REQUIRE(close(dfd) == 0);
+ RL(mkdir(DIR, 0755));
+ RL(fd = open(FILE, O_CREAT|O_RDWR, 0644));
+ RL(close(fd));
+
+ RL(dfd = open(getcwd(cwd, MAXPATHLEN), O_RDONLY, 0));
+ ATF_CHECK_ERRNO(ENOENT, utimensat(dfd, BASEFILE, tptr, 0) == -1);
+ RL(close(dfd));
}
ATF_TC(utimensat_fderr3);
ATF_TC_HEAD(utimensat_fderr3, tc)
{
- atf_tc_set_md_var(tc, "descr", "See that utimensat fails with fd as -1");
+ atf_tc_set_md_var(tc, "descr",
+ "See that utimensat fails with fd as -1");
}
ATF_TC_BODY(utimensat_fderr3, tc)
{
int fd;
- ATF_REQUIRE(mkdir(DIR, 0755) == 0);
- ATF_REQUIRE((fd = open(FILE, O_CREAT|O_RDWR, 0644)) != -1);
- ATF_REQUIRE(close(fd) == 0);
+ RL(mkdir(DIR, 0755));
+ RL(fd = open(FILE, O_CREAT|O_RDWR, 0644));
+ RL(close(fd));
- ATF_REQUIRE(utimensat(-1, FILE, tptr, 0) == -1);
+ ATF_CHECK_ERRNO(EBADF, utimensat(-1, FILE, tptr, 0) == -1);
}
ATF_TC(utimensat_fdlink);
@@ -179,23 +196,19 @@ ATF_TC_BODY(utimensat_fdlink, tc)
int dfd;
struct stat st;
- ATF_REQUIRE(mkdir(DIR, 0755) == 0);
- ATF_REQUIRE(symlink(FILE, LINK) == 0); /* NB: FILE does not exists */
+ RL(mkdir(DIR, 0755));
+ RL(symlink(FILE, LINK)); /* NB: FILE does not exists */
- ATF_REQUIRE((dfd = open(DIR, O_RDONLY, 0)) != -1);
+ RL(dfd = open(DIR, O_RDONLY, 0));
- ATF_REQUIRE(utimensat(dfd, BASELINK, tptr, 0) == -1);
- ATF_REQUIRE(errno == ENOENT);
+ ATF_CHECK_ERRNO(ENOENT, utimensat(dfd, BASELINK, tptr, 0) == -1);
- ATF_REQUIRE(utimensat(dfd, BASELINK, tptr, AT_SYMLINK_NOFOLLOW) == 0);
+ RL(utimensat(dfd, BASELINK, tptr, AT_SYMLINK_NOFOLLOW));
- ATF_REQUIRE(close(dfd) == 0);
+ RL(close(dfd));
- ATF_REQUIRE(lstat(LINK, &st) == 0);
- ATF_REQUIRE(st.st_atimespec.tv_sec == tptr[0].tv_sec);
- ATF_REQUIRE(st.st_atimespec.tv_nsec == tptr[0].tv_nsec);
- ATF_REQUIRE(st.st_mtimespec.tv_sec == tptr[1].tv_sec);
- ATF_REQUIRE(st.st_mtimespec.tv_nsec == tptr[1].tv_nsec);
+ RL(lstat(LINK, &st));
+ checkstattime(&st);
}
ATF_TP_ADD_TCS(tp)
