[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <mvm4mcrtski.fsf@hawking.suse.de>
Date: Mon, 29 Feb 2016 17:00:29 +0100
From: Andreas Schwab <schwab@...e.de>
To: Arnd Bergmann <arnd@...db.de>
Cc: linux-arm-kernel@...ts.infradead.org,
Yury Norov <ynorov@...iumnetworks.com>,
"Zhangjian \(Bamvor\)" <bamvor.zhangjian@...wei.com>,
pinskia@...il.com, Prasun.Kapoor@...iumnetworks.com,
catalin.marinas@....com, heiko.carstens@...ibm.com,
linux-kernel@...r.kernel.org, agraf@...e.de,
klimov.linux@...il.com, broonie@...nel.org,
jan.dakinevich@...il.com, joseph@...esourcery.com,
schwidefsky@...ibm.com, Nathan_Lynch@...tor.com,
christoph.muellner@...obroma-systems.com
Subject: Re: [RFC5 PATCH v6 00/21] ILP32 for ARM64
Arnd Bergmann <arnd@...db.de> writes:
> In https://github.com/norov/glibc/commit/5d4290435e428267171ece871539b76e1d079d11
> you are defining a struct __kernel_stat64 in the glibc. Is this the expected
> way to do it? I would have thought you'd get the definition from the kernel
> headers.
The problem really is that struct stat64 does not match the kernel
struct stat64. The latter uses the broken st_ino, where the 64-bit
inode is actually stored at the end of the structure, and there is
padding *before* the 32-bit inode, not after (so you cannot overlay a
64-bit inode field in little endian mode). You need to add the attached
file as sysdeps/unix/sysv/linux/aarch64/bits/stat.h, then you can use
the user-space struct stat64 to communicate with the *stat64 syscalls,
and __kernel_stat64 is not needed any more.
Andreas.
--
Andreas Schwab, SUSE Labs, schwab@...e.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."
View attachment "stat.h" of type "text/plain" (6893 bytes)
Powered by blists - more mailing lists