tools/nolibc: always use openat(2) instead of open(2)

kragniz · t-8ch · commit fbd4f52f5bf7 · 2025-03-06T22:30:20.000+01:00
All architectures support openat, so we don't need to make its use conditional. Signed-off-by: Louis Taylor <louis@kragniz.eu> Link: https://lore.kernel.org/r/20250306184147.208723-2-louis@kragniz.eu Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
diff --git a/tools/include/nolibc/sys.h b/tools/include/nolibc/sys.h
@@ -798,13 +798,7 @@ int openat(int dirfd, const char *path, int flags, ...)
 static __attribute__((unused))
 int sys_open(const char *path, int flags, mode_t mode)
 {
-#ifdef __NR_openat
 	return my_syscall4(__NR_openat, AT_FDCWD, path, flags, mode);
-#elif defined(__NR_open)
-	return my_syscall3(__NR_open, path, flags, mode);
-#else
-	return __nolibc_enosys(__func__, path, flags, mode);
-#endif
 }
 
 static __attribute__((unused))

Original file line number	Diff line number	Diff line change
`@@ -798,13 +798,7 @@ int openat(int dirfd, const char *path, int flags, ...)`
`798`	`798`	`static __attribute__((unused))`
`799`	`799`	`int sys_open(const char *path, int flags, mode_t mode)`
`800`	`800`	`{`
`801`		`-#ifdef __NR_openat`
`802`	`801`	`return my_syscall4(__NR_openat, AT_FDCWD, path, flags, mode);`
`803`		`-#elif defined(__NR_open)`
`804`		`- return my_syscall3(__NR_open, path, flags, mode);`
`805`		`-#else`
`806`		`- return __nolibc_enosys(__func__, path, flags, mode);`
`807`		`-#endif`
`808`	`802`	`}`
`809`	`803`
`810`	`804`	`static __attribute__((unused))`