Skip to content

Commit 571e730

Browse files
hpataxisdotcomHans-Peter Nilsson
authored and
Hans-Peter Nilsson
committed
Complete revert of 2019-08-19, st_atime in libc/include/sys/stat.h
The revert-part of the revert-and-fix commit, b99887c a.k.a. "Revert previous change to sys/stat.h and fix cris libgloss", apparently intending to revert f75aa67 a.k.a. "Fix regression in cris-elf caused by sys/stat.h change" and fix it in another way, wasn't complete. Although the fix-part added the prerequisite "#undef st_atime" (et al) to gensyscalls, the revert-part didn't revert the "&& !defined(__cris__)" in sys/stat.h, stopping st_atime (et al) from being defined. The effect of the unreverted change is that accessing the struct stat compatibility member names "st_atime" (et al) as in "struct stat mystat; mystat.st_atime;" yields errors, observable for example when building libgfortran in gcc: /x/gcc/libgfortran/intrinsics/stat.c:114:42: error: 'struct stat' has \ no member named 'st_atime'; did you mean 'st_atim'? 114 | sarray->base_addr[8 * stride] = sb.st_atime; | ^~~~~~~~ | st_atim (etc.) Trivially fixed by completing the reversion, removing the "&& !defined(__cris__)" in sys/stat.h. Beware: the net effect of the earlier related change to struct stat in sys/stat.h, leading up to the fix, *does* change its definition as a type. Thankfully, replacing members like "time_t st_atime; long st_spare1;" by "struct timespec st_atim;", ditto st_mtim and st_ctim, is layout-compatible. To wit, that change is "binary compatible". Incidentally, related to the simulator / Linux ABI, there's a transitional stage (see gensyscalls), reloading between "struct stat" (sys/stat.h) and "struct new_stat" (kernel/simulator) as necessary. Tested by a cris-elf gcc build (including libgfortran).
1 parent 2caca30 commit 571e730

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

newlib/libc/include/sys/stat.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ struct stat
5050
#endif
5151
};
5252

53-
#if !(defined(__svr4__) && !defined(__PPC__) && !defined(__sun__)) && !defined(__cris__)
53+
#if !(defined(__svr4__) && !defined(__PPC__) && !defined(__sun__))
5454
#define st_atime st_atim.tv_sec
5555
#define st_ctime st_ctim.tv_sec
5656
#define st_mtime st_mtim.tv_sec

0 commit comments

Comments
 (0)