Module Name: src
Committed By: riastradh
Date: Sat Aug 10 15:20:23 UTC 2024
Modified Files:
src/tests/lib/libc/c063: t_utimensat.c
Log Message:
t_utimensat: Don't check atime on file systems mounted noatime.
PR kern/58571: utimensat tests are failing on armv7, aarch64, and
riscv64
To generate a diff of this commit:
cvs rdiff -u -r1.8 -r1.9 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.8 src/tests/lib/libc/c063/t_utimensat.c:1.9
--- src/tests/lib/libc/c063/t_utimensat.c:1.8 Sat Aug 10 15:10:17 2024
+++ src/tests/lib/libc/c063/t_utimensat.c Sat Aug 10 15:20:22 2024
@@ -1,4 +1,4 @@
-/* $NetBSD: t_utimensat.c,v 1.8 2024/08/10 15:10:17 riastradh Exp $ */
+/* $NetBSD: t_utimensat.c,v 1.9 2024/08/10 15:20:22 riastradh Exp $ */
/*-
* Copyright (c) 2012 The NetBSD Foundation, Inc.
@@ -29,11 +29,14 @@
* POSSIBILITY OF SUCH DAMAGE.
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: t_utimensat.c,v 1.8 2024/08/10 15:10:17 riastradh Exp $");
+__RCSID("$NetBSD: t_utimensat.c,v 1.9 2024/08/10 15:20:22 riastradh Exp $");
#include <sys/param.h>
+
#include <sys/stat.h>
+#include <sys/statvfs.h>
#include <sys/time.h>
+
#include <atf-c.h>
#include <errno.h>
#include <fcntl.h>
@@ -58,15 +61,18 @@ static const struct timespec tptr[] = {
};
static void
-checkstattime(const struct stat *st)
+checkstattime(const struct stat *st, const struct statvfs *fs)
{
- 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);
+ if ((fs->f_flag & ST_NOATIME) == 0) {
+ 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);
@@ -85,6 +91,7 @@ ATF_TC_BODY(utimensat_fd, tc)
int dfd;
int fd;
struct stat st;
+ struct statvfs fs;
RL(mkdir(DIR, 0755));
RL(fd = open(FILE, O_CREAT|O_RDWR, 0644));
@@ -95,7 +102,8 @@ ATF_TC_BODY(utimensat_fd, tc)
RL(close(dfd));
RL(stat(FILE, &st));
- checkstattime(&st);
+ RL(statvfs(FILE, &fs));
+ checkstattime(&st, &fs);
}
ATF_TC(utimensat_fdcwd);
@@ -108,6 +116,7 @@ ATF_TC_BODY(utimensat_fdcwd, tc)
{
int fd;
struct stat st;
+ struct statvfs fs;
RL(mkdir(DIR, 0755));
RL(fd = open(FILE, O_CREAT|O_RDWR, 0644));
@@ -117,7 +126,8 @@ ATF_TC_BODY(utimensat_fdcwd, tc)
RL(utimensat(AT_FDCWD, BASEFILE, tptr, 0));
RL(stat(BASEFILE, &st));
- checkstattime(&st);
+ RL(statvfs(BASEFILE, &fs));
+ checkstattime(&st, &fs);
}
ATF_TC(utimensat_fdcwderr);
@@ -195,6 +205,7 @@ ATF_TC_BODY(utimensat_fdlink, tc)
{
int dfd;
struct stat st;
+ struct statvfs fs;
RL(mkdir(DIR, 0755));
RL(symlink(FILE, LINK)); /* NB: FILE does not exists */
@@ -208,7 +219,8 @@ ATF_TC_BODY(utimensat_fdlink, tc)
RL(close(dfd));
RL(lstat(LINK, &st));
- checkstattime(&st);
+ RL(statvfs(DIR, &fs)); /* XXX should do lstatvfs(LINK, &fs) */
+ checkstattime(&st, &fs);
}
ATF_TP_ADD_TCS(tp)