Messages in this thread Patch in this message | | | From | Tiezhu Yang <> | Subject | [PATCH v3] checksyscalls: Ignore fstat to silence build warning on LoongArch | Date | Tue, 30 Aug 2022 11:27:01 +0800 |
| |
fstat is replaced by statx on the new architecture, so an exception is added to the checksyscalls script to silence the following build warning on LoongArch:
CALL scripts/checksyscalls.sh <stdin>:569:2: warning: #warning syscall fstat not implemented [-Wcpp]
Suggested-by: WANG Xuerui <kernel@xen0n.name> Suggested-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn> ---
Hi Xuerui and Arnd,
Thank you for your reviews and suggestions.
v3: -- Move the definition to its own block with some explanation
v2: -- Change scripts/checksyscalls.sh -- Modify patch subject and commit message
scripts/checksyscalls.sh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/scripts/checksyscalls.sh b/scripts/checksyscalls.sh index f33e61a..1e5d2ee 100755 --- a/scripts/checksyscalls.sh +++ b/scripts/checksyscalls.sh @@ -114,7 +114,6 @@ cat << EOF #define __IGNORE_truncate #define __IGNORE_stat #define __IGNORE_lstat -#define __IGNORE_fstat #define __IGNORE_fcntl #define __IGNORE_fadvise64 #define __IGNORE_newfstatat @@ -255,6 +254,9 @@ cat << EOF /* 64-bit ports never needed these, and new 32-bit ports can use statx */ #define __IGNORE_fstat64 #define __IGNORE_fstatat64 + +/* Newer ports are not required to provide fstat in favor of statx */ +#define __IGNORE_fstat EOF } -- 2.1.0
| |